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

Add a cli tool to test nn training functionality #233

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thodkatz
Copy link
Collaborator

This PR adds a command line tool to test the functionality of the neural network training. I have included a README.md for its usage.

I wasn't really sure where to add it, I have created a seperate dir nn_training_cli, under the tiktorch/dev.

Copy link

codecov bot commented Jan 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.40%. Comparing base (15a0225) to head (2f61bce).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #233   +/-   ##
=======================================
  Coverage   62.40%   62.40%           
=======================================
  Files          47       47           
  Lines        2878     2878           
=======================================
  Hits         1796     1796           
  Misses       1082     1082           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thodkatz
Copy link
Collaborator Author

thodkatz commented Feb 12, 2025

To test it on a gpu, change the device attribute of tiktorch/dev/nn_training_cli/pytorch3d_unet_config.yaml from device: cpu to device: cuda.

Related issue #234, regarding silicon compatibility.

@k-dominik
Copy link
Collaborator

k-dominik commented Feb 12, 2025

Cool! MPS currently is still lacking support for certain operations that prevent it from running in 3D (Def. MaxPool3D is still missing). 2D should work by setting device: mps I'll try :)

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