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

Units bombarded down to 0hp should die #2382

Open
hugomflavio opened this issue Sep 10, 2024 · 6 comments
Open

Units bombarded down to 0hp should die #2382

hugomflavio opened this issue Sep 10, 2024 · 6 comments
Labels
bug Something isn't working server This issue requires changes to the server

Comments

@hugomflavio
Copy link
Contributor

hugomflavio commented Sep 10, 2024

Describe the bug
Extensively bombarded units inside a city. To the point they should have 0hp left - especially since they have 0 defence. But the ground troops who then ended them reported winning without taking out any hp. see screenshot below.

To Reproduce
Steps to reproduce the behavior:

  1. Let an idle city produce 10+ raiders so their defence becomes 0 (idk the actual endpoint that makes i t0)
  2. bombard that city with 10 archers, unload them all.
  3. attack the city with your own raiders after.
  4. See error

Expected behavior
defenders should have 1hp left?
edit: they should die when bombarded down to 0

Screenshots
image

Platform and version (please complete the following information):

  • OS: Linux
  • Freeciv21 version: 3.1-beta.2
  • Ruleset/Longturn game (if applicable): Sim06

Additional context
noticed during the test game

Edit:

@hugomflavio hugomflavio added bug Something isn't working Untriaged This issue or PR needs triaging labels Sep 10, 2024
@hugomflavio
Copy link
Contributor Author

(could be a bug in the message itself, of course)

@hugomflavio
Copy link
Contributor Author

not related to the 0 defence. But it seems that somehow if one bombards units hard enough, they can accidentally get to 0hp? Need to test with eyes on both sides to understand what's going on.

@hugomflavio hugomflavio changed the title units with 0 defence die without losing hp? heavily bombarded units may be getting to 0hp without dying? Sep 10, 2024
@hugomflavio
Copy link
Contributor Author

image

Confirmed units are getting to 0hp from bombarding. Is this expected?

@lmoureaux
Copy link
Contributor

No. I suppose Bombard_Limit_Pct is missing from the ruleset and +Bombard_Limit_Pct is set.

Bombarded units with 0hp should die.

@Corbeau-1
Copy link
Contributor

I can confirm that this does happen, units do get to 0 HP while surviving. Technically, this is what bombardment does: bombs them to oblivion without physically destroying them. Now, we may discuss if this is the desired behaviour and, if not, what to do and how to do it. I'd be in favour of defining some minimal HP value that units can be bombed to (maybe make it an effect so that it depends on the unit, terrain, veterancy...) but I'll survive if the current condition remains.

@lmoureaux lmoureaux changed the title heavily bombarded units may be getting to 0hp without dying? Units bombarded down to 0hp should die Sep 10, 2024
@lmoureaux lmoureaux added server This issue requires changes to the server and removed Untriaged This issue or PR needs triaging labels Sep 10, 2024
@lmoureaux
Copy link
Contributor

Edited the description to include the results of our investigation on Discord. Units bombarded down to 0hp should die, just like units reaching 0hp in combat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working server This issue requires changes to the server
Projects
None yet
Development

No branches or pull requests

3 participants