Skip to content

Conversation

@nducros
Copy link
Member

@nducros nducros commented Nov 28, 2025

No description provided.

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 modifies the display behavior of image visualization functions by changing the default value of the show parameter from True to False for three display functions. This allows users to create plots without immediately displaying them, providing more flexibility for programmatic image processing and plotting workflows.

Key changes:

  • Changed show parameter default from True to False in imagesc(), imagecomp(), and imagepanel() functions
  • Added conditional plt.show() calls in imagecomp() and imagepanel() functions (already present in imagesc())

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

title2="",
colormap1=plt.cm.gray,
colormap2=plt.cm.gray,
show=False,
Copy link

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

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

Missing comma after the show=False parameter. This will cause a syntax error. Add a comma after show=False on line 230.

Copilot uses AI. Check for mistakes.
colormap3=plt.cm.gray,
colormap4=plt.cm.gray,
show=False,
):
Copy link

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

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

Missing comma after the show=False parameter. This will cause a syntax error. The function signature should be:

show=False,

Copilot uses AI. Check for mistakes.
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.

2 participants