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

ruamel.yaml vs pyyaml #967

Open
WardDeb opened this issue Jan 20, 2024 · 0 comments
Open

ruamel.yaml vs pyyaml #967

WardDeb opened this issue Jan 20, 2024 · 0 comments

Comments

@WardDeb
Copy link
Member

WardDeb commented Jan 20, 2024

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:

chip_dict:
  sample1:
    control: sample7
    broad: False

&

chip_dict:
  sample1:
    control: sample7
    broad: "False"

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant