Tatsh / smgen

100%

push

github

project: break the four biggest modules into packages

chart, stems, train and constraints each become a package whose __init__ does
nothing but re-export, so a module is now the size of one idea rather than of
one prefix. constraints is divided by what a rule is about: the codes a row is
written in, where a note may land, keeping the panels evenly used, what the
chart has spent, and the gate a candidate pattern passes.

Two of the packages re-export less than everything under them. Importing
smlab.chart for the note characters used to be free, and would now drag in the
models and the caching layer that sit above those same characters, which is a
cycle rather than a slow import: cache imports the vocabulary, and building
training windows imports cache. So smlab.chart re-exports the vocabulary and
smlab.stems the separation, while the heavier modules are reached by name.

isort had no first-party section, so absolute imports of this project sorted in
among the third-party ones. That went unnoticed while every internal import was
relative, and a sub-package cannot be: reaching the rest of the project from one
means an absolute import, because relative imports from a parent are barred.
Adding the section puts them back where the convention says they go.

smlab itself now re-exports the parsing, timing, playability and writing API.
Generation is not among it, because that needs the models; importing the package
still costs nothing and does not pull in torch.

Signed-off-by: Andrew Udvare <audvare@gmail.com>

688 of 688 branches covered (100.0%)

Branch coverage included in aggregate %.

273 of 273 new or added lines in 18 files covered. (100.0%)

3458 of 3458 relevant lines covered (100.0%)

3.0 hits per line