Skip to content

Conversation

@HydrogenSulfate
Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate commented Nov 7, 2025

PR types

Bug fixes

PR changes

Others

Describe

This pull request introduces several improvements and fixes across documentation, dependency management, and code compatibility for the Transolver example and the CViT architecture. The most important changes include enhanced documentation for setup and data preprocessing, added missing dependencies, improved error handling for optional imports, and a correction in the CViT architecture's MLP block initialization.

Documentation improvements:

  • Updated docs/zh/examples/transolver.md to include installation instructions for required dependencies and moved the note about initial data preprocessing to relevant sections for better clarity. [1] [2] [3] [4]

Dependency management:

  • Added paddle_geometric and vtk to examples/transolver/requirements.txt to ensure all required packages are installed before running the example.

Code compatibility and robustness:

  • Improved error handling in examples/transolver/shapenet_car.py by using try/except blocks for optional imports from paddle_geometric, preventing crashes if dependencies are missing.
  • Updated tensor type references and data conversion in examples/transolver/shapenet_car.py to use PaddlePaddle types directly, improving compatibility and simplifying data handling. [1] [2]

Architecture correction:

  • Fixed the initialization of the MLP block in ppsci/arch/cvit.py by using self.coords_dim instead of self.latent_dim for the input dimension, ensuring correct model behavior for the "mlp" embedding type. [1] [2]

@paddle-bot
Copy link

paddle-bot bot commented Nov 7, 2025

Thanks for your contribution!

@HydrogenSulfate HydrogenSulfate merged commit 3961d68 into PaddlePaddle:develop Nov 10, 2025
4 of 5 checks passed
@HydrogenSulfate HydrogenSulfate deleted the fix_cvit branch November 10, 2025 04:10
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