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

Update train.py #3409

Merged
merged 2 commits into from
Jun 18, 2021
Merged

Update train.py #3409

merged 2 commits into from
Jun 18, 2021

Conversation

beet-aizu
Copy link
Contributor

Fix the help message of the argument "workers"

Fix the help message of the argument "workers"
@facebook-github-bot
Copy link

Hi @beet-aizu!

Thank you for your pull request and welcome to our community.

Action Required

In order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you.

Process

In order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with CLA signed. The tagging process may take up to 1 hour after signing. Please give it that time before contacting us about it.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

@codecov
Copy link

codecov bot commented Feb 16, 2021

Codecov Report

Merging #3409 (9886501) into master (0043e5f) will increase coverage by 17.95%.
The diff coverage is n/a.

❗ Current head 9886501 differs from pull request most recent head 6478dd9. Consider uploading reports for the commit 6478dd9 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           master    #3409       +/-   ##
===========================================
+ Coverage   56.87%   74.83%   +17.95%     
===========================================
  Files          43      105       +62     
  Lines        3562     9722     +6160     
  Branches      546     1563     +1017     
===========================================
+ Hits         2026     7275     +5249     
- Misses       1416     1960      +544     
- Partials      120      487      +367     
Impacted Files Coverage Δ
torchvision/utils.py 59.57% <0.00%> (-1.97%) ⬇️
torchvision/models/densenet.py 83.58% <0.00%> (-1.65%) ⬇️
torchvision/ops/__init__.py 100.00% <0.00%> (ø)
torchvision/models/__init__.py 100.00% <0.00%> (ø)
torchvision/datasets/__init__.py 100.00% <0.00%> (ø)
torchvision/transforms/__init__.py 100.00% <0.00%> (ø)
torchvision/models/quantization/mobilenetv2.py 87.75% <0.00%> (ø)
torchvision/models/mnasnet.py 75.75% <0.00%> (ø)
torchvision/datasets/samplers/__init__.py 100.00% <0.00%> (ø)
torchvision/ops/poolers.py 86.53% <0.00%> (ø)
... and 94 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 70b2edb...6478dd9. Read the comment docs.

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Good catch! thanks @beet-aizu

@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@NicolasHug NicolasHug merged commit 68b128d into pytorch:master Jun 18, 2021
facebook-github-bot pushed a commit that referenced this pull request Jun 21, 2021
….py (#3409)

Reviewed By: fmassa

Differential Revision: D29264324

fbshipit-source-id: 5f65e31adce7ee6b1698564972e28be3d26e5a59
@beet-aizu beet-aizu deleted the patch-1 branch November 25, 2022 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants