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

DJC hotfix's fix pr of the fix pr's fix pr #2230

Merged
merged 2 commits into from
Feb 2, 2024

Conversation

jaccob101
Copy link
Contributor

@jaccob101 jaccob101 commented Jan 29, 2024

sorry (forces dj aerials to inherit the movement arc before allowing cancels)
accidentally undid it oops
peachassets-1.zip
fixes (from first impl):
-unnecessary hitstop check during exec
-delaying check until frame 2 of aerial
-peach dj frames (changed bc i failed to realise the above issue)
note for what this does exactly:
-force djc to experience at least 1f of dj arc
-checks for cancel AFTER 2nd motion is added
-it doesn't run the check during hitstop

this combined makes it perform more accurately, but still loses more height with tapjump. no issues were noticed on other characters but could be limited to only ness/lucas if they arise

@WuBoytH WuBoytH added bug Something isn't working semver:patch labels Jan 29, 2024
@WuBoytH
Copy link
Collaborator

WuBoytH commented Jan 29, 2024

"Who are you?"
"I'm you."
"No, I'm me!"
"I'm me, he says."

Copy link

github-actions bot commented Jan 29, 2024

Download the artifacts for this pull request:

@jaccob101 jaccob101 marked this pull request as draft January 29, 2024 09:27
@jaccob101 jaccob101 marked this pull request as ready for review January 31, 2024 00:39
@O1yyy O1yyy merged commit 4d13d89 into HDR-Development:dev Feb 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working semver:patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants