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

[WIP] GEO importer base #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[WIP] GEO importer base #52

wants to merge 2 commits into from

Conversation

pcm32
Copy link
Member

@pcm32 pcm32 commented Feb 15, 2022

This PR adds a base container to use with GEO importer scripts. This shouldn't be merged until bioconda packages are added for:

@pcm32 pcm32 changed the title GEO importer base [WIP] GEO importer base Feb 15, 2022
@anjaf
Copy link
Member

anjaf commented Apr 11, 2022

Hi @pcm32 What do you think about that workaround? It is using the original perl-atlas-modules conda package but removes the line with the import that we can't get from conda. The code has been tested to run without this module import.

@pcm32
Copy link
Member Author

pcm32 commented Oct 4, 2022

Are we going to use this @anjaf or should I close this? Thanks!

@anjaf
Copy link
Member

anjaf commented Oct 5, 2022

Ideally, yes we should be using this. The blocker is a pending change on the perl-atlas-modules: ebi-gene-expression-group/perl-atlas-modules#21
This needs a new release of the modules, which was in itself blocked but Jon fixed it now, so we can tackle this again.

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.

2 participants