Update Mm/Ml tag name handling to meet the SAM spec

streetslab / dimelo

python package for analysis of dimelo-seq & nanopore modified base data

MIT License

3 stars 5 forks source link

Update Mm/Ml tag name handling to meet the SAM spec #33

Open thekugelmeister opened 1 year ago

thekugelmeister commented 1 year ago

According to the SAM file spec (https://samtools.github.io/hts-specs/SAMv1.pdf), the Mm/Ml tags are intended to now be fully uppercase (MM/ML). Currently the package is hard-coded to only use the mixed-case versions.

kennisjoseph commented 1 year ago

Hey there, I want to fix this issue as I new to open source and finding some good-first-issues!

thekugelmeister commented 1 year ago

Hi! Thank you for your interest. At this moment we are in the midst of a larger restructure, and fixing this now may not be useful in the long term. If you are interested in contributing in the future, you are welcome to come back once we have made progress on the greater restructuring.