Skip to content

Conversation

@mkundu1
Copy link
Contributor

@mkundu1 mkundu1 commented Dec 4, 2025

To avoid a later git error in self-hosted runner which is due to some folder generated within the pyfluent directory while running the external_compressible_flow.py example script:

  1. Make sure FM_* folder is generated in a temp directory.
  2. Delete the generated wing_workflow_files at the end of the script.

I think external_compressible_flow.py is the only problematic one based on errors found in github logs (see the issue details).

Copilot AI review requested due to automatic review settings December 4, 2025 02:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses a git error in self-hosted runners caused by folders generated within the pyfluent directory during the execution of the external_compressible_flow.py example script. The changes ensure that temporary folders are created in a proper temporary directory and cleaned up after execution.

Key changes:

  • Configure meshing workflow to use a system temp directory instead of generating folders within the pyfluent directory
  • Clean up both the temporary directory and wing_workflow_files directory at the end of script execution

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

@github-actions github-actions bot added documentation Documentation related (improving, adding, etc) examples Publishing PyFluent examples CI/CD Related to CI/CD labels Dec 4, 2025
Copilot AI review requested due to automatic review settings December 8, 2025 12:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


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

@mkundu1 mkundu1 merged commit 146355d into main Dec 8, 2025
61 of 62 checks passed
@mkundu1 mkundu1 deleted the ci/temp-folders branch December 8, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related to CI/CD documentation Documentation related (improving, adding, etc) examples Publishing PyFluent examples

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Gallery example scripts should not write files within pyfluent directory

5 participants