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

Create a DNAMutation class to mutate DNA sequences #460

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Paucey
Copy link

@Paucey Paucey commented Jan 30, 2025

Hi, first time contributor to this project and wanted to help. I took a look at issue #457 and came up with this.

Introduces a DNAMutation class which handles generating mutated DNA sequences. Specifically:

Mutate a DNA sequence by randomly changing a specified number of bases.
Generate 10 mutated sequences and print them to the console.

generateMutatedSequences(): Given a DNA sequence generates 10 mutated sequences, each with a specified number of mutations, and prints them to the console.

mutate(): Applies mutations by randomly selecting positions in the given DNA sequence and changing the base at that position.

getDifferentBase(): Ensures the mutation results in a base different from the given original base.

Related Issue: #457

Not sure if this was specifically what you were looking for (and it isn't integrated any further than just being the abstract methods) but again, wanted to contribute. Let me know if you would like anything tweaked.

@CLAassistant
Copy link

CLAassistant commented Jan 30, 2025

CLA assistant check
All committers have signed the CLA.

@Paucey
Copy link
Author

Paucey commented Jan 30, 2025

Currently mutate() simply replaces one base in a sequence with another base, but it could be refined to offer more specific kinds of mutations.

@VerisimilitudeX
Copy link
Owner

Hi @Paucey, thank you so much for your contribution! Taking a preliminary glance-through of your changes, everything looks good. I’ll look over it more in-depth over the weekend and will come back to you with feedback!

@Paucey
Copy link
Author

Paucey commented Jan 31, 2025

Thank you, let me know what you think! Happy to make any changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants