CPJKU / partitura

A python package for handling modern staff notation of music
https://partitura.readthedocs.io
Apache License 2.0
227 stars 15 forks source link

Running tests locally without miditok #337

Closed neosatrapahereje closed 8 months ago

neosatrapahereje commented 9 months ago

The new tokenizer functionalities require miditok and miditoolbox, which are not yet part of the requirements. Nevertheless, these packages are still used in test_utils.py, and result in an error when performing unittests locally.

manoskary commented 9 months ago

Hello, I commented on the relevant PR #338 . This should be the intended behaviour. The online tests should run with no problem and check that all external dependencies work. Locally you would expect tests failing when you don't have a dependency with a message dependency X is missing.