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

BUG: Temporary fix for recursive pipeline to json #1033

Merged
merged 1 commit into from
Aug 3, 2024

Conversation

JDuffeyBQ
Copy link
Collaborator

  • Attempting to create json for a pipeline with a import dream3d filter that has stored pipeline json that also has an import dream3d filter to itself results in an infinite recursive loop until stack overflow

@JDuffeyBQ JDuffeyBQ requested a review from imikejackson August 2, 2024 17:53
@imikejackson imikejackson force-pushed the fix/recursive_pipeline_json branch from dcb48a9 to 12b206b Compare August 2, 2024 18:47
* Attempting to create json for a pipeline with a import dream3d
filter that has stored pipeline json that also has an import dream3d
filter to itself results in an infinite recursive loop until stack
overflow

Signed-off-by: Jared Duffey <[email protected]>
@imikejackson imikejackson force-pushed the fix/recursive_pipeline_json branch from 12b206b to 32c818c Compare August 2, 2024 18:47
@imikejackson imikejackson merged commit b1a142c into develop Aug 3, 2024
7 checks passed
@imikejackson imikejackson deleted the fix/recursive_pipeline_json branch August 3, 2024 16:41
imikejackson pushed a commit to imikejackson/simplnx that referenced this pull request Oct 20, 2024
…#1033)

Temporary fix for recursive pipeline to json

* Attempting to create json for a pipeline with a import dream3d
filter that has stored pipeline json that also has an import dream3d
filter to itself results in an infinite recursive loop until stack
overflow

Signed-off-by: Jared Duffey <[email protected]>
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