Skip to content

Comments

[docs] Rename train_step_batch_size to train_micro_batch_size_per_gpu #1066

Merged
tjruwase merged 2 commits intodeepspeedai:masterfrom
wbuchwalter:wibuch/fix-doc
May 12, 2021
Merged

[docs] Rename train_step_batch_size to train_micro_batch_size_per_gpu #1066
tjruwase merged 2 commits intodeepspeedai:masterfrom
wbuchwalter:wibuch/fix-doc

Conversation

@wbuchwalter
Copy link
Contributor

Config file doc uses both train_step_batch_size and train_micro_batch_size_per_gpu to refer to the same thing, this is a bit confusing.
Renamed everything to train_micro_batch_size_per_gpu which is the actual key name in the config json.

@wbuchwalter
Copy link
Contributor Author

@tjruwase Please check updated version.
Also I'm not quite sure how to test that I didn't break the Jekyll formatting.

@tjruwase
Copy link
Contributor

@tjruwase Please check updated version.
Also I'm not quite sure how to test that I didn't break the Jekyll formatting.

Thanks so much for doing this. The Jekyll formatting looks fine. I will merge.

@tjruwase tjruwase merged commit 1f82ab7 into deepspeedai:master May 12, 2021
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