-
Notifications
You must be signed in to change notification settings - Fork 179
Topic recognition #168
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
base: topic-recognition
Are you sure you want to change the base?
Topic recognition #168
Conversation
…ttention constructor and transpose_for_scores utility function
…were removed for code simplicity previously)
… image data- reconfigured dataset.py so that it contains data loader for loading and preprocessing (simplified)
…an unrelated brain-dump solution from earlier coursework. Current model training development is being conducted in Colab, and a functional version will be converted from a Jupyter notebook to a .py file
…or CViT. Code migrated code from a Jupyter notebook to a standalone script
…GitHub markdown. MS doc was populated while updating the model for documentation- incomplete
…mit had an empty file, possibly due to sourcing from a different repository
Not relevant- was used for preliminary background context of images
This is an initial inspection, no action is required at this point Difficulty: Hard Readme: Commit messages: Excellent, detailed. Code:
Functionality/Performance:
General comments. |
MarkingGood Practice (Design/Commenting, TF/Torch Usage)Adequate design and implementation Recognition ProblemSolves problem Commit LogMeaningful commit messages DocumentationReadMe acceptable/good Pull RequestSuccessful Pull Request (Working Algorithm Delivered on Time in Correct Branch) |
Can't merge into repo because of large model file, but Feedback marks possible +2 if the requested changes are made (see above). |
No feedback attempt and no feedback marks granted. |
I am working on feedback- I am currently in hospital so it is difficult for me to do so. |
I just marked all problem pull requests and was making sure I had covered everyone. You can still get marks after finishing the adjustment. I will get another inspection before the due date. However, we cannot wait for endless time, we have to update your grades to the school in a few days. Please adjust it ASAP. |
This pull request contains my implementation of a Convolutional Vision Transformer (CViT) for classification of Alzheimer’s Disease using the ADNI dataset (Task 6). Files included are:
- dataset.py: data loader for loading and pre-processing data
- modules.py: source code of the CViT components
- train.py: source code for training, validating and saving model
- predict.py: example usage of trained model
- ./Images: images detailing results of testing, and supporting images for
- README.md: thorough documentation of project
- Models.pth: trained model weights saved
Thanks,
Noam Mendelson (47432923)