Missing dependency: simplejson

[alexanderwerning]

meeteval.io.seglst.SegLST.parse requires simplejson, which is neither listed in the requirements.txt nor in the pyproject.toml.

[thequilo]

It is listed in the [cli] dependencies in setup.py. The reasoning was that you don't need to parse SegLST files if you don't use the CLI. I'm fine with moving it to the install_requires requirements.

[alexanderwerning]

Ok, so there are three files with dependencies, setup.py requirements.txt and pyproject.toml? This is a bit confusing. If simplejson is not needed by the average user, we can keep it as an optional dependency.

[boeddeker]

How about updating the readme install instructions?

pip install meeteval  # pip install meeteval[all]

[thequilo]

I think we can delete requirements.txt. setup.py was required for cython extensions. It may now be possible to build python extensions with just a pyproject.toml, but I haven't tested it yet. I'm also not sure whether all issues with pyproject.toml (e.g., editable installs) are completely resolved. So I think that it's better to stick to setup.py for now.

It makes sense to update the readme instructions

[alexanderwerning]

I use editable installs from pyproject.toml, this seems to work?

[boeddeker]

Does it work with the user flag? We have several repositories, where we had to add a workaround, because some version oiof setuptools and/or pip don't work.

I would keep the current configuration with setup.py, until it's mandatory to change, or it gets buggy.