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

Creating a function to compare sequences of DNA #57

Closed
wants to merge 11 commits into from

Conversation

hssn-20
Copy link
Contributor

@hssn-20 hssn-20 commented Nov 23, 2022

The functions in this file make use of the sourmash library to compare sequences of DNA. There are a few parameter choices ( i.e. number of hashes and k-mer sizes) I've made default however we can change them at a later point in the future.

@mateibejan1
Copy link
Collaborator

mateibejan1 commented Nov 29, 2022

Can we rename the similarity function to contain the type of similarity computed?

hssn-20 and others added 3 commits March 8, 2023 17:17
@hssn-20
Copy link
Contributor Author

hssn-20 commented Mar 9, 2023

Done

@hssn-20
Copy link
Contributor Author

hssn-20 commented Mar 9, 2023

I've also added a function to ingest fasta files though I'm not sure if its needed.

@cameronraysmith cameronraysmith added this to the 0.0.1 milestone Mar 10, 2023
@cameronraysmith cameronraysmith added the metrics modifies definition or measurement of model metrics label Mar 10, 2023
Copy link
Collaborator

@cameronraysmith cameronraysmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many thanks @hssn-20!

It looks like something unintentional may have happened here resulting in changes to many more files than you intended to edit. The issue causing this was resolved by @ssenan in #99 .

Could you please synchronize your fork default branch to the tip of this upstream, make a branch for this particular contribution such as hssn-20:kmer_metric, and submit the PR from that branch instead?

You may find some of the information in the info pane of the project here helpful.

We will likely close this PR in the meantime, but you can still find it in the list of closed PRs. If we misunderstood what happened here we can always reopen it.

Related to #39.

@cameronraysmith cameronraysmith removed the request for review from ssenan March 10, 2023 00:57
@ssenan ssenan self-requested a review March 10, 2023 01:18
@cameronraysmith
Copy link
Collaborator

cameronraysmith commented Mar 10, 2023

Please rebase after #99 or later and resubmit from a non-default branch on your fork so that we can see only the intended changes.

Apologies for the inconvenience and thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metrics modifies definition or measurement of model metrics
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Implement a sequence quality metric based on k-mer composition
3 participants