Skip to content

Handle issues with ruamel-yaml version pin #2404

@opotowsky

Description

@opotowsky

In PR #2403 , we needed to add a max version pin for ruamel.yaml: https://github.com/terrapower/armi/pull/2403/changes#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711R43

v0.19.0 broke the code with:

Image

This is a bug with RoundTripLoader. Right now we pass around loaders for a presumed speedup. The fix will involve us removing all passing around of loader and just using YAML() directly. Once we get the code working, we need to make sure it doesn't slow down CI in ARMI or downstream. (This shouldn't affect case runs, since yaml loading is only done at the beginning of the run)

Metadata

Metadata

Assignees

Labels

low priorityStyle points and non-features

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions