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 file size for report #106

Open
pinin4fjords opened this issue Mar 29, 2023 · 3 comments
Open

Reduce file size for report #106

pinin4fjords opened this issue Mar 29, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@pinin4fjords
Copy link
Member

Description of feature

(sorry @WackerO , you were right!)

I'm noticing my browser having trouble coping with reports (even if they are only 30Mb or so).

We may have to do some sensible truncation of the d/e tables etc, and take steps to reduce the size of the HTML associated with different plots (maybe some rounding is in order).

@pinin4fjords pinin4fjords added the enhancement New feature or request label Mar 29, 2023
@WackerO
Copy link
Collaborator

WackerO commented Mar 30, 2023

yeah, I've had the same issue...So far my browser can handle the report, but it does take a noticeable time to load. I'm wondering also if we could make some parts of the report optional. I'm thinking maybe by default don't add the methods section; the parameters could instead just be written into a tsv file. Also, what do you think about by default only having the 2D PCA plots and adding the 3D plot only if the user sets this with some param?

@pinin4fjords
Copy link
Member Author

The method section won't be having a major impact, neither will the PCA (the number of points, and therefore the amount of embedded data is small). I would put money on the differential expression tables (not currently truncated) and many-point plots such as the volcano plots as the major culprits.

@pinin4fjords pinin4fjords added this to the 1.3.0 milestone Apr 21, 2023
@pinin4fjords pinin4fjords removed this from the 1.3.0 milestone Oct 10, 2023
@alanzos
Copy link

alanzos commented Apr 9, 2024

Yup, I have also suffered the long loading times and sometimes it freezes.

A sub-optimal solution for me has been to limit the number of contrasts to 3-4, and generate several reports.

An ideal solution would be to have a parameter that changes the interactive plots, to static ones, like outlier detection plots, density plots, PCAs, and volcanos (as @pinin4fjords mentioned). This should significantly reduce the size of the report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

3 participants