Skip to content

DOC Add 2 related projects for microcontroller export#26984

Merged
Micky774 merged 2 commits intoscikit-learn:mainfrom
jonnor:related-embedded
Aug 2, 2023
Merged

DOC Add 2 related projects for microcontroller export#26984
Micky774 merged 2 commits intoscikit-learn:mainfrom
jonnor:related-embedded

Conversation

@jonnor
Copy link
Copy Markdown
Contributor

@jonnor jonnor commented Aug 1, 2023

What does this implement/fix?

Documentation only change.
Added a couple of projects to "related projects" under "Model export for production".
These are similar to other tools that are already present, like m2cgen, sklearn-porter and sklearn-onnx - but are focused/specialized for microcontrollers/embedded (think Arduino).
Both have existed for several years, have a reasonable amount of visbility on Github. They are also both referenced in several academic papers within this niche.

Any other comments?

Disclaimer: I am the maintainer of emlearn, which was added. No affiliation with micromlgen, the other project

First pull request. Have attempted to follow the contribution guidelines as best as I could. Will happily adjust to feedback

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 1, 2023

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 1919650. Link to the linter CI: here

Copy link
Copy Markdown
Contributor

@Micky774 Micky774 left a comment

Choose a reason for hiding this comment

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

I always enjoy seeing scikit-learn ecosystem integrations 😄. Thanks for the PR!

@Micky774 Micky774 enabled auto-merge (squash) August 2, 2023 13:33
@jonnor
Copy link
Copy Markdown
Contributor Author

jonnor commented Aug 2, 2023

That is good to hear. Thank you for the fast review @Micky774 !

@Micky774 Micky774 merged commit 9c96671 into scikit-learn:main Aug 2, 2023
TamaraAtanasoska pushed a commit to TamaraAtanasoska/scikit-learn that referenced this pull request Aug 21, 2023
…6984)

Co-authored-by: Meekail Zain <34613774+Micky774@users.noreply.github.com>
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Sep 18, 2023
…6984)

Co-authored-by: Meekail Zain <34613774+Micky774@users.noreply.github.com>
jeremiedbb pushed a commit that referenced this pull request Sep 20, 2023
Co-authored-by: Meekail Zain <34613774+Micky774@users.noreply.github.com>
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
…6984)

Co-authored-by: Meekail Zain <34613774+Micky774@users.noreply.github.com>
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.

2 participants