NBISweden / swefreq

Swedish Frequency resource for genomics website
https://swefreq.nbis.se/
GNU General Public License v3.0
8 stars 1 forks source link

BND code cleanup #592

Closed MalinAhlberg closed 5 years ago

MalinAhlberg commented 5 years ago

Describe the pull request:

Pull request long description:

Clean up of the code for parsing manta files / adding structural variants / breakends. Includes both plain refactoring and functional changes.

Changes made:

Functional changes:

  1. 14c498c Assumption: the manta files should always add data to an existing dataset, and should therefore never add / change counts for the whole dataset. The code doing this is removed, to avoid unintended changes of the dataset counts.

  2. d1f783b The counter now counts: +1 for every row. This information is only used for logging and the progress bar, due to the commit above. No extra counting of reversed mates is done, since count_entries does not count these.

Bug fix:

  1. be0458c Bug fix: remove not so that the progress bar can be updated.

Code cleanup:

  1. 1cf6470 Improve comments etc.

  2. 69d9f01 Make the function parse_manta less long by breaking out parts into its own function.

  3. 1e698c5, 5521dd6 Break out various code from parse_manta.

TODOs

Look at separation between functions and methods (eg parse_info vs parse_baseinfo).