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

Round after prime mapping not in prime mapping #755

Closed
frostburn opened this issue Jun 15, 2024 · 0 comments · Fixed by #758
Closed

Round after prime mapping not in prime mapping #755

frostburn opened this issue Jun 15, 2024 · 0 comments · Fixed by #758
Labels
bug Something isn't working

Comments

@frostburn
Copy link
Member

https://sw3.lumipakkanen.com/scale/x5igOPjMt

galaxyspoobToday at 12:37 PM

In SW3, when CTE tempering is applied where the equave is split, the fraction of the equave (the half-octave in the below example) is sometimes off by 1 minimum unit of interval size, no matter the setting of cents digits after the decimal point.

E.g. when https://sw3.lumipakkanen.com/scale/x5ijC5mUP is tempered by {1716/1715 2080/2079}, 99/70 turns into 599.999, 599.9999, or 600.00001c for the 4-, 5-, and 6-digit settings.
(The JI scale was made by writing the first 5 notes, then Repeat, then the manual replacement of 9801/4900 by 2/1.)
(Raw version): https://sw3.lumipakkanen.com/scale/x5igaO9ML
Ah, so that occurs because PrimeMapping is rounded.

@frostburn frostburn added the bug Something isn't working label Jun 15, 2024
frostburn added a commit that referenced this issue Jun 15, 2024
frostburn added a commit that referenced this issue Jun 16, 2024
Update xen-dev-utils, moment-of-symmetry and sonic-weave to get addition of too accurate cents to coearce to reals.

ref #755
@frostburn frostburn linked a pull request Jun 16, 2024 that will close this issue
frostburn added a commit that referenced this issue Jun 16, 2024
Update xen-dev-utils, moment-of-symmetry and sonic-weave to get addition of too accurate cents to coearce to reals.

ref #755
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant