Skip to content

Freeplay DJ bops to BPM - #3606

Closed
KutikiPlayz wants to merge 1 commit into
FunkinCrew:developfrom
KutikiPlayz:freeplay-dj-bops-to-bpm
Closed

Freeplay DJ bops to BPM#3606
KutikiPlayz wants to merge 1 commit into
FunkinCrew:developfrom
KutikiPlayz:freeplay-dj-bops-to-bpm

Conversation

@KutikiPlayz

@KutikiPlayz KutikiPlayz commented Oct 6, 2024

Copy link
Copy Markdown
Contributor

Implements #2760

The Freeplay DJ idle animation just looped before, now it plays with the bpm of whatever song is selected.
Also the backingCard had stuff to do on beatHit but the Conductor wasn't being updated so that works as well now.

Before:

before.mp4

After:

after.mp4

@github-actions github-actions Bot added pr: haxe PR modifies game code. size: medium A medium pull request with 100 or fewer changes. labels Oct 6, 2024
@KutikiPlayz KutikiPlayz changed the title Freeplay DJ Bops to BPM Freeplay DJ bops to BPM Oct 6, 2024
@Hundrec

Hundrec commented Oct 6, 2024

Copy link
Copy Markdown
Member

This is super cool and I want to see it in the game!
I also really hope it doesn't break anything

@JackXson-Real

Copy link
Copy Markdown
Contributor

This is 100% needed

@amyspark-ng

Copy link
Copy Markdown
Contributor

oh this is very cool...
fixes #2760

@KutikiPlayz

Copy link
Copy Markdown
Contributor Author

thank you, I tried searching freeplay dj but didn't find anything under that specifically so yeah

@Hundrec

Hundrec commented Oct 7, 2024

Copy link
Copy Markdown
Member

I wonder why this doesn't show as linked to #2760 after your edit, at least for me

@amyspark-ng

Copy link
Copy Markdown
Contributor

thank you, I tried searching freeplay dj but didn't find anything under that specifically so yeah

i didn't know it was called dj back then lol!

@amyspark-ng

amyspark-ng commented Oct 7, 2024

Copy link
Copy Markdown
Contributor

I wonder why this doesn't show as linked to #2760 after your edit, at least for me

not sure, if this is merged and it does link that issue will the issue close when this gets merged?

@Hundrec

Hundrec commented Oct 7, 2024

Copy link
Copy Markdown
Member

I'm curious to find out! I'll let you know if that happens with any of my PRs in funkin.assets

@KutikiPlayz

Copy link
Copy Markdown
Contributor Author

maybe it doesn't like it if I try to add it in an edit

@Hundrec

Hundrec commented Oct 7, 2024

Copy link
Copy Markdown
Member

Weird, I think I've edited an issue in before, and that was from funkin.assets

@NotHyper-474

NotHyper-474 commented Oct 7, 2024

Copy link
Copy Markdown
Collaborator

I wonder why this doesn't show as linked to #2760 after your edit, at least for me

It only links it if the base branch is the default branch (main), but changing from the default branch to another doesn't unlink it.

@ninjamuffin99

Copy link
Copy Markdown
Member

might make a change where pico doesn't bop / spin to beat, or other lil cleanings, but generally this looks yummy and I think we had something like this in progress but swapped over to other tasks, very epic will poke at this :)

@AbnormalPoof

AbnormalPoof commented Oct 8, 2024

Copy link
Copy Markdown
Member

might make a change where pico doesn't bop / spin to beat

I recommend adding a boolean to the Player data (somewhere in freeplayDJ like danceToBeat?) so it isn't hardcoded to just Pico!

@EliteMasterEric EliteMasterEric added the status: pending triage Awaiting review. label Oct 8, 2024
@KutikiPlayz

Copy link
Copy Markdown
Contributor Author

I was thinking of doing a -hold animation type thing so that the records can still spin if the bpm is too slow, but that requires stuff on the assets side and idk if I can do that easily myself since it's not a sparrow sprite, besides the code for using a -hold anim once the main one finishes

