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

Applied some animation corrections for buildings when player is on low powered. #728

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Shin-Aska
Copy link
Contributor

Modified the following rules:

  1. Allied Construction Yard(gacnst)
  2. Allied Airforce Command Headquarters (gaairc)
  3. Allied Ore Purifier (gaorep)

Changed constructions yard and airforce command to not pause the animation when it is on low power (This is how it works on the original game)
Changed the ore purifier to halt on animation when it is on low power (This is how it works on the original game)

This pull request is related to the issue filed in #727

1. Allied Construction Yard(gacnst)
2. Allied Airforce Command Headquarters (gaairc)
3. Allied Ore Purifier (gaorep)

Changed constructions yard and airforce command to not pause the animation when it is on low power (This is how it works on the original game)
Changed the ore purifier to halt on animation when it is on low power (This is how it works on the original game)
@abcdefg30
Copy link
Member

Tbh, not sure if pausing the Airforce HQ isn't a bugfix over the original; will have to think about that. (Doesn't it look a bit weird when that doesn't pause but the Spy Sat does?)

@Shin-Aska
Copy link
Contributor Author

Shin-Aska commented May 25, 2020

From what I originally understood, if the animation pauses, it is a visual indicator that you can't use it. However, Westwood is inconsistent on this rule because Ore purifier for example, has its animation paused but the passive effect still remains in effect regardless.

For the airforce command, I think they didn't let it pause because you can still use the aircrafts? But then again this could also be one of the inconsistencies too.

@penev92
Copy link
Member

penev92 commented Mar 23, 2023

  1. Thanks for the very detailed research and presentation that you did.
  2. At this point saying "sorry for the insane delay" is probably redundant.
  3. On the changes:
  • I am definitely with @abcdefg30 on the Airforce HQ disabling being an improvement over the original.
  • I feel a bit less strongly about the ConYard, but it still almost makes sense to me to stop the fans (wouldn't it be cool if we could instead slow them down? Solving the dilemma of "do we stop them if the building still works, although slower due to low power?").
  • As for the Ore Purifier I'm torn because if it continues to work (I wouldn't be surprised if we disable the cash effect too for balance?) it should probably keep the visual effect so the player has some feedback...

@penev92
Copy link
Member

penev92 commented Apr 6, 2023

So what do people think about slowing down animations on structures whose effects get slowed down on low power to reflect it? (like production structures)
This isn't specific to this PR and would affect a bunch more structures and even may not be a bad thing to extend to other mods, btw.

@abcdefg30
Copy link
Member

Why slow down instead of stopping?

@penev92
Copy link
Member

penev92 commented Apr 6, 2023

To better reflect what happens to the underlying effect - some structures stop working (radar) and some slow down (production). So we can stop the animation on some and slow down the animation on others to make that clear.
I think it could be a nice bit of polish.

@abcdefg30
Copy link
Member

Ok that makes sense to me. (As long as it does look good ingame, 👍 )

@Shin-Aska
Copy link
Contributor Author

So what do people think about slowing down animations on structures whose effects get slowed down on low power to reflect it? (like production structures) This isn't specific to this PR and would affect a bunch more structures and even may not be a bad thing to extend to other mods, btw.

Hi!

Sounds good to me as well, I'll try to apply the changes tonight or tomorrow.

…tion named idle-fans-low-power which is just like idle-fans but with larger ticks to make the animation appear slower
…ction yard's fan animation slow down when low power and reverted animation pauses on Airforce command and Ore Purifier
@Shin-Aska
Copy link
Contributor Author

Shin-Aska commented Apr 6, 2023

@penev92 and @abcdefg30 i reverted changes 2 and 3 to what it originally was and as for 1, I made some changes base on the discussion where if player is in LOWPOWER state, then it will slow down the fans.

Copy link
Member

@penev92 penev92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll need to test, but the way it is set up looks correct.
On a side note, would you be willing to do this for other production structures that have idle (or production) animations?

mods/ra2/rules/allied-structures.yaml Outdated Show resolved Hide resolved
Added safeguard lowpower condition in the Sequence idle-fans-low-power

Co-authored-by: Pavel Penev <penev92@gmail.com>
@Shin-Aska
Copy link
Contributor Author

I'll need to test, but the way it is set up looks correct. On a side note, would you be willing to do this for other production structures that have idle (or production) animations?

Sure, I just need to know which structures need this implementation.

@penev92
Copy link
Member

penev92 commented Apr 7, 2023

Actually taking a quick look I can't spot any other candidates.

Copy link
Member

@penev92 penev92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍
Thank you!

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.

None yet

3 participants