Skip to content

Conversation

wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Aug 7, 2025

Briefly, what does this PR introduce?

This PR propagates the workaround in bf2857d (for issue #1961) to the two other factories that use a similar construct. Before #1959 was merged, this was done by our JOmniFactory.

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Documentation update
  • Other: __

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

No.

Does this PR change default behavior?

No.

@wdconinc wdconinc enabled auto-merge August 7, 2025 21:29
@wdconinc
Copy link
Contributor Author

wdconinc commented Aug 7, 2025

No diffs.

@wdconinc wdconinc requested a review from veprbl August 7, 2025 21:30
@wdconinc wdconinc added this pull request to the merge queue Aug 8, 2025
Merged via the queue into main with commit d21c332 Aug 8, 2025
120 checks passed
@wdconinc wdconinc deleted the clear-output-vectors branch August 8, 2025 00:49
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