Dan Hemberger

Is #538 a duplicate of this?

Okay, so if I'm understanding you correctly, I won't make a PR at this time. Please feel free to assign this issue to me in the future, if you'd like!

Thanks for the quick response! I can certainly appreciate that mutation testing may be the only 100% reliable way to assess code coverage for match arms, but that's asking a...

Thanks for all the effort everyone put into this issue!

Original colors:  New colors: 

The requested changes have been made. I'm not a huge fan of the name I chose, `SmrPlayer.getPlayerRelationship`, but everything else I could think of seemed like it would be too...

One additional thought I had: should we make the same color changes to the local map? It might be confusing to have the two be different.

The local map does in fact check for friendly/enemy/neutral players [here](https://github.com/smrealms/smr/blob/d526cb05c81f00f93267837816f11c16cf1a8a91/templates/Default/engine/Default/includes/SectorMap.inc#L97-L107). So unless I hear otherwise, I'll go ahead and re-use the logic from the current sector in the local...

The code change for the local map won't affect performance. However, this is actually a little more complicated than I expected. In the piece of code I linked above ([here](https://github.com/smrealms/smr/blob/d526cb05c81f00f93267837816f11c16cf1a8a91/templates/Default/engine/Default/includes/SectorMap.inc#L97-L107)),...

I stopped working on this for the exact reason you pointed out -- I couldn't decide how this should behave on the Galaxy Map. I think before we make any...