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.

Sign up for GitHub

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

[Refactor] SFTTrainer SFTConfig #9318

Merged
merged 20 commits into from
Nov 28, 2024

add missing.

6356a56
Select commit
Loading
Failed to load commit list.
Merged

[Refactor] SFTTrainer SFTConfig #9318

add missing.
6356a56
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 28, 2024 in 0s

79.87% of diff hit (target 80.00%)

View this Pull Request on Codecov

79.87% of diff hit (target 80.00%)

Annotations

Check warning on line 117 in paddlenlp/datasets/dataset.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

paddlenlp/datasets/dataset.py#L117 

Added line #L117 was not covered by tests

Check warning on line 54 in paddlenlp/trl/sft_config.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

paddlenlp/trl/sft_config.py#L54 

Added line #L54 was not covered by tests

Check warning on line 76 in paddlenlp/trl/sft_config.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

paddlenlp/trl/sft_config.py#L56-L76 

Added lines #L56 - L76 were not covered by tests

Check warning on line 16 in paddlenlp/trl/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

paddlenlp/trl/utils.py#L15-L16 

Added lines #L15 - L16 were not covered by tests

Check warning on line 20 in paddlenlp/trl/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

paddlenlp/trl/utils.py#L19-L20 

Added lines #L19 - L20 were not covered by tests

Check warning on line 43 in paddlenlp/trl/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

paddlenlp/trl/utils.py#L39-L43 

Added lines #L39 - L43 were not covered by tests
View more details on Codecov