-
Notifications
You must be signed in to change notification settings - Fork 85
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
ruamel.yaml vs pyyaml #967
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems pytests would require the use of ruamel yaml instead of pyyaml (#966) (at least if we want to keep generating all the data 'on the fly'). This seems to be pulled in as a dependency already, though should probably be an explicit dependency.
The 'regular yamls' (i.e. organism, chipdicts, ...) are parsed with pyyaml, but show some inconsistent behavior when being parsed, leading to different number of jobs inferred. This showed up in #965.
For example:
&
Lead to a different number of jobs inferred.
Ideally parsing the yamls are made a bit more robust, and for consistency would also be done with ruamel's yaml.
The text was updated successfully, but these errors were encountered: