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

Initial attempt at implementing resnet50 for use with CIFAR data. #7

Merged
merged 2 commits into from
Oct 11, 2024

Conversation

drewoldag
Copy link
Contributor

Attempting to run this from the command line with:
fibad train --runtime-config example_config.toml

However, it's not currently running for a couple of reasons:

  1. Resnet50 accepts an input, num_classes which we can define in our config, but if num_classes isn't also defined in the main FIBAD config, it throws an error. This is a FIBAD bug that must be addressed.
  2. Something is wrong with the example CIFAR dataset. It doesn't seem to load as expected. This is likely also a bug on the FIBAD side that is a result of the work to move away from DataLoaders and toward Datasets.

@drewoldag drewoldag self-assigned this Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

Before [d5515ed] After [e5807bc] Ratio Benchmark (Parameter)
2.45±0.6s 2.17±0.5s ~0.89 benchmarks.time_computation
1.74k 3.05k 1.76 benchmarks.mem_list

Click here to view all benchmarks.

@drewoldag drewoldag requested a review from maxwest-uw October 11, 2024 20:34
@drewoldag drewoldag merged commit 6a7bf63 into main Oct 11, 2024
3 of 8 checks passed
@drewoldag drewoldag deleted the awo/implement-resnet50-model branch October 11, 2024 20:35
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