Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scout trait not working properly #42853

Closed
ConnorMartin opened this issue Aug 10, 2020 · 2 comments · Fixed by #42923
Closed

Scout trait not working properly #42853

ConnorMartin opened this issue Aug 10, 2020 · 2 comments · Fixed by #42923
Labels
<Bug> This needs to be fixed Mechanics: Character / Player Character / Player mechanics Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies

Comments

@ConnorMartin
Copy link
Contributor

ConnorMartin commented Aug 10, 2020

Even with the scout trait, my character's map vision is still 5 tiles.

Steps To Reproduce

Create a character with the scout trait
Walk around for a little while on a field
Observe the 5-tile map vision (should be 10 with scout)

Expected behavior

Scout states that it improves map vision range, so the trait should do that.

Screenshots

image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1903
  • Game Version: 0.E-4942-gc8ac471 [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Alternative Map Key [alt_map_key],
    Aftershock [aftershock]
    ]

Additional context

@wapcaplet
Copy link
Contributor

wapcaplet commented Aug 15, 2020

@ConnorMartin Did you happen to also have the "Self-Aware" trait? I can confirm this issue, but only when I also have "Self-Aware".

The counterpart of the "Scout" trait that should reduce overmap sight range ("Topographagnosia") also has no apparent effect.

With "Scout" and "Self-Aware", overmap vision is 5 overmap tiles:

image

With "Topographagnosia" and "Self-Aware", overmap vision is still 5 overmap tiles:

image

However, without the "Self-Aware" trait, I am getting a distinct increase (~7 tiles) with "Scout", and a significant decrease (~2-3 tiles) with "Topographagnosia":

image

I don't know if this is intended behavior, but it's pretty strange.

Edit I have figured out how to fix this, and will submit a PR shortly.

  • OS: Linux
    • OS Version: Distributor ID: Pop; Description: Pop!_OS 20.04 LTS; Release: 20.04; Codename: focal;
  • Game Version: 0.E-5007-g80e32f911b [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food]
    ]

wapcaplet added a commit to wapcaplet/Cataclysm-DDA that referenced this issue Aug 15, 2020
Correct the "overmap_sight" mutation value map to be additive, rather
than multiplicative. Include a regression test that failed before this
change, and passes afterward.

Fix CleverRaven#42853

For reasons I do not understand, using "multiplicative" here caused the
"Self-Aware" trait to nullify the overmap sight adjustments from both
the "Scout" and "Topographagnosia" traits.
feinorgh pushed a commit to feinorgh/Cataclysm-DDA that referenced this issue Aug 15, 2020
Correct the "overmap_sight" mutation value map to be additive, rather
than multiplicative. Include a regression test that failed before this
change, and passes afterward.

Fix CleverRaven#42853

For reasons I do not understand, using "multiplicative" here caused the
"Self-Aware" trait to nullify the overmap sight adjustments from both
the "Scout" and "Topographagnosia" traits.
feinorgh pushed a commit to feinorgh/Cataclysm-DDA that referenced this issue Aug 15, 2020
Correct the "overmap_sight" mutation value map to be additive, rather
than multiplicative. Include a regression test that failed before this
change, and passes afterward.

Fix CleverRaven#42853

For reasons I do not understand, using "multiplicative" here caused the
"Self-Aware" trait to nullify the overmap sight adjustments from both
the "Scout" and "Topographagnosia" traits.
@wapcaplet
Copy link
Contributor

Worth repeating here from comments on #42923 - the "Self-Aware" trait was a red herring; it turns out the "Scout" visibility range was nullified by having any other trait; in other words, "Scout" only worked if it was the only trait you had. Problems like this may be mitigated in the future by this change to mutation/trait modifiers: #42932

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Mechanics: Character / Player Character / Player mechanics Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants