search

Natooz / MidiTok

MIDI / symbolic music tokenizers for Deep Learning models 🎶
https://miditok.readthedocs.io/
MIT License
665 stars 81 forks source link

decoding: automatically making sure to decode BPE then completing `tokens` #138

Closed Natooz closed 8 months ago

Natooz commented 8 months ago

Following #137 , this PR makes sure the TokSequence in input of tokens_to_midi is automatically preprocessed: tokens have been completed, does it if necessary, and decode BPE if necessary

📚 Documentation preview 📚: https://miditok--138.org.readthedocs.build/en/138/

codecov[bot] commented 8 months ago

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (db3ad1d) 91.11% compared to head (576032e) 91.10%.

Files Patch % Lines
miditok/midi_tokenizer.py 84.21% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #138 +/- ## ========================================== - Coverage 91.11% 91.10% -0.02% ========================================== Files 33 33 Lines 4886 4891 +5 ========================================== + Hits 4452 4456 +4 - Misses 434 435 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.