Skip to content
This repository was archived by the owner on Nov 30, 2020. It is now read-only.

Move matrix generation code to its own module #15

Open
js-jslog opened this issue May 24, 2020 · 1 comment
Open

Move matrix generation code to its own module #15

js-jslog opened this issue May 24, 2020 · 1 comment

Comments

@js-jslog
Copy link
Owner

There's a lot of similarity between the matrix generation logic.

It's possible that they should all be in the same module, as a sibling to Pitch, Degree and Pozition.

It would be good if those 3 modules could all subscribe to an interface of exported functions for getting ordered lists of pitches, degrees and pozitions. Then it would be clear that they are all partner contributers to the matrix generation functions.

While we're at it, those listing functions could be in OrderedPitch, OrderedDegree and OrderedPozition modules, perhaps.

@js-jslog
Copy link
Owner Author

js-jslog commented Jun 4, 2020

#36

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant