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

Fix typo s/might have reach/might have reached #623

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

kinow
Copy link
Contributor

@kinow kinow commented Apr 25, 2022

Hi,

Quick fix for a typo. Also, I had to Google Bayes error rate after reading this paragraph, but I couldn't find a link in the notebook for that. Are there any plans to have footnotes (like in GitHub Markdown 1) with useful links? Or maybe these links are in the forum or somewhere else?

Thanks!

Footnotes

  1. for example

@ArturoAmorQ
Copy link
Collaborator

Are there any plans to have footnotes (like in GitHub Markdown 1) with useful links? Or maybe these links are in the forum or somewhere else?

I think having footnotes is a good idea, maybe you can join the discussion in Issue #427 and add the term "Bayes error rate" there.

Copy link
Collaborator

@ArturoAmorQ ArturoAmorQ left a comment

Choose a reason for hiding this comment

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

On the meantime, this PR LGTM as it is, thanks for the contribution @kinow!

@ArturoAmorQ ArturoAmorQ merged commit c85ba14 into INRIA:main Apr 26, 2022
github-actions bot pushed a commit that referenced this pull request Apr 26, 2022
@kinow kinow deleted the typo-bayes-rate branch April 26, 2022 09:46
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.

2 participants