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

PY: Add python bindings to allow Node based geometries to resize correctly #906

Conversation

imikejackson
Copy link
Contributor

No description provided.

@imikejackson imikejackson force-pushed the topic/python_bindings_update branch from 3b68bfe to 06f926d Compare April 2, 2024 19:13
# that you are trying to debug in the next 3 lines of code
# ------------------------------------------------------------------------------
import DataAnalysisToolkit
nx.load_python_plugin(DataAnalysisToolkit)
Copy link
Collaborator

@JDuffeyBQ JDuffeyBQ Apr 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should only be necessary if you are trying to load a pipeline with a python filter from this plugin. For just programmatically running a filter in python you could omit this though having it shouldn't harm anything.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we decided that it was needed?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's needed for loading a pipeline from json for sure.

src/nxrunner/src/nxrunner.cpp Show resolved Hide resolved
src/Plugins/SimplnxCore/wrapping/python/simplnxpy.cpp Outdated Show resolved Hide resolved
@imikejackson imikejackson force-pushed the topic/python_bindings_update branch from 06f926d to c13d10f Compare April 3, 2024 03:50
@imikejackson imikejackson requested a review from JDuffeyBQ April 3, 2024 03:53
@imikejackson imikejackson force-pushed the topic/python_bindings_update branch from 44e52be to b6ff844 Compare April 3, 2024 12:53
@imikejackson imikejackson force-pushed the topic/python_bindings_update branch from b6ff844 to eae6d2b Compare April 3, 2024 14:47
@imikejackson imikejackson merged commit 8b1f237 into BlueQuartzSoftware:develop Apr 3, 2024
8 checks passed
@imikejackson imikejackson deleted the topic/python_bindings_update branch April 3, 2024 15:55
imikejackson added a commit to imikejackson/simplnx that referenced this pull request Apr 4, 2024
imikejackson added a commit to imikejackson/simplnx that referenced this pull request Oct 20, 2024
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