mlund / faunus

A Framework for Metropolis Monte Carlo Simulation of Molecular Systems
https://faunus.readthedocs.io
MIT License
66 stars 33 forks source link

Use upstream ruamel.yaml instead of ruamel_yaml #435

Closed mbargull closed 10 months ago

mbargull commented 10 months ago

Description

I'm currently looking at what packages on conda-forge have a dependency on the non-maintained ruamel_yaml package.

It'd be good if we could have this package depend on ruamel.yaml instead of ruamel_yaml, too.

NB: The latest conda-forge::ruamel_yaml is still at version 0.15.80 -- upstream released that version 5 years ago.

Conda-forge will stop (re-)building (the severly outdated) ruamel_yaml package; tracking issue: https://github.com/conda-forge/ruamel_yaml-feedstock/issues/107

xref: https://github.com/conda-forge/faunus-feedstock/issues/42


This should also add compatibility for ruamel.yaml=0.18. ruamel.yaml=0.18 removed the top-level load, dump, etc. functions. For details see https://pypi.org/project/ruamel.yaml/0.18.4/ .


As indicated in the incomplete checklist below, the changes have unfortunately, not been tested by myself. Feel free to cherry-pick these changes or push to this PR's branch directly (Allow edits by maintainers is enabled) if changes are needed.

Checklist

mlund commented 10 months ago

Thanks for looking into this.

mbargull commented 10 months ago

Thanks for the extremely fast reaction :)