Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

--modified-bases all option #227

Open
Kirk3gaard opened this issue Jun 7, 2023 · 5 comments
Open

--modified-bases all option #227

Kirk3gaard opened this issue Jun 7, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@Kirk3gaard
Copy link

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

@iiSeymour
Copy link
Member

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.

@Kirk3gaard
Copy link
Author

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

@gwl2
Copy link

gwl2 commented Jun 13, 2023

Good point of Rasmus

@amodrek
Copy link

amodrek commented Jun 15, 2023

I second the feature mentioned by @Kirk3gaard

@colindaven
Copy link

Interesting, but a couple of thoughts

  1. wouldn't specifying many models increase runtime of mod basecalling, or increase GPU RAM requirements?
  2. what about displaying mod bases as bigwig in downstream applications ? How is this handled, eg by modkit ?
  3. 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.

@HalfPhoton HalfPhoton added the enhancement New feature or request label Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants