Skip to content
New issue

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

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

Already on GitHub? # to your account

Add explanation of how to enable maths feature #681

Merged
merged 4 commits into from
Oct 9, 2024

Conversation

olgabot
Copy link
Contributor

@olgabot olgabot commented Sep 30, 2024

Hello,
I hope you are doing well! I am relatively new to Rust, and was trying to use MathematicalOps from this crate (sourmash-bio/sourmash_plugin_branchwater#458), but I could not for the life of me figure out how to "enable the maths feature" as mentioned in the documentation. I was finally able to see an example of how to do it after reading this PR comment: #431 (comment)

This PR adds a bit more text to the MathematicalOps documentation to help future users more quickly be able to use stably computed logarithms!

Warmest,
Olga

@Tony-Samuels
Copy link
Collaborator

Thank you for the contribution. Please can you run cargo fmt over the changes?

@olgabot
Copy link
Contributor Author

olgabot commented Oct 8, 2024

Thank you for the contribution. Please can you run cargo fmt over the changes?

Done!

@Tony-Samuels Tony-Samuels enabled auto-merge October 8, 2024 19:05
@Tony-Samuels Tony-Samuels merged commit fa88579 into paupino:master Oct 9, 2024
15 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants