-
Notifications
You must be signed in to change notification settings - Fork 14
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
BUG: evaluate-busco
brakes when input mags > ~1,300
#140
Comments
Possible solution?
|
Hey @Sann5, thanks for investigating! I like your solution of splitting those into tabs. Maybe we can even go a bit lower? 500? 100? (could be configurable as well, with a max set to, say, 500?) |
Just as a reference what Pau is trying to plot are 5,260 MAGs, distributed among 150 samples which contain a median of 30 MAGs, (min 1, max 127). |
evaluate-busco
brakes when input mags > 5000evaluate-busco
brakes when input mags > ~1,300
@misialq this would in Pau use case create around 11 tabs. Would that be ok? |
Hmmm, I'm thinking now that actually it's more complicated than saying how many MAGs you want to display - they are grouped per sample, right? We don't want to split MAGs from the same sample between two tabs - would this be possible? I don't think the number of tabs is a problem per se... It's either that or one thousand of rows in less tabs... That's why I thought to make it configurable, but maybe the default could still be something higher (like 1k) if you think that's better. I have not seen a visualization with so many MAGs so it's a bit tough to imagine what this would look like... |
I would make it so that doesn't happen. Selecting samples iteratively until the threshold is reached.
I can send you the one that is on the limit (~1,300) haha. I ll do it over Slack.
The other option that I am looking into is having a dropdown menu from which users can select which range of samples to look at (like in the second tab of the chekm viz), e.g. the first 500, the second 500, and so on. As you said the number of MAGs per range can be left configurable. |
Sounds good - I like the solution with a drop-down or something similar - let me know when you have something to look at 😎 |
As the solution included the vegafusion transformer which caused trouble at conda installation (also, technically it is not available for all platforms yet) I'm reopening this - we'll need to find a different one. |
The actual problem is vega, the plotting engine. See this post.
Error message:
The text was updated successfully, but these errors were encountered: