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

added \timeScale spec #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

added \timeScale spec #56

wants to merge 1 commit into from

Conversation

JVanBuskirk
Copy link

envelope_rel.scd was not rescale to the event time. I added copied the \timeScale spec from line_rel.scd. It seems to fix the problem.

@woutersnoei
Copy link
Contributor

Hi JVanBuskirk, thanks for the pull request. I'm not sure though how and if this fix would solve the problem. What it effectively does is make the timeScale arg init rate instead of control rate, and I can't see in the source code why that would make any difference. The problem with the two _rel UMapDefs is that sometimes their 'u_dur' arg is not set correctly. This happens to both of them in some cases where the event is copied or immediately after the UMap is added to a chain. I've done some work on solving this a while ago, but perhaps there are still cases where it happens. Could you maybe explain when the envelope_rel did and did not rescale? Or was it all the time :-)?

@JVanBuskirk
Copy link
Author

Hello. Now I can't recreate it. It was not working for envelope_rel.scd consistently in one session. After I changed the file to match line_rel, it worked. I reinstalled Unit-Lib everything works as expected. Sorry for the noise.

I did have one issue where the envelope did not update after editing the graphic envelope. I closed the gui. When I reopened it all of my edits were gone, but I could not recreate this issue after this first occurrence either.

Unit-Lib is outstanding. I will be using it in my computer music class this fall.

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.

2 participants