AdamaJava / adamajava

Other
15 stars 5 forks source link

Qmotif revisit #379

Closed holmeso closed 1 month ago

holmeso commented 1 month ago

Description

Updated some of the qmotif documentation, adding the ini section, and adding an explanation of the elements in the summary section.

Fixed a bug where if you run in includes_only: false mode on GRCh38, you get a "Too many open files" error as it creates (and I guess opens) SAMFileReader objects for all the contigs in the genome, of which there are lots for GRCh38.

Some minor mode that my trusty copilot suggested.

Type of change

How Has This Been Tested?

Unit tests pass. New code was run against a COLO BAM and compared to the output of the existing code run against the same BAM. They were identical.

Are WDL Updates Required?

No. No new options names etc

Checklist: