Skip to content

Update Downloads page v4 checksums and sizes #1296

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

Merged
merged 2 commits into from
Nov 16, 2023
Merged

Conversation

mike-w-wilson
Copy link
Contributor

This updates the MD5s for both exome and genome chrY VCFs. It looks like something happened during creation or transfer of the chrY exome VCF so that it was a duplicate of the genomes chrY VCF. The exome VCF was regenerated and is currently syncing to the same location in the public bucket. For this file, the size and MD5 are updated in this PR.

gsutil hash -h gs://gnomad-public-requester-pays/release/4.0/vcf/exomes/gnomad.exomes.v4.0.sites.chrY.vcf.bgz

Hashes [hex] for release/4.0/vcf/exomes/gnomad.exomes.v4.0.sites.chrY.vcf.bgz:
	Hash (md5):		c8414744cf75fa498e2497a9391eb1ad
	Hash (crc32c):		7eadc442

For the genome VCF, a user pointed out our downloads page MD5 did not match the public file's MD5. I confirmed they do not and have also updated that files MD5 in this PR.

gsutil hash -h gs://gcp-public-data--gnomad/release/4.0/vcf/genomes/gnomad.genomes.v4.0.sites.chrY.vcf.bgz

Hashes [hex] for release/4.0/vcf/genomes/gnomad.genomes.v4.0.sites.chrY.vcf.bgz:
	Hash (md5):		8c231b75745b6670555915847c9999e8
	Hash (crc32c):		4b5875ee

Note the exome VCF will likely sync sometime tonight so this probably shouldn't go in until tomorrow. Also sorry if this is not how I should update this, was just trying to save you a bit of time!

@rileyhgrant
Copy link
Contributor

Thanks for doing this @mike-w-wilson!

@rileyhgrant rileyhgrant changed the title Update v4 chr Y md5s and sizes Update Downloads page v4 checksums and sizes Nov 16, 2023
Copy link
Contributor

@rileyhgrant rileyhgrant left a comment

Choose a reason for hiding this comment

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

LGTM, thanks again for updating these.

Just to leave a paper trail

  • Added a smol commit with an updated snapshot for our test suite
  • Changed the PR title since the "squash and merge" button defaults to PR title as the commit message

@rileyhgrant rileyhgrant merged commit b971c99 into main Nov 16, 2023
@rileyhgrant rileyhgrant deleted the mw/update-chr-y-md5 branch November 16, 2023 13:10
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

Successfully merging this pull request may close these issues.

MD5 check FAILED: gnomad.genomes.v4.0.sites.chrY.vcf.bgz
2 participants