Skip to content

Feature: ONCE launch mode for clips - #1018

Merged
m-m-adams merged 17 commits into
SynthstromAudible:communityfrom
soymonitus:monitus/once_play_mode
Jan 25, 2024
Merged

Feature: ONCE launch mode for clips#1018
m-m-adams merged 17 commits into
SynthstromAudible:communityfrom
soymonitus:monitus/once_play_mode

Conversation

@soymonitus

@soymonitus soymonitus commented Jan 19, 2024

Copy link
Copy Markdown
Collaborator

Similar to FILL launch mode but starts at beat 1 and auto-mutes after playing once.

Tasks:

  • Implement the basics
  • Write documentation in the community features document
  • Test and test to find edge cases

@soymonitus
soymonitus marked this pull request as draft January 19, 2024 09:12
@soymonitus
soymonitus marked this pull request as ready for review January 20, 2024 18:40
@soymonitus soymonitus changed the title [Draft] ONCE launch mode for clips Feature: ONCE launch mode for clips Jan 20, 2024
@soymonitus

soymonitus commented Jan 20, 2024

Copy link
Copy Markdown
Collaborator Author

@m-m-adams @entzmingerc Ready for testing and review. Tested several edge cases and both Row and Grid modes. At worst it will have the same limitations as Fill clips, but I made sure it works as good as the Fill clips regarding instant launches, section launch launches, cloned clips, etc

# Conflicts:
#	src/deluge/model/clip/clip.h
#	src/deluge/playback/mode/session.cpp
@soymonitus

Copy link
Copy Markdown
Collaborator Author

Had to solve merge conflicts due to latest change of launchstyle to enum.
The feature is complete and tested. Could someone review the PR?

# Conflicts:
#	docs/community_features.md
#	src/deluge/playback/mode/session.cpp
Comment thread src/deluge/model/song/song.cpp Outdated
Comment thread src/deluge/model/song/song.cpp
Comment thread src/deluge/playback/mode/session.cpp Outdated
Comment thread src/deluge/playback/mode/session.cpp Outdated
Comment thread src/deluge/gui/context_menu/launch_style.cpp Outdated
Comment thread src/deluge/gui/context_menu/launch_style.cpp Outdated
Comment thread src/deluge/gui/context_menu/launch_style.cpp Outdated
Comment thread src/deluge/playback/mode/session.cpp
@soymonitus

Copy link
Copy Markdown
Collaborator Author

@m-m-adams Answered all questions and applied patch from stellar-aria to improve menu items

@m-m-adams
m-m-adams added this pull request to the merge queue Jan 25, 2024
Merged via the queue into SynthstromAudible:community with commit ae7900e Jan 25, 2024
@soymonitus
soymonitus deleted the monitus/once_play_mode branch January 26, 2024 08:33
tastycode pushed a commit to tastycode/DelugeFirmware that referenced this pull request Jul 3, 2024
* Initial commit. Created all the enums and UI handling

* Play clip once if set that way

Currently the clip loops and plays forever if:
- Armed before deluge has started playing
- You start playing it wih Shift + Arm

* Clip also plays once even when solo’ed

* dbt format

* undo this change. not necessary

* Disable instant launch for ONCE clips

* Fixed several edge cases

* Update communituy documentation

* Updated launchstyle option

* Simplified if conditions

* Simplify condition
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.

3 participants