@KutikiPlayz

Copy link
Copy Markdown
Contributor Author

I recommend adding a boolean to the Player data (somewhere in freeplayDJ like danceToBeat?) so it isn't hardcoded to just Pico!

I can do that

@KutikiPlayz

Copy link
Copy Markdown
Contributor Author

alright it's there now
it's gonna say there's a bunch of other changes in assets but that's cuz I think they're doing other stuff in develop that aren't on the main assets branch

@Cartridge-Man

Copy link
Copy Markdown
Contributor

Ngl this is pretty cool looking, hope this can be tested to make sure nothing breaks and then eventually maybe will get merged in the next update

@github-actions github-actions Bot added pr: documentation PR modifies documentation or README files. size: large A large pull request with more than 100 changes. and removed size: medium A medium pull request with 100 or fewer changes. labels Oct 12, 2024
Comment thread source/funkin/ui/freeplay/FreeplayState.hx Outdated
@github-actions github-actions Bot added size: medium A medium pull request with 100 or fewer changes. and removed size: large A large pull request with more than 100 changes. labels Nov 29, 2024
@KutikiPlayz
KutikiPlayz marked this pull request as draft November 29, 2024 23:36
@KutikiPlayz
KutikiPlayz marked this pull request as ready for review November 29, 2024 23:43
@EliteMasterEric EliteMasterEric removed the status: pending triage Awaiting review. label Jan 17, 2025
@github-actions github-actions Bot added pr: documentation PR modifies documentation or README files. pr: github PR modifies GitHub metadata files. and removed pr: documentation PR modifies documentation or README files. pr: github PR modifies GitHub metadata files. labels Apr 6, 2025
@KutikiPlayz

Copy link
Copy Markdown
Contributor Author

all better 👍

@Hundrec Hundrec added status: pending triage Awaiting review. and removed status: needs revision Cannot be approved because it is awaiting some work by the contributor. labels Apr 6, 2025
@Hundrec Hundrec added topic: art and animation Related to visual details and errors in the game. topic: freeplay Related to the operation of the Freeplay menu. and removed topic: polish Involves minor polish to the UI or gameplay. labels Apr 30, 2025
@EliteMasterEric EliteMasterEric added size: medium A medium pull request with 100 or fewer changes. and removed size: medium A medium pull request with 100 or fewer changes. labels May 14, 2025
@AbnormalPoof AbnormalPoof added status: needs revision Cannot be approved because it is awaiting some work by the contributor. and removed status: pending triage Awaiting review. labels May 31, 2025
@Hundrec Hundrec added status: pending triage Awaiting review. and removed status: needs revision Cannot be approved because it is awaiting some work by the contributor. labels Jun 5, 2025
@AbnormalPoof AbnormalPoof added status: needs revision Cannot be approved because it is awaiting some work by the contributor. and removed status: pending triage Awaiting review. labels Jul 29, 2025

@AbnormalPoof AbnormalPoof left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please resolve the merge conflicts on this PR within 90 days.

@Hundrec Hundrec added status: rejected Issue did not pass review or PR cannot be approved. and removed status: needs revision Cannot be approved because it is awaiting some work by the contributor. type: enhancement Involves an enhancement or new feature. pr: haxe PR modifies game code. topic: art and animation Related to visual details and errors in the game. topic: freeplay Related to the operation of the Freeplay menu. labels Sep 3, 2025
@github-actions github-actions Bot closed this Sep 3, 2025
@Hundrec

Hundrec commented Sep 3, 2025

Copy link
Copy Markdown
Member

This was a cool concept, but Dave says that in practice the DJ took away too much focus from the song capsules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: medium A medium pull request with 100 or fewer changes. status: rejected Issue did not pass review or PR cannot be approved.

Development

Successfully merging this pull request may close these issues.

Enhancement: Make the Freeplay DJ bop at the bpm of the current song that is being previewed

9 participants