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

Improvements for SONG #235

Closed
scottcain opened this issue Apr 11, 2022 · 2 comments
Closed

Improvements for SONG #235

scottcain opened this issue Apr 11, 2022 · 2 comments
Assignees
@joneubank @UmmulkiramR
Labels
awaiting-confirmation critical enhancement New feature or request WP_1

Comments

@scottcain
Copy link
Contributor

From @lindaxiang: with the growing number of analysis for each project, the time which spent on getting the song analysis becomes a blocker. As the current song api endpoint does not support pagination, it kept timing out and took almost half day just to get back the song analysis (34k analysis) for a given study like LSPQ-QC. Any plan when we will get Alexis’s song performance improvement into Virusseq SONG server? I hope that would help to solve the problem.

This is presumably related to #222
@scottcain scottcain added critical enhancement New feature or request labels Apr 11, 2022
@ghost ghost added the WP_1 label Jun 3, 2022
@b-f-chan b-f-chan assigned ghost and UmmulkiramR Jun 3, 2022
@b-f-chan
Copy link
Contributor

b-f-chan commented Jun 3, 2022

@sifavahora @UmmulkiramR couple suggestions:

(1) We should definitely upgrade to the latest SONG with pagination features before starting the work package features
(2) We should also audit the versions of other Overture components to see if any upgrade needed; feel free to create separate ticket and also consult Jon/Justin on this

@ghost ghost assigned joneubank and unassigned ghost Jun 17, 2022
@leoraba
Copy link
Contributor

leoraba commented Jan 30, 2023

Virusseq is currently using the latest SONG version (v5.0.3) .
Pagination features in SONG were implemented since v4.8.0

@leoraba leoraba closed this as completed Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joneubank joneubank

@UmmulkiramR UmmulkiramR

Labels
awaiting-confirmation critical enhancement New feature or request WP_1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@scottcain @joneubank @leoraba @b-f-chan @UmmulkiramR