search

dopefishh / pympi

A python module for processing ELAN and Praat annotation files
MIT License
93 stars 39 forks source link

Release version 1.7 has a lower version number than the previous release (1.69) #40

Closed gedankenexperimenter closed 3 years ago

gedankenexperimenter commented 3 years ago

When specifying a dependency on pympi-ling in a setup.cfg file for a python package, if include the following:

install_requires =
    pympi-ling>=1.69

…and version 1.7 is already installed, it will be removed in favour of 1.69 when installing the dependent package with pip. Alternatively, with this:

install_requires =
    pympi-ling>=1.7

…version 1.69 will satisfy that dependency, and will not be replaced with version 1.7.

The correct semantic version number should be 1.70, not 1.7.

dopefishh commented 3 years ago

thank you

gedankenexperimenter commented 3 years ago

Would you mind publishing 1.70 to PyPI? It would be really nice to not get the warnings about version 3.0 of EAF files when running tests.

dopefishh commented 3 years ago

Yes, there is one blocking issue still. If that's fixed the new version will be pushed. I'll try to allocate some time this week for that.

gedankenexperimenter commented 3 years ago

The main issue I had was that call to warnings.warn() in Elan.py, which was printing a message to STDOUT instead of STDERR. I've worked out how to use contextlib.redirect_stdout() to fix that, so my use case is not urgent.