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

Minor improvements and some test results #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jkyamog
Copy link

@jkyamog jkyamog commented Dec 28, 2023

I have made some minor improvements:

  • README tweaks
  • add support for multi GPU on pytorch cifar100 and food101. I am still learning ML its the first time I used Pytorch so hopefully this is already acceptable enough.
  • test results for
    • M1 Pro (10 GPU + 16 GPU) with 16GB Ram: Very similar results to your 32GB, most slower slightly and can't run bigger batches. Interesting to see on Pytorch NLP at 128 batch size it was swapping a lot, however the additional 2GB ram of M3 Pro 18GB was enough for squeeze that run. Was slightly faster on pytorch cifar100, however I did see run slower at times significantly. Maybe I was running a background task, I rerun tensorflow food101 as my first run was 2,000 secs.. unexpected.
    • Nvidia 1070 4 cards. Old miner i3-7100 and 16GB ram. I revived my friend's old miner, the PSU wasn't enough for most benchmarks. Interesting though on pytorch cifar100 it was fastest on 3072 batch size and also beat single card numbers. However this is real evidence that multi-GPU doesn't scale well.... worse on the other incomplete benchmarks.

- python 3.11
- move dependcy scikit-learn
- just added nvidia/cuda notes.
…SU is not enough to run all 4 in anger. For the 3 and 4 GPU setup I also lowered the wattage from 180w to 150w (nvidia reference design). It was a bit unreliable. I am getting also core dump on tensorflow, need to further investigate it.
@jkyamog
Copy link
Author

jkyamog commented Dec 28, 2023

I can make this to smaller PR, if don't like to get other commits.

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