Skip to content

Conversation

mike-w-wilson
Copy link
Contributor

These resources were previously v3 specific. This update changes them to dictionaries where the key is the version and the value is a list.

Copy link
Contributor

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ugh, I hate that we need to add a breaking change here, but I see no reasonable way around it.

Just one confirmation comment and then it's good to go. Thanks for adding.

"tgp",
"hgdp",
],
"v4": ["ukb", "non-ukb", "non-topmed"],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just confirming there is an intentional switch to - from _ I don't remember all thee fine details associated with that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah no good catch. The 4.3 vcf spec does not allow hyphens so underscores are better here to be consistent with the vcf. I took this straight from the v4 release HT schema but I'll update it there as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mike-w-wilson mike-w-wilson self-assigned this Mar 28, 2023
@mike-w-wilson mike-w-wilson merged commit b30747c into main Mar 28, 2023
@jkgoodrich jkgoodrich deleted the mwqh/update_resources_to_dict branch April 5, 2023 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants