Skip to content

Conversation

@awccoppFC
Copy link
Contributor

@awccoppFC awccoppFC commented Feb 11, 2026

Removed validation check for write single file output surface output preventing paraview format


Note

Low Risk
Removes a validation guardrail and expands supported configuration; risk is limited to potential downstream solver/runtime incompatibility if Paraview single-file writing isn’t actually supported.

Overview
SurfaceOutput no longer raises/ warns when write_single_file=True is used with output_format="paraview" or "both", effectively enabling single-file surface outputs for Paraview.

Tests are updated to reflect the new behavior by removing the expected validation error and asserting these configurations now validate successfully.

Written by Cursor Bugbot for commit 7db3c0b. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@awccoppFC awccoppFC enabled auto-merge (squash) February 11, 2026 17:08
@awccoppFC awccoppFC merged commit c26c207 into main Feb 11, 2026
21 checks passed
@awccoppFC awccoppFC deleted the alexander/ParaviewSingleFile branch February 11, 2026 17:16
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