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

[Question]: typo in the "Occurrence table pruning" reference site #1265

Closed
3 tasks done
adcascone opened this issue Jul 12, 2024 · 2 comments · Fixed by #1266
Closed
3 tasks done

[Question]: typo in the "Occurrence table pruning" reference site #1265

adcascone opened this issue Jul 12, 2024 · 2 comments · Fixed by #1266
Assignees
Labels
documentation Improvements or additions to documentation sme

Comments

@adcascone
Copy link

What is your question?

In the example section of the "Occurrence table pruning" reference site, should this text (and the associated output) be adjusted to reflect the usage of has_fraction_in_any_col()?

Current:

# `has_fraction_in_any_col`
any_atleast_five_percent <- has_fraction_in_any_col(atleast = 0.05, col_names = names(tab))
prune_table(tab, keep_rows(more_than_five_percent))

Suggested adjustment:

# `has_fraction_in_any_col`
any_atleast_five_percent <- has_fraction_in_any_col(atleast = 0.05, col_names = names(tab))
prune_table(tab, keep_rows(any_atleast_five_percent))

Thank you!
Arianna C.

Code of Conduct

  • I agree to follow this project's Code of Conduct.

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines.

Security Policy

  • I agree to follow this project's Security Policy.
@adcascone adcascone added the question Further information is requested label Jul 12, 2024
@edelarua edelarua self-assigned this Jul 12, 2024
@edelarua
Copy link
Contributor

Thanks for your question @adcascone! This is in fact a typo that I'll have corrected.

@edelarua edelarua added documentation Improvements or additions to documentation sme and removed question Further information is requested labels Jul 12, 2024
@adcascone
Copy link
Author

Thank you!

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

Successfully merging a pull request may close this issue.

2 participants