Skip to content

Conversation

@mayankansys
Copy link
Collaborator

No description provided.

Copy link

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 updates reference labels and titles in the documentation examples to use consistent naming based on file names, and comments out gallery generation and cheatsheet configurations in the Sphinx documentation build.

  • Updates reference labels in postprocessing examples to match their file names
  • Disables Sphinx gallery generation and related build settings
  • Adds a new cheatsheet extension configuration file

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
examples/00-postprocessing/post_processing_plotter_context_manager.py Updated reference label and title to match file name
examples/00-postprocessing/post_processing_plotter.py Updated reference label and title for consistency
examples/00-postprocessing/post_processing_legacy.py Updated reference label and title to reflect legacy status
examples/00-postprocessing/post_processing_callbacks_and_animation.py Updated reference label to match file name
examples/00-postprocessing/post-processing_context_manager.py Updated reference label and simplified title
doc/source/conf.py Commented out gallery generation, pyvista settings, and cheatsheet configuration
doc/source/cheatsheet/_extensions/ansys/cheat_sheet/_extension.yml Added new cheatsheet extension configuration
doc/api_rstgen.py Modified API index title generation to use "API reference" for visualization folder

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

"ignore_pattern": r"script_manifold\.py|post_processing_exhaust_manifold\.py|"
r"updated_script_manifold_example\.py|enhanced_postprocessing\.py",
"ignore_pattern": r"post_processing_callbacks_and_animation\.py|post_processing_legacy\.py|"
r"post_processing_plotter\.py|*_context_manager\.py",
Copy link

Copilot AI Nov 13, 2025

Choose a reason for hiding this comment

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

The wildcard pattern '_context_manager.py' will not match files correctly in a regex context. The asterisk '' has special meaning in regex and should be '.' for proper matching. This should be '._context_manager.py'.

Suggested change
r"post_processing_plotter\.py|*_context_manager\.py",
r"post_processing_plotter\.py|.*_context_manager\.py",

Copilot uses AI. Check for mistakes.
@prmukherj prmukherj marked this pull request as draft November 13, 2025 07:14
@@ -0,0 +1,55 @@
# Copyright (C) 2024 ANSYS, Inc. and/or its affiliates.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this file required?

pyvista.FIGURE_PATH = os.path.join(os.path.abspath("./images/"), "auto-generated/")
if not os.path.exists(pyvista.FIGURE_PATH):
os.makedirs(pyvista.FIGURE_PATH)
# pyvista.FIGURE_PATH = os.path.join(os.path.abspath("./images/"), "auto-generated/")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please avoid committing these changes. Keep them locally only.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sorry, I'll keep that in mind.

Enhanced Postprocessing
-----------------------
Post-processing context manager
Copy link
Collaborator

Choose a reason for hiding this comment

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

Context-managed post-processing workflow

Copy link
Collaborator

Choose a reason for hiding this comment

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

@mayankansys, update the description as required as well for all examples.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure.

""".. _ref_post_processing_plotter:
Enhanced Postprocessing with PyVista and Matplotlib
Post-processing with PyVista and Matplotlib plotter
Copy link
Collaborator

Choose a reason for hiding this comment

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

Advanced post-processing using PyVista and Matplotlib plotter apis

Enhanced Postprocessing
-----------------------
Post-processing with access to plotter and context manager
Copy link
Collaborator

Choose a reason for hiding this comment

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

Context managed advanced Post-Processing using PyVista and Matplotlib Plotter APIs

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.

3 participants