Closed logstar closed 1 year ago
This looks good!
Since this PR was for updated TPMs and v10 differential gene expression table is still being used, I did not review the differential gene expression result tables. Did look at the plots however and they looked good.
@kelseykeith - Thank you for reviewing the TPM boxplot PRs. I will merge this PR soon.
Description
Updated API database using OpenPedCan-analysis v11 TPM data release.
Resolved #76.
Notes for reviewers:
v11-tpm
branch tov10-dge
branch. Therefore, it is recommended to review https://github.com/PediatricOpenTargets/OpenPedCan-api/pull/65 before reviewing this PR, because https://github.com/PediatricOpenTargets/OpenPedCan-api/pull/65 intends to mergev10-dge
branch tomain
branch. After https://github.com/PediatricOpenTargets/OpenPedCan-api/pull/65 is approved and merged, the base branch of this PR can be changed tomain
branch.tests/results
,tests/http_response_output_files
, andtests/plots
. The test output files only need to be reviewed by their content, e.g., whether a plot looks as expected or not.v11-tpm
branch, i.e., versionv0.5.3-beta
, has been deployed to the DEV site, https://openpedcan-api-dev.d3b.io/__docs__/. If the DEV site has been deployed with another branch at a later point, thev11-tpm
branch needs to be evaluated locally.Type of change
Non-breaking change. Database update.
How Has This Been Tested?
Terminal returns for testing DEV host:
For more details about testing environment, options, and resources, see https://github.com/PediatricOpenTargets/OpenPedCan-api/blob/98632b538a0416fe0f27d0ad8ebd117d2969466c/README.md.
Checklist