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

Don't Stop ( TheFatRat Remix ) #18

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

DeadlyBoizZ
Copy link

Chart Verification PR

Foster the People - Don't Stop ( TheFatRat Remix )

album

Instruments and difficulties charted

Instrument Easy Medium Hard Expert
Drums No No No Yes
Bass No No No Yes
Lead No No No Yes
Vocals No No No Yes
Pro Drums No No No No
Pro Bass No No No No
Pro Lead No No No No

Additional details

None

@tposejank tposejank added the pending review Song is pending review label Jul 7, 2024
@NarrikSynthfox
Copy link
Contributor

While the song is 133 bpm, the count-in needs to be adjusted to be at 133 as well, as well as having the song actually start at the start of a measure.

There are many notes that are off beat.

The vocals have chords when there aren't any.

Generally, note placement should follow the pitch, unless it would cause it to be hard to play on gamepad in which case adding lifts or changing the pattern to alternate between left and right thumbs on a controller is the best thing to do.

You need to account for gamepad players as well, not just keyboard players, so alternating between lanes 1/2 or any combination of 3/4/5 quickly makes it very hard to play on gamepad. Same with ascending and descending patterns, if they're too quick then alternate between left and right thumbs.

@NarrikSynthfox NarrikSynthfox added needs revising Edits are necessary and removed pending review Song is pending review labels Jul 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs revising Edits are necessary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants