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

Update manu #79

Merged
merged 16 commits into from May 11, 2019
Merged

Update manu #79

merged 16 commits into from May 11, 2019

Conversation

Chilipp
Copy link
Member

@Chilipp Chilipp commented May 11, 2019

This PR includes some updates exported by @mchevalier2. This involves new metadata for binney and epdcoretop samples, a revised groupid → higher_groupid mapping, and a percentage column for the p_counts table

@EMPD-admin
Copy link

Hi! I'm your friendly automated EMPD-admin bot!

Thank you very much for your data submission! I will now run some tests on your data.
In the meantime: please review your test data on the EMPD viewer using
this link: https://EMPD2.github.io/?repo=EMPD2/EMPD-data&branch=update-Manu&meta=manu-update.tsv

This bot is helping you merging your data. I am checking for common issues and I can fix your climate, country or elevation data.
Type @EMPD-admin help in a new comment in this PR for usage information.

Make sure, that all the commands for the @EMPD-admin are on one single line and nothing else. Only
@EMPD-admin help
will work, not something like
<other words> @EMPD-admin help
or
@EMPD-admin help <other words>

Note that you can also run these tests on your local machine with
pytest.

@EMPD-admin
Copy link

Hi! I'm your friendly automated EMPD-admin bot!

This is just to inform you that I tested your data submission in your PR (manu-update.tsv) and found it in an excellent condition!

@EMPD-admin
Copy link

Hi! I'm your friendly automated EMPD-admin bot!

Thank you very much for your data submission! I will now run some tests on your data.
In the meantime: please review your test data on the EMPD viewer using
this link: https://EMPD2.github.io/?repo=EMPD2/EMPD-data&branch=update-Manu&meta=meta.tsv

This bot is helping you merging your data. I am checking for common issues and I can fix your climate, country or elevation data.
Type @EMPD-admin help in a new comment in this PR for usage information.

Make sure, that all the commands for the @EMPD-admin are on one single line and nothing else. Only
@EMPD-admin help
will work, not something like
<other words> @EMPD-admin help
or
@EMPD-admin help <other words>

Note that you can also run these tests on your local machine with
pytest.

@EMPD-admin
Copy link

Hi! I'm your friendly automated EMPD-admin bot!

This is just to inform you that I tested your data submission in your PR (meta.tsv) and found it in an excellent condition!

Skipping postgres import because of too many rows

@Chilipp Chilipp merged commit 1f75e30 into master May 11, 2019
@Chilipp Chilipp deleted the update-Manu branch May 11, 2019 10:48
Chilipp added a commit that referenced this pull request May 11, 2019
as continuation of #79
@Chilipp Chilipp mentioned this pull request May 11, 2019
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.

None yet

2 participants