Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pyyaml now require a loader #6

Open
GuillaumeBeaudin opened this issue Jul 5, 2022 · 2 comments
Open

pyyaml now require a loader #6

GuillaumeBeaudin opened this issue Jul 5, 2022 · 2 comments

Comments

@GuillaumeBeaudin
Copy link

The latest version of pyyaml (6.0) requires to specify explicitly a loader. To correct this issue, this line from srim/core/elementdb.py :

return yaml.load(open(dbpath, "r"))

must be changed to:

return yaml.load(open(dbpath, "r"), yaml.Loader)

Thanks for this wonderful python package!

@Allentro
Copy link

Allentro commented Jul 6, 2022

I made a pull request for this the other day. From what I can tell this repository is not maintained.

#5

@mcnanneyd
Copy link

I do not know how much more active it is, but the documentation states that this project is maintained on gitlab, and github is solely for visibility.

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

No branches or pull requests

3 participants