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

Add mouse features #32

Merged
merged 90 commits into from
Mar 28, 2019
Merged

Add mouse features #32

merged 90 commits into from
Mar 28, 2019

Conversation

yingqiuz
Copy link
Contributor

add mouse module, including:
(1) retrieve gene info, do gene id <->name/acronym conversions, check gene validity
(2) retrieve structure info, do structure id <->name/acronym conversions, check structure validity
(3) fetch unionisation data, including expression-energy, expression-density, sum-pixels, etc.

@rmarkello
Copy link
Owner

rmarkello commented Jan 21, 2019

Hi @yingqiuz ! It looks like the TravisCI is failing because of a dependency issue. I've updated the master branch to fix this so merging it into your PR should fix the issue right up!

Let me know if you have any questions about how to do that! The first answer to this StackOverflow question should do the trick 🙂

…-mouse-feature

[TEST] merge pytest dependency fixes
@rmarkello
Copy link
Owner

Hi @yingqiuz ! I wanted to let you know that I haven't forgotten about this but I have just been super bogged down by other things. I will try and get to reviewing this soon ! Apologies for the delay, and thank you again for this awesome contribution ✨ 🙌

@rmarkello
Copy link
Owner

The Travis is definitely going to fail, but I'm working on refactoring some things and will address the tests as soon as I'm done!

@rmarkello
Copy link
Owner

The coverage decreased a bit, but the tests are passing! I'm going to go ahead and merge this in now, @yingqiuz. Thanks so much for your work!

@rmarkello rmarkello merged commit 0791601 into rmarkello:master Mar 28, 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.

4 participants