Skip to content

Conversation

@Rojuinex
Copy link
Contributor

Created a simple implementation in Zig following the Go pattern.

@dlidstrom
Copy link
Owner

Hi @Rojuinex! Thanks a lot for your contribution. This looks great and is a very nice addition. Also thank you for adding the test. Would you be able to add a GitHub Actions workflow for this implementation? Hopefully it is just a small addition to the .github/workflows/ci.yaml file.

@Rojuinex
Copy link
Contributor Author

@dlidstrom Thanks for the warm welcome 🙂 This repo def helped me shore up my understanding on how neural networks work at a fundamental level, so I'm very happy to contribute.

I've Updated the PR to include running the tests in the Github actions as requested. Let me know if there's anything else you'd like to see before merging it in.

@dlidstrom
Copy link
Owner

Looks great! I'll merge this now and will update the Readme too.

@dlidstrom dlidstrom merged commit 407b99b into dlidstrom:main Apr 10, 2025
10 checks passed
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