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

[GEN-1612] Use 16.6-consortium for CRC 3.1 #144

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

rxu17
Copy link
Contributor

@rxu17 rxu17 commented Nov 14, 2024

Purpose:

  • Updates _MG_RELEASE_SYNID to use 16.6-consortium for CRC 3.1 release.
  • Waiting on updates to _REDCAP_TO_CBIOMAPPING_SYNID

Testing:

  • Run sample run on staging and make sure it runs through. Do comparison report on results with previous.

Depends on #143

@rxu17 rxu17 requested a review from a team as a code owner November 14, 2024 23:11
@rxu17 rxu17 changed the title [GEN-1612] Use 16.7-consortium for CRC 3.1 [GEN-1612] Use 16.6-consortium for CRC 3.1 Nov 15, 2024
Copy link

sonarcloud bot commented Nov 15, 2024

@@ -521,6 +521,7 @@ class BpcProjectRunner(metaclass=ABCMeta):
_SPONSORED_PROJECT = ""
# Redcap codes to cbioportal mapping synid and form key is in
# version 38, 42 were last stable version(s)
# NOTE: Should be pointed towards latest version of table
Copy link
Member

Choose a reason for hiding this comment

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

There will be a new mapping version, did we want to wait until that is in to update both at the same time?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes this PR can be in place here.

Copy link

Choose a reason for hiding this comment

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

Are we waiting on MSK cBioPortal team to share the newer mapping file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes

Copy link

dpulls bot commented Nov 15, 2024

🎉 All dependencies have been resolved !

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.

3 participants