Skip to content

fix bugs when compiling with VS2022#75611

Closed
youge325 wants to merge 27 commits intoPaddlePaddle:developfrom
youge325:vs2022
Closed

fix bugs when compiling with VS2022#75611
youge325 wants to merge 27 commits intoPaddlePaddle:developfrom
youge325:vs2022

Conversation

@youge325
Copy link
Copy Markdown
Contributor

PR Category

Environment Adaptation

PR Types

Bug fixes

Description

修复了使用VS2022编译时遇到的bug,目前cmake .. -GNinja -DWITH_GPU=OFF -DWITH_UNITY_BUILD=ON可以正确编译CPU版本,paddle.utils.run_check()也能跑通,但是GPU版本对CUDA13.0的适配还有点问题,后面会再提一个PR

@paddle-bot
Copy link
Copy Markdown

paddle-bot Bot commented Sep 29, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot Bot added the contributor External developers label Sep 29, 2025
@youge325
Copy link
Copy Markdown
Contributor Author

youge325 commented Oct 1, 2025

使用cuda13在Linux上编译GPU版本的的wheel包无法链接cublas的问题已在 #75509 中解决,在Windows上的解决方案已在 #75654 中解决,cmake命令cmake .. -GNinja -DWITH_GPU=ON -DWITH_UNITY_BUILD=ON -DCUDA_ARCH_NAME=Auto -DWITH_TENSORRT=ON -DTENSORRT_ROOT=D:\TensorRT-10.13.3.9,在RTX3070上编译,驱动程序版本581.29,cuda版本13.0.1,cudnn版本9.13.1,paddle.utils.run_check()可以正常跑通

@youge325 youge325 changed the title [API compatibility] fix bugs when compile with VS2022 fix bugs when compiling with VS2022 Oct 2, 2025
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Oct 9, 2025
@luotao1 luotao1 assigned luotao1, YqGe585 and swgu98 and unassigned YqGe585 Oct 9, 2025
@youge325
Copy link
Copy Markdown
Contributor Author

youge325 commented Oct 9, 2025

/re-run all-failed

@youge325 youge325 marked this pull request as draft October 10, 2025 00:58
@youge325 youge325 marked this pull request as ready for review October 10, 2025 00:59
@youge325 youge325 marked this pull request as draft October 10, 2025 00:59
@youge325 youge325 marked this pull request as ready for review October 10, 2025 01:03
@youge325
Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@youge325
Copy link
Copy Markdown
Contributor Author

#75684 由于代码规范原因尚未合入,这里先提交修改保证通过CI

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@9f7f2ba). Learn more about missing BASE report.

Files with missing lines Patch % Lines
paddle/phi/core/framework/data_type_transform.cc 0.00% 16 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop   #75611   +/-   ##
==========================================
  Coverage           ?    0.00%           
==========================================
  Files              ?        1           
  Lines              ?       16           
  Branches           ?        0           
==========================================
  Hits               ?        0           
  Misses             ?       16           
  Partials           ?        0           

☔ 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.

@youge325
Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@youge325
Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@youge325
Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@youge325 youge325 marked this pull request as draft October 13, 2025 06:23
@youge325 youge325 closed this Oct 28, 2025
@youge325 youge325 deleted the vs2022 branch November 8, 2025 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants