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

Document SparseCoding #3709

Merged
merged 7 commits into from
May 18, 2024
Merged

Document SparseCoding #3709

merged 7 commits into from
May 18, 2024

Conversation

rcurtin
Copy link
Member

@rcurtin rcurtin commented May 17, 2024

I wrote up documentation for the SparseCoding class. You can see it here:

https://www.ratml.org/misc/mlpack-markdown-doc/user/methods/sparse_coding.html

I also changed SparseCoding so that it supports different MatTypes. Otherwise, not much was needed here, which made the documentation process pretty straightforward (hooray!).

Copy link
Member

@shrit shrit left a comment

Choose a reason for hiding this comment

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

Everything looks good to me, the docs are really nice and well-written with a lot of details. 👍

Copy link

@mlpack-bot mlpack-bot bot left a comment

Choose a reason for hiding this comment

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

Second approval provided automatically after 24 hours. 👍

@rcurtin rcurtin merged commit f12ec42 into mlpack:master May 18, 2024
18 of 20 checks passed
@rcurtin rcurtin deleted the sparse-coding-doc branch May 18, 2024 17:30
This was referenced May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants