Connects to #205. QC Data Monitor updates for QC automation version 2.

MoTrPAC / motrpac-frontend

User interface and Frontend for MoTrPAC Bioinformatics Center

MIT License

3 stars 0 forks source link

Connects to #205. QC Data Monitor updates for QC automation version 2. #209

Closed jimmyzhen closed 1 year ago

jimmyzhen commented 1 year ago

Key Changes:

On the QC Data Monitor page and under the main Metabolomics tab, there are now 2 sub-tabs: Processed and Raw, with Processed being the default sub-tab for Metabolomics QC data.
On the QC Data Monitor page and under the main Proteomics tab, there are now 3 sub-tabs: Processed by BIC, Processed by CAS, and Raw. The default sub-tab for Proteomics QC data is Processed by BIC.
The HTTP GET requests for metabolomics and proteomics QC data from the Service API are now split into 2 queries, one for processed QC data and the other for raw files QC data.

cteng585 commented 1 year ago

Tabs appear and behave as expected.

One thing I noticed is that the first time the Phase section of the data monitor needs to load, it takes ~10-15 seconds. Also, there are only 7 metabolomics entries and 12 proteomics entries. Is this expected?

jimmyzhen commented 1 year ago

Hi @cteng585, thanks for helping to review this PR.

The initial data fetch does take a little bit of time. Although the GET requests are async, the data transfer rate does vary from user to user depending on the network. We will look into ways to optimize the performance going forward.

The demo instance is using dev QC data. I am not expecting it to mirror what we have in production.