Skip to content

Conversation

@napulen
Copy link
Owner

@napulen napulen commented Jun 20, 2022

Fixes #68

  • No need to "autocorrect" rntxt files. Output the number of measures the model wants to output.
  • Fix the syntax of beats, changes of key, and Roman numerals

@codecov-commenter
Copy link

Codecov Report

Merging #81 (be76bbb) into main (0c91b48) will increase coverage by 0.49%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   74.95%   75.45%   +0.49%     
==========================================
  Files          47       47              
  Lines        2380     2363      -17     
==========================================
- Hits         1784     1783       -1     
+ Misses        596      580      -16     
Impacted Files Coverage Δ
AugmentedNet/inference.py 14.72% <0.00%> (+0.83%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c91b48...be76bbb. Read the comment docs.

@napulen napulen merged commit c67d5d4 into main Jun 20, 2022
@napulen napulen deleted the rntxt_fix branch June 22, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix RomanText writer

3 participants