Remove star-imports

[akx]

The liberal use of star-imports makes miditoolkit export quite a lot of names that probably shouldn't be.

>>> import miditoolkit as m
>>> dir(m)
['BLACK_KEY_SATUR', 
 'CHROMA_CMAP', 
 'ControlChange', 
 'DEFAULT_BPM', 
 'Instrument', 
 'KeySignature', 
 'Lyric', 
 'Marker', 
 'MidiFile', 
 'NOTE_CMAP', 
 'Note', 
 'OFFSET_OCTAVE', 
 'PITCH_RANGE', 
 'PITCH_TO_NAME', 
 'PR_CMAP', 
 'Pedal', 
 'PitchBend', 
 'SM_CMAP', 
 'TempoChange', 
 'TimeSignature', 
 'WHITE_KEY_SATUR', 
 'XLABEL_FONT_SIZE', 
 'YLABEL_FONT_SIZE', 
 '__builtins__', 
 '__cached__', 
 '__doc__', 
 '__file__', 
 '__loader__', 
 '__name__', 
 '__package__', 
 '__path__', 
 '__spec__', 
 '__version__', 
 'collections', 
 'containers', 
 'csc_matrix', 
 'ct', 
 'deepcopy', 
 'downsample', 
 'example_midi_file', 
 'functools', 
 'matplotlib', 
 'midi', 
 'mido', 
 'normalize', 
 'notes2pianoroll', 
 'np', 
 'os', 
 'parser', 
 'pianoroll', 
 'pianoroll2notes', 
 'pitch_padding', 
 'plot', 
 'plot_background', 
 'plot_chroma', 
 'plot_grid', 
 'plot_heatmap', 
 'plot_note_entries', 
 'plot_xticks', 
 'plot_yticks', 
 'plt', 
 'pylab', 
 're', 
 'tochroma', 
 'utils', 
 'vis', 
 'warnings']

This removes all of the star imports (so this is a semver major change) in favor of just regular imports from the respective modules.

[Natooz]

Hello 👋
Thank you for the suggestion, and sorry for the late reply, the repo has been idle for some time, I'm currently fixing and improving things here.
I agree with you that star import should be removed.
Here however I thing it would still make sens and be convenient to directly access to the MidiFile and container (Note, ...) classes directly from the main module.

Would you still be interested in applying these last changes, and solve the branch conflicts that appeared as some changes has been made since?
This should hopefully be featured in the next release.

[akx]

@Natooz Rebased, added exports for MidiFile and all of the classes from containers. Will fix the tests...

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4f94fbd) 62.23% compared to head (b467678) 58.34%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   62.23%   58.34%   -3.89%     
==========================================
  Files          13       12       -1     
  Lines         789      785       -4     
==========================================
- Hits          491      458      -33     
- Misses        298      327      +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Natooz]

Wonderful thank you, everything seems good + tests are passing!