-
Notifications
You must be signed in to change notification settings - Fork 36
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
Comments
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? |
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. |
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. |
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).
The text was updated successfully, but these errors were encountered: