Skip to content

[feature]add fd whl version info #2698

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

Merged
merged 2 commits into from
Jul 4, 2025

Conversation

gzy19990617
Copy link
Contributor

add whl version info like this:
image

@CLAassistant
Copy link

CLAassistant commented Jul 3, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

paddle-bot bot commented Jul 3, 2025

Thanks for your contribution!

@@ -22,6 +22,7 @@
os.environ["GLOG_minloglevel"] = "2"
# suppress log from aistudio
os.environ["AISTUDIO_LOG"] = "critical"
from .utils import version
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
from .utils import version
from fastdeploy.utils import version

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

@gzy19990617 gzy19990617 force-pushed the develop branch 2 times, most recently from a204b39 to af01776 Compare July 3, 2025 12:56
@ming1753
Copy link
Collaborator

ming1753 commented Jul 3, 2025

scripts里面的build_wheel_pipeline_cu123.sh看下还要不要用,不用就删掉了,用的话也要加上

@gzy19990617
Copy link
Contributor Author

scripts里面的build_wheel_pipeline_cu123.sh看下还要不要用,不用就删掉了,用的话也要加上

确认不用了,删掉了

Copy link
Collaborator

@ming1753 ming1753 left a comment

Choose a reason for hiding this comment

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

LGTM

@yuanlehome
Copy link
Collaborator

能不能支持成这种形式,跟主框架保持一致
image

@Jiang-Jia-Jun Jiang-Jia-Jun merged commit a5ae88d into PaddlePaddle:develop Jul 4, 2025
2 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.

5 participants