CI TEST #6
Merged
CI TEST #6
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This makes it easier to use pdb within operators. Otherwise, it will always complain when you use any spaces. This happens because VTK has its own stdin/stdout that it uses with Python. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
We can load all of these in as separate arrays. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This gives some nice template stuff. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This allows one to put a conditional enable/visibility connection on widgets defined in JSON files. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This is probably harmless most of the time, unless the data is massive. But it's annoying when it appears for every single array in a dataset. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
pyfftw requires these arrays to be contiguous. This fixes the issue. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
I had a segmentation fault one time where `unitsModified()` was called. The only possible cause I can see is if the `newChildDataSource` was deleted before it was called. One way to fix it is to add `newChildDataSource` as a context object, so that the offending signal will be disconnected if `newChildDataSource` is destroyed. That is what is done here. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This allows us to specify in the operator definition if we want to apply the transformation to each arrays, one at a time. It currently only works with internal datasets. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This just helps us ensure we don't seg fault... Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This fixes an issue where old values for combo boxes and paths were not remembered. The default system is currently being used for setting these, so we just needed to make sure the default system worked properly. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This adds a multi-array aware image alignment operator that uses PyStackReg. By default, the operator uses a reference of the currently selected slice index, if a slice module is present. This has the capability to save/load transformation matrices. Translation transformation matrices can also be applied to images of different resolution. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
The `data` should definitely be present, but the others might not be. This fixes an error if the others are missing. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
C++20 was causing issues with pybind11 (in its use of tbb). We don't need the C++20 stuff so just switch back to C++17. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
`np.lib.pad` no longer exists and we need to use `np.pad` instead. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This avoids a name conflict with "operator.py" in the Python "collections" library. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This is so that we can be notified as to why it failed Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This actually won't be required once we update tiled/data broker. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
We were encountering a seg fault earlier for accessing a child of one that had been destroyed. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This defaults to `true`, and it is done so that we can easily use the XRF datasets in Tomviz's reconstruction algorithms.. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
np.int no longer exists Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
822a23c to
6b12b47
Compare
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
6b12b47 to
efebc8e
Compare
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
efebc8e to
a107df4
Compare
This is done so that old, invalid paths are not loaded automatically from the settings when the user has a completely new dataset. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
a107df4 to
ff5b10b
Compare
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
85f983a to
eed87a7
Compare
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
eed87a7 to
3d86a3f
Compare
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This also uses a random number generator seed in order to ensure it is deterministic when reran. This also takes into account the differences in pixel sizes when applying the shift from one array to another. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
3d86a3f to
41dffd5
Compare
This even takes into account the padding. It seems to be working very well for all cases. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This seems to be working fine with the Pt_Zn_Phase example we have. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Otherwise, it fails to run because it uses the wrong PYTHONPATH Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This sets them on the spacing in nanometers. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
41dffd5 to
2a92cc1
Compare
These were requested by Aaron. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
The tomviz List class has some crashy issues. We'd better avoid it and just use a Variant instead. This also applies a few more updates needed for the Ptycho workflow. Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
psavery
force-pushed
the
ptycho-workflow
branch
from
c2abf6d to
6688b40
Compare
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.