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

Issue #5543 (nation happiness from misc_mission) #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Namerutan
Copy link
Contributor

  • Should fix Issue #5543 (nation happiness from misc_mission, like in mansion map)

  • Performance for missions is now composed of 4 factors, instead of just 2; added factors for MissionGoal and InfectedHumans (just basic implementation yet).

  • Added 2 new fields to missionResults_t (float goalAccomplished, int infectedHumans), to prepare for new score calculations in future updates.

  • Performance from killed aliens is now always zero or positive (small value), so the player will not get a negative score when completing missions different than simple xenocide; this was the main reason for the issue #5543, a negative value from not killed aliens a lot higher than the positive score from saved civilians.

  • Moved all used constant values into the .h header file to gather all them, easing if they ever become scriptable.

Signed-off-by: Namerutan namerutan@hotmail.com

Signed-off-by: Namerutan <namerutan@hotmail.com>
@DarkRainX
Copy link
Collaborator

Since geever is the campaign master I'll defer to him, he doesn't have a github account I think, so you might want to ping him on IRC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants