[Build] deduplicate data_feed Python proto generation

[ooooo-create]

PR Category

Execute Infrastructure

PR Types

Others

Description

😺 在 #78713 的 coverage test 流水线的单测 test_deprecated_decorator 和 test_trainer_desc 一直挂，顾让 codex 来帮忙分析一下，由于没有实际环境和产物并没有分析出合理原因，后来重跑 coverage clone 通过，但是 codex 对代码提出了修改建议，觉得有道理就提交上来了

🤖 本 PR 去掉了 data_feed.proto 在 Python proto 构建中的重复声明，让 data_feed_pb2.py 只由一个 CMake target 生成（😺 py_proto_compile 的 SRCS 之后的参数，是根据 xxx.proto 生成 xxx_pb2.py 文件）；同时将 python/paddle/base/trainer_desc.py 中的 trainer_desc_pb2 导入改为包内导入，简化 Python 侧 proto 加载路径。

验证

• prek
• ./.venv/bin/python -m py_compile python/paddle/base/trainer_desc.py

是否引起精度变化

否

[paddle-bot]

你的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.

[Copilot]

Pull request overview

This PR removes duplicate Python protobuf generation rules for data_feed.proto so data_feed_pb2.py is generated by a single CMake target, and updates TrainerDesc to import trainer_desc_pb2 via an in-package relative import to make the Python proto import path more deterministic.

Changes:

• Deduplicate Python proto generation by removing data_feed.proto from the trainer_py_proto CMake target inputs.
• Replace the sys.path-based protobuf import workaround in python/paddle/base/trainer_desc.py with from .proto import trainer_desc_pb2.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
python/paddle/base/trainer_desc.py	Switches trainer_desc_pb2 import to a package-relative import and removes sys.path manipulation.
paddle/fluid/framework/CMakeLists.txt	Ensures data_feed.proto is only included once in Python proto generation targets (via framework_py_proto).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@adbe498). Learn more about missing BASE report.

Additional details and impacted files

@@             Coverage Diff             @@
##             develop    #78718   +/-   ##
===========================================
  Coverage           ?   100.00%           
===========================================
  Files              ?         1           
  Lines              ?         1           
  Branches           ?         0           
===========================================
  Hits               ?         1           
  Misses             ?         0           
  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.

[SigureMo]

对比修改前后生成的 trainer_desc_pb2.py，一致吗？

[ooooo-create]

是的,一样的,用 CI 上的包对比过之前的 https://paddle-github-action.bj.bcebos.com/PR/h-coverage/78718/e007b3d66f6d22bdb200d2ba1540156707eec531/paddlepaddle_gpu-0.0.0-cp310-cp310-linux_x86_64.whl