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

RW-12661 update qiime #502

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

RW-12661 update qiime #502

wants to merge 11 commits into from

Conversation

Qi77Qi
Copy link
Collaborator

@Qi77Qi Qi77Qi commented Feb 24, 2025

https://precisionmedicineinitiative.atlassian.net/browse/RW-12661

I was able to build the image locally, and running the image. But in order to verify if qiime is installed properly, I'll need to install a jupyter-server extension and upgrade jupyter-server, which I wasn't able to do locally. The main reason is because after installing the server extension, it requires restarting the jupyter server. But if we restart the jupyter server, the docer container will restart and losing the extesion. Lmk if you have suggestions how to test this. If not, I'd like to get this merged in, have the image generated and uploaded to GCP so that I can test things a bit better in a BEE

@Qi77Qi Qi77Qi marked this pull request as ready for review March 7, 2025 22:08
@Qi77Qi Qi77Qi requested review from Copilot and LizBaldo March 7, 2025 22:08

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Overview

This PR updates the changelog to document the recent qiime updates and the installation of qiime2R.

  • Updated changelog with version 0.1.2 to record the qiime upgrade and addition of qiime2R.

Reviewed Changes

File Description
terra-rstudio-aou/CHANGELOG.md Added version 0.1.2 changelog entries for qiime enhancements

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

@@ -1,5 +1,9 @@
# Changelog

## 0.1.2 - 2025-03-07
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this changelog and version control be for terra-jupyter-aou instead? I am alsways a bit confused about version control since there is a single image for both jupyter and r.

Would you be able to update the conf.json file accordingly?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh yea...updated the wrong file..good catch!

Copy link
Collaborator

@LizBaldo LizBaldo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use the GHA to build and publish the aou image you want to test to GCR, and then open a PR in Leo to test on a BEE.
That way you can merge this PR once your testing is done, let me know if you have any questions

@Qi77Qi
Copy link
Collaborator Author

Qi77Qi commented Mar 10, 2025

ah cool...forgot it's a GHA for building the image now...will try that

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