-
Notifications
You must be signed in to change notification settings - Fork 39
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
Request python package Qiime2 and RPy2 for class CCB 293! #2737
Comments
Teaching team will make edits to this request! |
Hi here are a few edits to this request: Which package do you want installed? Please provide more information about the package, such as the version you require, missing functionality you need, and a URL to the website for the package. qiime2: version: QIIME 2 Core 2021.8 distribution Are there specific package dependencies? Which hub do you want it installed on? For example r.datahub, datahub, data100.datahub, etc. Which class will use this package? Include a link to appropriate entry in https://classes.berkeley.edu/ if available. If not, please mention class name & instructor. Which semester will this package be used for? Any additional information we should know about? |
@yuvipanda here's a link to the installation instructions. What do you think? |
@petersudmant giving you a heads up on this too to see if it's something you'd be willing to tackle. |
https://raw.githubusercontent.com/qiime2/environment-files/master/2021.8/release/qiime2-2021.8-py38-linux-conda.yml is the environment.yml file it needs, including its own channel. I think this is something we can try on biology hub as we already get many things from conda there. But we'll need to figure out which packages from there we actually need here. I'm not sure how exactly to do that - maybe there's documentation for it somewhere I can't find. @felder rpy2 should be easy to add tho - I think it's just a python package? |
@balajialg @kferger320 is DESeq2 and rpy2 still part of this request? If so should it still be installed on datahub or should it be installed on biology hub? |
@yuvipanda well it's a bit confusing because there's also DESeq2 which might be runable in a jupyter notebook via rpy2? |
@felder Teaching team is thinking about using other alternatives to the DESeq2 package and will get back with a new package request for the same. Currently, they require qiime2 package to be installed. It should get installed in the biology hub as they will require complex computations. |
DEseq2 should be super easy - it's just an R package. I've done lots of similar packages. Happy to do that if you'd like. Re qiime2, looks possible. Would prefer someone else to tackle, but, happyt ogive it a shot. |
Tell them that EBseq is already on there which works really well |
@petersudmant it seems for the time being the DEseq2 is not required. @yuvipanda maybe this gives us a reasonable list of packages to think about? I dunno, they do say that there are a lot of dependencies. Would it make sense to do as the qiime2 folks recommend and install it as a whole new conda environment and perhaps the students will need to activate it? |
|
installing as a "whole new conda env" makes sense to me - I have done this before. You can see deployments/biology/image/ib134-packages.bash - we can make one for CCB293 as well. I think this should be smooth?! Shall I try? |
@petersudmant if you're willing, that'd be awesome! |
OK, done! @balajialg @kferger320 @felder works great! Run w/ the following
It's building on staging now - i'll double check it works in the a.m. and push it to prod. (or anyone else can feel free to do that if the build looks good) |
@petersudmant @balajialg @felder thanks so much for getting this done so quickly! I will test it out once everything's pushed |
it's all uploaded now. I had to do
then it works. Works great |
@petersudmant Has this been merged into prod? If you need that done, please let me know or open a PR and make me a reviewer. |
YEP - should be! Yuvi did this a.m. |
@petersudmant ok thanks! |
@kferger320's request here - I'm trying to test out qiime2 in a Jupyter notebook on biology.datahub.berkeley.edu and I'm having a hard time figuring out how to load it in the notebook since it was installed as a conda environment. We want to be able to use the package in a Jupyter notebook for the class, so could you let me know the best way to get this working? @petersudmant I tried running the steps you outlined, but I am also having difficulty importing the qiime2 package. Can you check and let us know what we are missing? Ran this command in the terminal and tried importing the qiime2 package in the jupyter notebook,
but getting this error `--------------------------------------------------------------------------- ModuleNotFoundError: No module named 'qiime2'` Any idea what are we missing? |
Right - So the commands I wrote out are for access to qiime2 in the command line, eg:
Your use case is different - I didn't realize you wanted to code in python. I'm not sure how to do this. The only way I can think is to create a new jupyter-notebook kernel - here are instructions on how this CAN work. Not sure if this would work though. We coudl try |
@petersudmant Thank you so much for clarifying the installation set up! I didn't realize it was for terminal-based usage. @kferger320 Would you be able to use the terminal commands for qiime2 (as outlined above by @petersudmant )? or having qiime2 as part of jupyter-notebooks is a must? @petersudmant Is this something that you would be able to help with? I tried replicating the blog's suggestions in my notebook, but I had difficulty running the import statement despite switching to the new conda environment with the qiime package installed. (Followed the first two steps outlined in the blog).
|
@balajialg Ideally we would use this package as part of a python tutorial, so it would be best if we could get it working in a Jupyter notebook if possible. |
@kferger320 @balajialg I have a solution - but, it's a tiny bit tedious. I'm trying something else ( I think it's a good fix ) - but it's compiling on my computer now and I want to go to bed - I'll check in tomorrow ok? |
@petersudmant Makes sense! Considering that @kferger320 reported an issue using the qiime package sometime back. Sharing the log content for your reference,
Based on my google search, I found that this could be a suitable solution for this error. Assuming it is correct, I created this PR. Can you let me know if both my diagnosis and the solution for this issue are correct? |
Hmm - I mean, the question is DOES this fix the issue? @balajialg If so, GREAT! |
@petersudmant Great question! I am not sure. I assumed this might be a fix as few other folks who ran into a similar timezone-related issue installed this library. What would be a good way to test whether this solution works without messing with the docker environment considering that sudo apt install command is not available. We tried doing pip install tzdata in the notebook but that did not seem to fix the error. I couldn't find any other solution that may be relevant here. @kferger320 was open to testing whether this fix worked in the staging environment. What do you think? |
@balajialg I think that's a reasonable approach! I test on my laptop, but, this seems like a good way too! |
@petersudmant sounds great! I am merging the changes to staging. @kferger320 Please do test this and let us know whether this solution actually works! |
Installing library tzdata as part of docker image in biology hub for issue #2737
@petersudmant @balajialg I think this solved it! I was able to get all of our commands to run on staging now without error. Thanks to you both for all of your help, we really appreciate it!! |
Great! good idea @balajialg ! |
Thanks, @kferger320! That's great to know. FYI, Changes from staging got merged in biology.datahub.berkeley.edu. |
Wonderful! Just tested it again and it works great. Thanks again everyone for all of your help with this! |
Hi all @yuvipanda @balajialg @petersudmant @arwolf0808 , it seems the same error that seemed to be solved by installing tzdata is popping up again, so now the same commands that were working great last week are now suddenly not working, and throwing the same |
@kferger320 Sorry to hear that this issue is recurring (which is unfortunate). Were you able to run this command yesterday? We made changes to the biology hub today and trying to see whether that correlates with this behavior. @yuvipanda @petersudmant Is this issue related to this change which seems to be the only change after the addition of tzdata in the docker file. |
@balajialg thanks for looking into this. I think both Ashley and I only tested it today, so I can't say if it was working yesterday- we just know that it was working fine a few days ago like I mentioned |
@balajialg The |
Trying to set timezone to UTC instead of /UTC for #2737
The issue here is the machine’s timezone was somehow set to “/UTC” as opposed to “UTC” and pytz did not like that. I’m deploying a fix now to prod that sets the timezone correctly. The error went away in our test notebook when I deployed the fix to staging. @petersudmant In a related question, is UTC the timezone you want for biology hub vs America/Los_Angeles ? |
@felder GOOD CATCH! Sure, that's the time zone we are in why not! @arwolf0808 @kferger320 why in the WORLD does qiime need to know the time >.< 🥇 |
@felder great, ty! I think we set all our other hubs to AMerica/Los_Angeles, so let's do that for this one too? |
@petersudmant no clue ! Thanks all, is this set up in staging or deployed?
Ashley
…On Wed, Oct 20, 2021 at 11:11 PM Yuvi Panda ***@***.***> wrote:
@felder <https://github.com/felder> great, ty! I think we set all our
other hubs to AMerica/Los_Angeles, so let's do that for this one too?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2737 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AU73GSNIOTJV54VEWCYCUNTUH6VILANCNFSM5DZ4EVUQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
@arwolf0808 the fix for the UTC timezone was deployed to staging and prod last night. |
@felder Thank you so much for fixing this issue! Appreciate it. @arwolf0808 I hope you are able to run the code. Here is a snapshot from @kferger320's instance in https://biology.datahub.berkeley.edu/ for your reference |
@kferger320 can you confirm that this is working correctly now? If so I'll close the issue. |
@felder @balajialg Yes just tried a few commands and looks like everything's working normally again! Thanks again all for your help with this. |
Thanks a lot for fixing it up, @felder! |
Which package do you want installed? Please provide more information about the package, such as the version you require, missing functionality you need, and a URL to the website for the package.
qiime2: https://docs.qiime2.org/2021.4/interfaces/artifact-api/
DESeq2: which is an R package, but I think there is a way to run it in a Jupyter notebook using the package 'rpy2'?
Are there specific package dependencies?
Which hub do you want it installed on? For example r.datahub, datahub, data100.datahub, etc.
Datahub
Which class will use this package?
CCB 293
Include a link to appropriate entry in https://classes.berkeley.edu/ if available. If not, please mention class name & instructor.
Which semester will this package be used for?
Fall, 2021
This helps us clean up unused packages after a term ends.
Any additional information we should know about?
The text was updated successfully, but these errors were encountered: