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

Fix planet stealth tech cost #4128

Merged
merged 1 commit into from Sep 18, 2022
Merged

Conversation

o01eg
Copy link
Contributor

@o01eg o01eg commented Sep 9, 2022

Fixes #4127

@o01eg o01eg added category:bug The Issue/PR describes or solves a perceived malfunction within the game. component:content scripting The Issue/PR deals with the FOCS language, turn events or the universe generator. labels Sep 9, 2022
@o01eg o01eg added this to the v0.5 milestone Sep 9, 2022
@o01eg o01eg self-assigned this Sep 9, 2022
@Grummel7
Copy link
Contributor

Just had a closer look at it:
Pedia says, cost is reduced by cost of the ship part, which whoever is wrong for when you have a sneaky species. The sneaky reduction is missing.
Actually what the code does is it halfs the cost if:

  • ship part has been researched
  • empire owns a sneaky species.

I suggest to update the pedia accordingly and also implement it, using / (1 + Statistic If... notation for both.

@o01eg
Copy link
Contributor Author

o01eg commented Sep 18, 2022

@Grummel7 I suppose it should be other PR.

@o01eg o01eg added the status:merged All relevant commits of this PR were merged into the master development branch. label Sep 18, 2022
@o01eg o01eg merged commit 30ce9e0 into freeorion:master Sep 18, 2022
@o01eg o01eg deleted the fix-techs-stealth-cost branch September 18, 2022 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:bug The Issue/PR describes or solves a perceived malfunction within the game. component:content scripting The Issue/PR deals with the FOCS language, turn events or the universe generator. status:merged All relevant commits of this PR were merged into the master development branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong tech costs for planet stealth
2 participants