Skip to content

fix(core): replace bare assert with ValueError in ImageBlock/AudioBlock validation - #1490

Open
markstur wants to merge 1 commit into
generative-computing:mainfrom
markstur:issue_1445
Open

fix(core): replace bare assert with ValueError in ImageBlock/AudioBlock validation#1490
markstur wants to merge 1 commit into
generative-computing:mainfrom
markstur:issue_1445

Conversation

@markstur

Copy link
Copy Markdown
Contributor

Pull Request

Issue

Fixes #1445

Description

fix(core): replace bare assert with ValueError in ImageBlock/AudioBlock validation

Testing

  • Tests added to the respective file if code was changed
  • New code has 100% coverage if code was added
  • Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

Attribution

  • AI coding assistants used

Adding a new component, requirement, sampling strategy, or tool?

If your PR adds or modifies one of the types below, check the matching box. A checklist of type-specific review items will be posted as a comment.

  • Component
  • Requirement
  • Sampling Strategy
  • Tool

NOTE: Please ensure you have an issue that has been acknowledged by a core contributor and routed you to open a pull request against this repository. Otherwise, please open an issue before continuing with this pull request.

…ck validation

Signed-off-by: Mark Sturdevant <mark.sturdevant@ibm.com>
Assisted-by: IBM Bob
@markstur
markstur requested a review from a team as a code owner July 31, 2026 23:15
@github-actions github-actions Bot added the bug Something isn't working label Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AudioBlock/ImageBlock use bare assert for validation, silently disabled under python -O

1 participant