Skip to content

Add check in contour for data and xyz - #4796

Closed
willschlitzer wants to merge 1 commit into
mainfrom
error-handling-contour-data-xyz
Closed

Add check in contour for data and xyz#4796
willschlitzer wants to merge 1 commit into
mainfrom
error-handling-contour-data-xyz

Conversation

@willschlitzer

Copy link
Copy Markdown
Contributor

The documentation states that either data or x/y/z can be passed to contour, but there is no check that both are not passed. This adds a check for it, and an associated test.

Guidelines

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash command is:

  • /format: automatically format and lint the code

@willschlitzer willschlitzer added this to the 0.20.0 milestone Aug 4, 2026
@willschlitzer willschlitzer self-assigned this Aug 4, 2026
@willschlitzer willschlitzer added the enhancement Improving an existing feature label Aug 4, 2026
@seisman

seisman commented Aug 4, 2026

Copy link
Copy Markdown
Member

it's already checked in Session.virtualfile_in at

_validate_data_input(

@willschlitzer

Copy link
Copy Markdown
Contributor Author

it's already checked in Session.virtualfile_in at

_validate_data_input(

Strange; got an error the other day and thought that might have been the bug. Thanks!

@seisman
seisman deleted the error-handling-contour-data-xyz branch August 6, 2026 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants