Skip to content

GEOPY-2745: Refresh screenshots of UIJsons in docs - #66

Merged
domfournier merged 12 commits into
release/GA_4.8from
GEOPY-2745
May 29, 2026
Merged

GEOPY-2745: Refresh screenshots of UIJsons in docs#66
domfournier merged 12 commits into
release/GA_4.8from
GEOPY-2745

Conversation

@benk-mira

@benk-mirabenk-mira commented May 26, 2026

Copy link
Copy Markdown
Contributor

GEOPY-2745 - Refresh screenshots of UIJsons in docs

CopilotAI review requested due to automatic review settings May 26, 2026 17:57
@github-actionsgithub-actionsBot changed the title GEOPY-2745GEOPY-2745: Refresh screenshots of UIJsons in docsMay 26, 2026

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Sphinx/reStructuredText documentation pages related to block model workflows, adjusting how an (octree grid) figure is represented so the docs build/render cleanly.

Changes:

  • Replaced line-block “commented” figure markup with reST comment-style markup in block_model.rst.
  • Replaced line-block “commented” figure markup with reST comment-style markup in block_model_to_octree.rst.

Reviewed changes

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

FileDescription
docs/source/block_model.rstAdjusts the (currently commented) octree_grid.png figure markup to avoid rendering it as literal text.
docs/source/block_model_to_octree.rstSame figure-markup adjustment for the Block Model → Octree documentation page.

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

Comment threaddocs/source/block_model.rst Outdated
Comment threaddocs/source/block_model_to_octree.rst Outdated

@domfournierdomfournier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great, thanks for the first round.
Would you mind keeping the common layout for the docs of Block model to octree and Block model creation? Should be something along the lines of:

Intro (with nice figure after first paragraph)
Interface (show UI and bullet the inputs)
Methodology
Example

Comment threaddocs/source/block_model.rst Outdated
Comment threaddocs/source/block_model.rst Outdated
Comment threaddocs/source/block_model_to_octree.rst Outdated
Comment threaddocs/source/block_model_to_octree.rst Outdated
Comment threaddocs/source/block_model_to_octree.rst
Comment threaddocs/source/block_model.rst Outdated
Comment threaddocs/source/block_model.rst Outdated
Comment threaddocs/source/block_model_to_octree.rst Outdated
Comment threaddocs/source/block_model_to_octree.rst Outdated
benk-miraand others added 5 commits May 29, 2026 08:18
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
@domfournier
domfournier merged commit f3fb858 into release/GA_4.8May 29, 2026
17 checks passed
@domfournier
domfournier deleted the GEOPY-2745 branch May 29, 2026 16:06
Sign up for freeto 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

@benk-mira@domfournier