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

PyTorch Versions of TC1 and P1B2 Benchmarks #96

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

knangia04
Copy link

  • Code is generic and handles the neural network model as defined in the model_pytorch.txt file (similar to the TensorFlow/Keras version).
  • baseline_pytorch.py is the main function of the benchmark.
  • Updated pytorch_utils.py to add additional functionality.
  • torch_deps folder consists of dependencies that are required for the PyTorch benchmark, including a model class, dataset DataLoader class, classification network, training, validation, and test functions, and weight initialization.

* baseline_pytorch.py is the main function of the benchmark.
* Updated pytorch_utils.py to add additional functionality.
* torch_deps folder consists of dependencies that are required for the benchmark to function correctly, including the model, dataset, classification network, training, validation, and test functions, and weight initialization.
Copy link
Contributor

@hyoo hyoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good work. I suggest a few changes to keep the code consistent and maintainable. Will add a few more as we move on.

Pilot1/P1B2/p1b2_baseline_pytorch.py Outdated Show resolved Hide resolved
Pilot1/P1B2/p1b2_default_model_pytorch.txt Outdated Show resolved Hide resolved
Pilot1/P1B2/pytorch_utils.py Outdated Show resolved Hide resolved
@jmohdyusof
Copy link
Contributor

How does this commit relate to #95 which seems to be similar (at least in name)

1. Updated to use the original default_model txt file.
2. Updated common/pytorch_utils.py based on the the local pytorch_utils.py and deleted the local versions.
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.

3 participants