-
-
Notifications
You must be signed in to change notification settings - Fork 41
[REVIEW]: Explainable Artificial Intelligence with MicroPython: Lightweight Neural Networks for Students’ Deeper Learning #8039
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
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: ✅ License found: |
Review checklist for @kalpan80Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @samiralaviConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@osorensen - Noted a typo in the paper, Page 2, paragraph 2, Line 3. Lightweight is spelled incorrectly as leightweight. |
@kalpan80 This made me sweat: Couldn't find the word “leightweight” anywhere in the JOSS paper... then I found it in the ArXiv PrePrint / Tutorial. THX :-) |
@statistical-thinking and @osorensen - Have completed my review. Thank you |
@kalpan80 Thanks for the quick review! Since AI-ANNE can be put into operation simply by transferring main.py to the Raspberry Pi Pico and maps the exact results of a pre-trained neural network from Python to MicroPython via the transfer of weights and biases, I wouldn't know how to further demonstrate the functionality :-) But I have added the transfer of main.py (with hardware) or ai-anne.py (without hardware) again on GitHub and added the above mentioned functionality in the section “How to Use and Support AI-ANNE”. I have also added the Community Guidelines there... Thanks for your feedback! |
Thanks @kalpan80. Do you have any specific suggestions or comments related to how @statistical-thinking can address the unchecked items on your review checklist? |
Thank you for your responses. Agree with @statistical-thinking on the functionality aspect of the library. Have marked it as completed. Community guidelines - Any enthusiastic contributor can be allowed to create a PR for the KI.ENNA project. If the PR is found helpful, author can decide to merge it with the master branch. Automated unit testing - We can leverage some of the pointers mentioned in below article. |
@kalpan80 @osorensen Thanks again for your input! I have added a test_ai_anne() function in the basic code of AI-ANNE. This is based on a small neural network with just a few neurons and the easy-to-understand sigmoid function. The successful calculation of a Confusion Matrix and the Accuracy are defined as test criteria, as these are the goal of AI-ANNE. I had considered implementing the Pi Pico's built-in LED as an additional signal in the test, but since not all users will have a Pi Pico and teaching with AI-ANNE can also take place only in Thonny as software environment, I decided against it :-) |
Thank you @statistical-thinking and @osorensen |
I have just added a GIF to the GitHub Readme, which shows AI-ANNE in use as a didactic tool, where you can select and compare different neural networks. In addition to the simple codes in Thonny, this is the most fun for learners... |
Submitting author: @statistical-thinking (Prof. Dr. habil. Dennis Klinkhammer)
Repository: https://github.com/statistical-thinking/KI.ENNA
Branch with paper.md (empty if default branch):
Version: 2.0
Editor: @osorensen
Reviewers: @samiralavi, @kalpan80
Archive: Pending
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@samiralavi & @kalpan80, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @osorensen know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Checklists
📝 Checklist for @kalpan80
📝 Checklist for @samiralavi
The text was updated successfully, but these errors were encountered: