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.

Sign up for GitHub

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

Jump to bottom

TASK-1111 - Add Pharmacogenomics view in IVA #770

Draft
wants to merge 41 commits into
base: develop
Choose a base branch
from
Draft

TASK-1111 - Add Pharmacogenomics view in IVA #770

wants to merge 41 commits into from

Conversation

jmjuanes
Copy link
Member

This PR adds a new Pharmacogenomics view in IVA.

jmjuanes added 30 commits May 8, 2023 18:28
@jmjuanes
wc: added new button for pharmacogenomics in browser-toolbar
dcf5ba4
@jmjuanes
wc: minor style fixes in interpreter browser toolbar
143d9d9
@jmjuanes
wc: dispatch event when clicking on pharmacogenomics variants button
6c0981e
@jmjuanes
wc: added listener in interpreter browser template for PGx variants f…
a154b6f 
…ilter
@jmjuanes
wc: add new step for pharmacogenomics in interpreter
392b40b
@jmjuanes
wc: initialize pharmacogenomics components
cbd2300
@jmjuanes
wc: added pharmacogenomics tab to interpreter browser RD
73abf91
@jmjuanes
wc: using pharmacogenomics-report component in interpreter step
81b1965
@jmjuanes
wc: minor fixes in pharmacogenomics report component
73a9e9f
@jmjuanes
wc: added listener to filter for pgx variants in browser template
2ea4321
@jmjuanes
wc: fix pgx filter in interpreter browser template
c9ae798
@jmjuanes
wc: import pgx variants from cellbase in pgx-report component
c832af8
@jmjuanes
wc: increase chunk size in pgx variants of browser-template
cc6a8a7
@jmjuanes
Merge branch 'develop' into TASK-1111
31bb203
@jmjuanes
iva: remove enabledComponents print in iva-app
92fb002
@jmjuanes
wc: remove pharmacogenomics step in interpreter
350c4ef
@jmjuanes
wc: add new tab in variant interpreter brwoser for pgx
1868d3d
@jmjuanes
wc: fix getting pgx drugs info in pharmacogenomics-report component
664d760
@jmjuanes
wc: work in progress in pharmacogenomics grid
58b0695
@jmjuanes
wc: keep previous filters when filtering by pharmacogenomics
55b2c14
@jmjuanes
wc: keep previous ids when filtering by pharmacogenomics
c82d60e
@jmjuanes
wc: change endpoint to obtain pgx annotation in pharmacogenomics-report
a2277e5
@jmjuanes
wc: display a loading indicator in pharmacogenomics-report
804a20d
@jmjuanes
wc: minor style fixes in loading view of pharmacogenomics-report
7aeb002
@jmjuanes
wc: initialize pharmacogenomics grid and add basic columns
b4d2266
@jmjuanes
wc: include pharmacogenomics grid in pgx-report component
774fcca
@jmjuanes
wc: some improvements in pharmacogenomics-grid
0e18641
@jmjuanes
wc: overwrite id field in query after applying pharmacogenomics filter
6f9d02f
@jmjuanes
wc: fix importing variants and annotation in pharmacogenomics-report …
80788e8 
…component
@jmjuanes
wc: added initial variant ccolumns in pharmacogenomics-grid
8655438
@jmjuanes
Copy link
Member Author

Task linked: TASK-1111 Add Pharmagenomics data to CellBase

@jmjuanes jmjuanes requested a review from imedina May 15, 2023 13:15
@jmjuanes jmjuanes marked this pull request as draft May 15, 2023 13:15
@jmjuanes jmjuanes marked this pull request as ready for review May 18, 2023 14:35
@halender halender requested review from gpveronica and removed request for imedina May 19, 2023 08:12
gpveronica
gpveronica previously approved these changes May 19, 2023
View reviewed changes
@sonarcloud
Copy link

sonarcloud bot commented May 22, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@imedina imedina marked this pull request as draft May 22, 2023 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gpveronica gpveronica gpveronica left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jmjuanes @gpveronica