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

TRANSIT - add phase input #182

Closed
stoermelder opened this issue Sep 8, 2020 · 5 comments
Closed

TRANSIT - add phase input #182

stoermelder opened this issue Sep 8, 2020 · 5 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@stoermelder
Copy link
Owner

No description provided.

@stoermelder stoermelder added the enhancement New feature or request label Sep 8, 2020
@stoermelder stoermelder self-assigned this Sep 8, 2020
@xandramax
Copy link
Contributor

This patch crashes Rack on my end, as soon as Transit is right-clicked and the Port CV mode is changed from "Trigger forward" to "Scan".

Oddly, Rack's log appears as if no crash happened.

@stoermelder
Copy link
Owner Author

Thanks, I missed that, fixed in c886e05...

@xandramax
Copy link
Contributor

xandramax commented Dec 6, 2020

I'm seeing behavior where when Port CV mode is set to Scan, and the Port CV input is not connected, clicking on snapshots to load them in Read mode no longer works. Also, in this case shift-click to load in Write mode doesn't work either. Snapshot loading works as expected as long as Port CV mode is not set to Scan.

I understand Port Scan CV should override manual shapshot loading, but the port should probably be ignored if it isn't connected.

For write mode, maybe Scan CV could be temporarily ignored in order to allow for editing without disconnecting the CV or changing the Port CV mode? Algomorph behaves something like this, freezing the visual state in edit mode, though clearly in the case of Transit it isn't possible to separate the audio from the visual (in terms of the state of mapped parameters).

For Read mode, another possible direction would be to have the Scan CV and manual selection act together in an additive way. Manually selecting a snapshot would then act like an offset, changing which snapshot corresponds to 0V at the Scan CV input. Again this is similar to the way Algomorph works. but maybe it doesn't fit as well here.

In any case, I'm really enjoying this enhancement. In the beta 3 branch for Algomorph, I've added a Phase output. This way, other modules can track Algomorph's phase even when multiple morph CV inputs are wrapping things around past the edges (another new feature: no internal limit on morph amount). Now together with Transit, the operators themselves can easily morph in lock-step with the algorithm morphing. Really cool!

Here's a demo patch. It requires Algomorph Beta 3.

@stoermelder
Copy link
Owner Author

I'm seeing behavior where when Port CV mode is set to Scan, and the Port CV input is not connected, clicking on snapshots to load them in Read mode no longer works. Also, in this case shift-click to load in Write mode doesn't work either. Snapshot loading works as expected as long as Port CV mode is not set to Scan.

I understand Port Scan CV should override manual shapshot loading, but the port should probably be ignored if it isn't connected.

Yes, good point, I modified it behaving this way a few moments ago.
I will check and respond to the other suggestions later.

@stoermelder stoermelder reopened this Dec 6, 2020
@stoermelder stoermelder added this to the v1.9 milestone Dec 12, 2020
@stoermelder stoermelder changed the title Transit - add phase input TRANSIT - add phase input Jun 3, 2021
@stoermelder
Copy link
Owner Author

I'm closing this issue and will pursue any further enhancements in a new issue (which will be considered in a future update).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants