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.

Sign up for GitHub

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

Jump to bottom

add seed to param string docs #1732

Closed
ctb opened this issue Aug 30, 2021 · 1 comment · Fixed by #2836
Closed

add seed to param string docs #1732

ctb opened this issue Aug 30, 2021 · 1 comment · Fixed by #2836
Labels
good first issue

Comments

@ctb
Copy link
Contributor

ctb commented Aug 30, 2021

per #1731, the command line docs for sourmash sketch do not mention seed, which (based on inspecting the code) should work!
@ctb ctb mentioned this issue Sep 23, 2021
Closed
@ctb
Copy link
Contributor Author

ctb commented Sep 23, 2021

Do the following:

In doc/sourmash-sketch.md, add the following text under ### Parameter strings -

* `seed=<int>` - set the random number seed used for k-mer hashing. This is for advanced users who want to choose a completely different set of k-mers for sketches! The default is 42.

@rialc13 rialc13 mentioned this issue Nov 13, 2023
Merged
@ctb ctb closed this as completed in #2836 Nov 25, 2023
ctb added a commit that referenced this issue Nov 25, 2023
@rialc13 @ctb
Added seed to parameter strings #1732 (#2836)
30aa79d 
This PR adds 'seed' to parameter string in doc/sourmash-sketch.md as per
#1732 . Fixes #1732 .

Co-authored-by: C. Titus Brown <titus@idyll.org>
@ctb ctb mentioned this issue Dec 12, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added seed to parameter strings #1732 rialc13/sourmash
1 participant
@ctb