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

Summary stats release report email needs to be more human-readable #109

Open
ljwh2 opened this issue Jul 7, 2020 · 3 comments
Open

Summary stats release report email needs to be more human-readable #109

ljwh2 opened this issue Jul 7, 2020 · 3 comments

Comments

@ljwh2
Copy link
Contributor

ljwh2 commented Jul 7, 2020

The purpose of this email is to alert curators to manual errors in the sumstats release process, eg typo in folder name, curator forgot to check "full pvalue set" box.
Now that the folders are named with GCSTs only, this has become non-intuitive. The first author and PMID should be included in the email together with the folder names to allow curators to easily check that expected sumstats have been released.

@ljwh2
Copy link
Contributor Author

ljwh2 commented Jan 26, 2021

@earlEBI Please review as to whether this issue is still relevant

@earlEBI
Copy link

earlEBI commented Jan 27, 2021

I don't think this is important anymore. Folders are listed as renamed with author name and PMID once published.
Typos in folder names and not ticking 'full p-value set' box are not really possible curator errors anymore.

@ljwh2
Copy link
Contributor Author

ljwh2 commented Feb 1, 2021

@jdhayhurst tagging you for reference, I am moving this ticket to the icebox

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

No branches or pull requests

2 participants