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

Reduce api calls for generic assay meta in study view page #5019

Open
wants to merge 44 commits into
base: clickhouse
Choose a base branch
from

Conversation

dippindots
Copy link
Member

No description provided.

alisman and others added 30 commits June 12, 2024 10:59
Adjust patient treatments and sample treatments to use new treatment-counts api endoint (clickhouse driven)
* Add rfc80 test generator UI and test runner
* fix production build after webpack.config changes
* fix has generation based on url

* add back all endpoints
…#4981)

* add tests for sample lists counts and custom data filters
* add tests for custom data bin counts

---------

Co-authored-by: Bryan Lai <[email protected]>
* improvements to apiTests reporting

* add skips to validation, refine report

* skip test

* improve report

* Move api e2e to backend
@dippindots dippindots added the performance This PR is related to a performance issue label Oct 11, 2024
@dippindots dippindots self-assigned this Oct 11, 2024
Copy link

netlify bot commented Oct 11, 2024

Deploy Preview for cbioportalfrontend failed. Why did it fail? →

Name Link
🔨 Latest commit 2b27301
🔍 Latest deploy log https://app.netlify.com/sites/cbioportalfrontend/deploys/670e078144041400086bb3d1

this.genericAssayProfiles.result
);
},
});

readonly genericAssayStableIdToMeta = remoteData<{
Copy link
Member Author

Choose a reason for hiding this comment

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

No reference, deleting this function

@dippindots dippindots force-pushed the generic-assay-meta-study-view-reduce-call branch 3 times, most recently from b7d4fa5 to 2b27301 Compare October 15, 2024 06:11
* add runner for validation of datadog logged traffic
* use axios for validation ajax to normalize 
* get rid of specs
@alisman alisman changed the base branch from master to clickhouse October 17, 2024 16:34
@dippindots dippindots force-pushed the generic-assay-meta-study-view-reduce-call branch from 2b27301 to 73f2ebe Compare October 17, 2024 16:49
@alisman alisman force-pushed the clickhouse branch 2 times, most recently from ecba99f to 22501dd Compare November 25, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance This PR is related to a performance issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants