Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework IOutput and add tests #699

Open
wants to merge 1 commit into
base: protos
Choose a base branch
from

Conversation

oerc0122
Copy link
Collaborator

@oerc0122 oerc0122 commented Mar 4, 2025

Description of work
Rework IOutputVariable to ensure internal validity and add naive test to ensure failures are working.

Fixes
Fixes #697

To test
Standard tests.

@oerc0122 oerc0122 added styling Any issue related to styling Testing Something which adds/modifies tests. labels Mar 4, 2025
@oerc0122 oerc0122 requested a review from MBartkowiakSTFC March 4, 2025 16:01
@oerc0122 oerc0122 self-assigned this Mar 4, 2025
@oerc0122 oerc0122 force-pushed the 697-ioutput-raise branch 3 times, most recently from bb816e6 to a8c981c Compare March 5, 2025 10:52
@oerc0122 oerc0122 force-pushed the 697-ioutput-raise branch from a8c981c to 65d2004 Compare March 5, 2025 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
styling Any issue related to styling Testing Something which adds/modifies tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Invalid dims in IOutputVariable raises for wrong reason
1 participant