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

Teach dogs to stay #3718

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

Conversation

SentientSupper
Copy link
Contributor

Refresh dog duration every time you cast it to avoid situations where the dog times out immediately after you cast the spell to heal it.

Refresh dog duration every time you cast it to avoid situations where
the dog times out immediately after you cast the spell to heal it.
@DracoOmega
Copy link
Contributor

It was originally intentional that recasting to heal did not give additional duration, at least partially to avoid encouraging the player to 'top up' their dog periodically to prevent it from disappearing. I could see maybe see an argument for giving it a very brief bit of additional duration on recast so that the bonus attack the player expects to get from recasting actually happens, but am unsure about any more than that.

However, I'm additionally leery about how you are adjusting duration. Not only have you pulled out a private method that isn't otherwise used externally, but you are resetting the duration to its initial baseline it in a way that has significant randomization afterward, and so it is very possible that recasting will now decrease remaining duration sometimes.

@SentientSupper
Copy link
Contributor Author

SentientSupper commented Mar 21, 2024

I suppose there could be a cleaner way of coding this once I take another look at it.

The reason I want this is because of situations where you cast the dog for a fight, finish that fight then head to the next one, heal up your dog when the fight begins, but then have to resummon the dog because it decided to time out shortly after. This encourages you to wait for your dog to disappear at the end of each fight just so the next one you summon has a longer duration.

Possibly decreasing the duration when it's reset is intentional, as we don't want players casting it multiple times just to get a dog that lasts longer. In practice it doesn't matter much.

@SentientSupper SentientSupper marked this pull request as draft March 21, 2024 02:15
@SentientSupper SentientSupper marked this pull request as ready for review March 21, 2024 05:48
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

2 participants