-
Notifications
You must be signed in to change notification settings - Fork 29
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
Stack overflow exception from VisualizerMapping with second VisualizerMapping as input #1769
Comments
Looked into this a bit more, the issue comes from entering an infinite recursive call in I can't think of any situation where you would want to combine visualizers as in the workflow example above as the information is redundant. In that case the fix may be just to disallow visualizer combinations where the data source is the same for visualizers in the combination. |
…are combined together Fixes bonsai-rx#1769
…are combined together Fixes bonsai-rx#1769
…are combined together Fixes bonsai-rx#1769
While investigating #1591 I noticed that creating this workflow is possible in the editor:
When run this results in a terminating crash from a stack overflow exception, presumably due to a cyclical dependency.
The text was updated successfully, but these errors were encountered: