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

DH Audit - Polygons #80

Open
christianscarlett opened this issue Jan 11, 2022 · 2 comments
Open

DH Audit - Polygons #80

christianscarlett opened this issue Jan 11, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@christianscarlett
Copy link
Contributor

Polygons

Assessment

The concept of this tool seems pretty complete. Besides a few bugs, it was pretty easy to get started making sounds with it in a way that made sense.

The frequencies can get high very quickly and can become too high to hear with a human ear. The “restrict frequencies” toggle helps, but maybe we want to reconsider how the pitch is chosen for each side.

The settings largely make sense except for the “Base frequency” setting. I see the tooltip that says it is the frequency of the first pitch, but does that mean the rest of the pitches are determined relative to the first? Maybe it would make more sense if it was clear in the UI how each pitch is determined.

Ideas

The “Unsynced” message should be on every tool, it’s very helpful.

The instrument picker should appear on the page the same way it does on all the other tools.

Does each successive pitch have to increase monotonically? Would be cool to experiment generating melodies with the polygons (angle size/direction could determine interval, for example)

@christianscarlett christianscarlett added the enhancement New feature or request label Jan 11, 2022
@ryaanahmed
Copy link
Member

Instead of multiplying a base frequency times the angle factor, let's try creating ratios in the range [1, 2], so we get divisions of an octave.

@ryaanahmed
Copy link
Member

Or instead of ratios, we can treat each side segment as a fraction of the whole length of the perimeter, where the perimeter represents an octave by default (but could be adjusted to a larger interval).

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