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

New Beats with UI [GSoC 2020] #2961

Draft
wants to merge 352 commits into
base: main
Choose a base branch
from

Conversation

hacksdump
Copy link
Contributor

@hacksdump hacksdump commented Jul 23, 2020

PR history:
#2861 (Continuation of #2512)
#2844

Beatgrid editing operations for editing BPM (variable) and Time Signature (variable) will be implemented in this PR. The backend and GUI will be simultaneously developed.

TODO:

  • Edit time signature at arbitrary downbeats.

  • Edit BPM at arbitrary beats.

  • Hybridise BeatGrid and BeatMap implementations to allow beat jump before track.

  • Fix TimeSignatureShouldNotChangeBpm Test.

  • Write more tests for new code.

  • Improve editing UX.

  • The leftmost beat disappears too early when waveform scrolls. The features such as markers and downbeat triangles should be visible even if the beat is not on screen.

  • Use interbeat distance to eliminate beats.

Update "New Beats with UI" PR to latest main
@Holzhaus
Copy link
Member

Holzhaus commented Mar 2, 2021

GitHub is confused and doesn't show the CI results properly. They are still accessible here: https://github.com/hacksdump/mixxx/actions/runs/614728642

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

This PR is marked as stale because it has been open 90 days with no activity.

@github-actions github-actions bot added the stale Stale issues that haven't been updated for a long time. label Jul 5, 2021
@Be-ing Be-ing removed this from the 2.4.0 milestone Nov 14, 2021
@prez
Copy link

prez commented Apr 9, 2022

I see this is no longer in the 2.4 milestone, but some features have been incorporated as other PRs - what's the timeline for downbeat markers in mixxx currently? The PR history is long and confusing from a potential contributor POV.
Thanks!

@ronso0
Copy link
Member

ronso0 commented Apr 9, 2022

You may take a look at #4489 and Jan's other PRs

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

Successfully merging this pull request may close these issues.