Skip to content

Conversation

@HydrogenSulfate
Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate commented Oct 27, 2025

PR types

Bug fixes

PR changes

Others

Describe

This pull request updates documentation and code to improve clarity, device handling, and compatibility across several examples and model architectures. The most significant changes include correcting device usage in code (removing unnecessary .cuda() and device assignments), refining training and evaluation command documentation for models, and ensuring proper data type handling in plotting routines.

Documentation improvements:

  • Swapped and clarified the "模型训练命令" and "模型评估命令" sections in the Chinese documentation for nsfnet.md and nsfnet4.md, ensuring that commands for training and evaluation are correctly labeled and use the appropriate arguments. [1] [2] [3] [4]
  • Added missing "模型评估命令" sections in example documentation files for cgcnn.md, phygeonet.md, and velocity_gan.md to improve usability and completeness. [1] [2] [3]
  • Updated dataset structure formatting in unetformer.md for clarity, changing the code block type to sh.

Device handling and compatibility:

  • Removed unnecessary .cuda() calls in phycrnet/functions.py and phycrnet.py, making code device-agnostic and compatible with both CPU and GPU execution. [1] [2] [3] [4] [5]
  • Eliminated explicit device assignment in training and evaluation routines for tgcn/run.py, and removed device transfer in unetformer/vaihingen_test.py, simplifying code and reducing potential device mismatches. [1] [2] [3]
  • Removed device string manipulation and .to(self.device) calls in the stafnet.py model, further improving device compatibility. [1] [2] [3]

Data type and plotting fixes:

  • Ensured proper conversion to Python scalars and explicit data type specification for plotting routines in VP_NSFNet4.py, preventing type errors and improving compatibility. [1] [2] [3] [4]

Miscellaneous:

  • Added a missing pass statement to the ModuleNotFoundError exception in chemprop_molecule_utils.py to avoid runtime errors.

@paddle-bot
Copy link

paddle-bot bot commented Oct 27, 2025

Thanks for your contribution!

@HydrogenSulfate HydrogenSulfate changed the title Fix bugs [Refine] Fix several bugs on custom device Oct 27, 2025
@HydrogenSulfate HydrogenSulfate marked this pull request as draft October 28, 2025 09:08
@HydrogenSulfate HydrogenSulfate marked this pull request as ready for review October 29, 2025 02:45
@HydrogenSulfate HydrogenSulfate merged commit 8db1a1c into PaddlePaddle:develop Oct 29, 2025
4 of 5 checks passed
@HydrogenSulfate HydrogenSulfate deleted the fix_bugs branch October 29, 2025 05:08
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.

1 participant