-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix csrc readme #9515
fix csrc readme #9515
Conversation
Thanks for your contribution! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #9515 +/- ##
===========================================
+ Coverage 52.87% 53.03% +0.15%
===========================================
Files 687 694 +7
Lines 109186 110190 +1004
===========================================
+ Hits 57727 58434 +707
- Misses 51459 51756 +297 ☔ View full report in Codecov by Sentry. |
llm/predict/export_model.py
Outdated
@@ -18,7 +18,7 @@ | |||
|
|||
import paddle | |||
from paddle.distributed import fleet | |||
from predict.predictor import ModelArgument, PredictorArgument, create_predictor | |||
from llm.predict.predictor import ModelArgument, PredictorArgument, create_predictor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里的话,这样就假定根目录是 PaddleNLP 不是 PaddleNLP/llm了。对大多数case 可能有问题。另外起pr改吧。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
是指CI上的单测吗?那根目录设置在那里改呢?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
因为一般根目录大家都会设为PaddleNLP
cb9fdcc
to
47cfeb7
Compare
PR types
Others
PR changes
Docs
Description
fix csrc readme