-
Notifications
You must be signed in to change notification settings - Fork 1
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 R script #256
add R script #256
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #256 +/- ##
==========================================
- Coverage 47.48% 47.19% -0.29%
==========================================
Files 49 49
Lines 3513 3545 +32
==========================================
+ Hits 1668 1673 +5
- Misses 1845 1872 +27
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Looks like there are some image building problems to deal with before this can be merged though |
@@ -28,6 +28,7 @@ coverage = "==7.2.2" | |||
pytest-cov = "==4.0.0" | |||
ipython = "==8.12.0" | |||
pytest-asyncio = "==0.21.0" | |||
rpy2 = "==3.5.10" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move rpy2 temporally to dev-packages so that docker image can be built. create a ticket to fix it later.
https://kbase-jira.atlassian.net/browse/RE2022-149
.. and now codecov is choking |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything passed, yay
Although codecov still hasn't updated
Edit: oop, there it goes
No description provided.