broadinstitute / gatk-sv

A structural variation pipeline for short-read sequencing
BSD 3-Clause "New" or "Revised" License
170 stars 70 forks source link

Making the Vapor plots optional to store as a final output #692

Closed kirtanav98 closed 3 months ago

kirtanav98 commented 3 months ago

This PR addresses issue 586. Vapor produces a large amount of storage (~1.5 GiB per sample), 99% of which is related to the plots. To reduce storage, a boolean parameter 'save_plots' was added to the WDL to toggle whether those plots are kept as a final workflow output or not. When set to false, the plots are not produced as a final output. When set to true, the plots are produced as a final output. The updated wdl and json were tested using womtool with the 1kgp data for both instances, when the parameter is set to false and when it is set to true. Testing was also done by running updated script in Terra with the boolean set to "true" and "false" to ensure the plots were generated or not as per the input.