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

Scorched turfs are sometimes incorrect when map edited #5754

Open
3 tasks done
Drulikar opened this issue Feb 19, 2024 · 0 comments
Open
3 tasks done

Scorched turfs are sometimes incorrect when map edited #5754

Drulikar opened this issue Feb 19, 2024 · 0 comments
Labels
Bug Admin events aren't bugs Mapping did you remember to save in tgm format?

Comments

@Drulikar
Copy link
Contributor

Testmerges

#5007: 'Project ARES TM Holder (v4)' by realforest2001 at commit 6c41b6a #5392: 'Yautja Weapon Gimmicks, Round Two' by realforest2001 at commit f164a72 #5584: 'Jumping out of a dropship heading towards an LZ now throws your body at the ground, instead of deleting it.' by Contrabang at commit fec4de9 #3633: 'Black Goo Patch' by realforest2001 at commit 9e7c29b #5401: 'M540-A Armored Recon Carrier' by Zonespace27 at commit d8e390d #5432: 'Abomination rework and predator plasma rifle buff.' by InsaneRed at commit a7a2e82 #5657: 'Reactor changes' by BeagleGaming1 at commit 98c5e34

Round ID

20949

Description of the bug

Map var edited turfs can have the incorrect scorched state such as here on LV624:
image

What's the difference with what should have happened?

In this case since there's no grass to scorch, it ultimately should not be scorchable.

Ultimately the var edits like this on the map probably should use a subtype rather than just edited.

However, another aspect that is funky is that the var scorched is defined to be treated like a bool; but its half used purely as a bool, and other times used to determine a mask to use. So likely the comment for the var is incorrect.

How do we reproduce this bug?

  1. Go to 146,102, 2 on LV624
  2. Ignite the ground
  3. Notice it suddenly turns to grass
  4. Wait for fire to burn out
  5. Repeat 1-2 once or twice more
  6. Observe broken sprite

Issue Bingo

@Drulikar Drulikar added Bug Admin events aren't bugs Mapping did you remember to save in tgm format? labels Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs Mapping did you remember to save in tgm format?
Projects
None yet
Development

No branches or pull requests

1 participant