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

Make circos plots optional in the orange report #113

Open
bounlu opened this issue Nov 29, 2024 · 1 comment
Open

Make circos plots optional in the orange report #113

bounlu opened this issue Nov 29, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@bounlu
Copy link

bounlu commented Nov 29, 2024

If there are many structural driver events identified by Purple, the Orange report might be very long and big in file size due to the circos plots displayed for each of the reportable drivers individually, which may be in the order of hundreds. To prevent this, the user might want to skip or limit the circos plots in the final report. I suggest to make this part to be optional based on a config parameter in the Linx config so that the user may choose this optionally. Alternatively, a tool parameter to plot a maximum of n number of plots can be specified after ranking the driver events. Currently, I get like hundreds of circos plots in the Orange report which is dozens of pages long and several hundreds of MB file size which makes the report hard to navigate and share.

@bounlu bounlu added the enhancement New feature or request label Nov 29, 2024
@bounlu bounlu changed the title Make circos plots optional in the Orange report Make circos plots optional in the orange report Dec 2, 2024
@scwatts
Copy link
Collaborator

scwatts commented Dec 3, 2024

Hi @bounlu, this is an issue that I've run into myself and have thought about ways to solve, including an approach similar to your suggestion. For the oncoanalyser 1.0.0 release there wasn't a straightforward solution since the particular version of ORANGE used always required LINX plot inputs.

The latest release of ORANGE no longer has this requirement for LINX plot inputs, so this can be addressed in the next oncoanalyser release. I would say the most simple approach would to be avoid including LINX plots in the ORANGE report if there are more than some user-configurable amount.

I'll add this to our milestones for the 2.0.0 release, thanks!

@scwatts scwatts self-assigned this Dec 3, 2024
@scwatts scwatts added this to the Release 2.0.0 milestone Dec 3, 2024
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
None yet
Development

No branches or pull requests

2 participants