dorado
dorado copied to clipboard
--modified-bases all option
Hi
I think for an improved user experience it would be great to have an option to activate "all" modifications without manually specifying the models.
Another thing I noticed is that it is confusing that --modified-bases takes a space separated list whereas --modified-bases-models takes a comma separated list. Probably easier to stick with one way of supplying arguments.
Best regards Rasmus
Hey @Kirk3gaard
What do you imagine all
doing with all-context models vs context models, i.e.5mC
and 5mCG
for example?
Good point on the consistency of the argument separators.
Hi @iiSeymour
Detecting all modifications that you have models for. I think for most users whether they want to call modifications or not is a yes/no question rather than I only want to call 6mA or 5mC.
Best regards Rasmus
Good point of Rasmus
I second the feature mentioned by @Kirk3gaard
Interesting, but a couple of thoughts
- wouldn't specifying many models increase runtime of mod basecalling, or increase GPU RAM requirements?
- what about displaying mod bases as bigwig in downstream applications ? How is this handled, eg by modkit ?
- I guess most use cases and the best models are for 5mC
In general, I'd prefer to specify exactly which models I want unless all of the above points are solved or not detrimental.