Skip to content

Conversation

@ZHUI
Copy link
Contributor

@ZHUI ZHUI commented Oct 25, 2024

PR types

Others

PR changes

Others

Description

Others

@paddle-bot
Copy link

paddle-bot bot commented Oct 25, 2024

Thanks for your contribution!

@codecov
Copy link

codecov bot commented Oct 25, 2024

Codecov Report

❌ Patch coverage is 79.87421% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.07%. Comparing base (3ef14dc) to head (6356a56).
⚠️ Report is 673 commits behind head on develop.

Files with missing lines Patch % Lines
paddlenlp/trl/sft_config.py 45.00% 22 Missing ⚠️
paddlenlp/trl/utils.py 0.00% 9 Missing ⚠️
paddlenlp/datasets/dataset.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9318      +/-   ##
===========================================
- Coverage    53.09%   53.07%   -0.02%     
===========================================
  Files          699      699              
  Lines       110471   110348     -123     
===========================================
- Hits         58651    58569      -82     
+ Misses       51820    51779      -41     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@CLAassistant
Copy link

CLAassistant commented Nov 6, 2024

CLA assistant check
All committers have signed the CLA.

return fn

return docstring_decorator
from paddlenlp.trainer.utils.doc import add_start_docstrings
Copy link
Contributor

Choose a reason for hiding this comment

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

预训练的config没有考虑到

Copy link
Contributor Author

Choose a reason for hiding this comment

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

你说预训练的这部分也重构进入到主库吗?

@@ -0,0 +1,102 @@
# Copyright (c) 2024 PaddlePaddle Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

这里放到paddlenlp library我想到的一个问题是,如果用户想新增相关的参数就必须要改动library,因此最近developllm工具必须适配最新的library,看起来会导致一些兼容问题。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

可以引导 用户使用

  1. 正式版lib + 正式版 llm
  2. dev lib + dev llm

不要混用。

实际是上,大部分更新的参数,用户并不会使用到。

@ZHUI ZHUI merged commit 5926c30 into PaddlePaddle:develop Nov 28, 2024
6 of 12 checks passed
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.

4 participants