Skip to content

Conversation

@lewis-yeung
Copy link
Contributor

What does this PR do?

Correct docstrings of the following methods:

  • TrainingArguments.set_save
  • TrainingArguments.set_logging

Who can review?

@stevhliu

Copy link
Member

@stevhliu stevhliu 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 for the fixes!

@lewis-yeung lewis-yeung changed the title docs: correct docstrings of TrainingArguments.set_* methods Correct docstrings of TrainingArguments.set_* methods Oct 25, 2023
lewis-yeung and others added 3 commits October 26, 2023 10:04
Correct docstrings of these methods in `TrainingArguments`:

- `set_save`
- `set_logging`
@lewis-yeung lewis-yeung changed the title Correct docstrings of TrainingArguments.set_* methods Correct docstrings and a typo in comments Oct 26, 2023
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@stevhliu stevhliu merged commit 1892592 into huggingface:main Oct 26, 2023
EduardoPach pushed a commit to EduardoPach/transformers that referenced this pull request Nov 19, 2023
* docs(training_args): correct docstrings

Correct docstrings of these methods in `TrainingArguments`:

- `set_save`
- `set_logging`

* docs(training_args): adjust words in docstrings

Co-authored-by: Steven Liu <[email protected]>

* docs(trainer): correct a typo in comments

---------

Co-authored-by: Steven Liu <[email protected]>
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