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

Project 2: Caroline Lachanski #24

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

clach
Copy link

@clach clach commented Sep 19, 2019

  • Repo link

  • Implemented various implementations of scan and stream compaction

  • Implemented all code for character recognition

  • XOR example seems to work, actual character recognition seems to have issues with back propagation

  • I tried very very hard (and almost destroyed my repo in the process) to push my weights file for character recognition but it was too large (if you need it I have it and can send it another way)

  • Feedback on project: This project (really two projects) was an insane amount of work, especially for someone who's never done machine learning. At the very least, the readme could have been a little more descriptive about how back propagation works, or provide tips on how to organize/pass around data and create kernels for the process. Additionally, it was frustrating to later see stream compaction elevated to the more important assignment, especially after we were told to finish it quickly and spend more time on character recognition. I would have liked to have more spent more time actually optimizing my scan and stream compaction code and doing extra credit, rather than rushing through two assignments.

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.

1 participant