Skip to content

fix: fixed the indep coord linkage in EDD strainanalysis - #387

Merged
rolfverberg merged 2 commits into
mainfrom
381-fix-the-linkage-of-the-independent-dims-in-the-edd-strain-analysis-workflow
Oct 20, 2026
Merged

rolfverberg merged 2 commits into
mainfrom
381-fix-the-linkage-of-the-independent-dims-in-the-edd-strain-analysis-workflow

Conversation

@rolfverberg

Copy link
Copy Markdown
Contributor

No description provided.

@keara-soloway keara-soloway left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does wrapping the nxprocess in an nxroot work if we want to be able to write the nxprocess back to a nexus file already containing a map nxentry without overwriting the map nxentry? I want to make sure this still supports pipelines like this strain pipeline

@rolfverberg

rolfverberg commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor Author

@keara-soloway Yes, wrapping into NXroot only happens in non-standalone use. In standalone use it will not wrap it and return the NXprocess directly. I fixed this in the docstring for the_get_nxprocess() method. This should still work as it did before, but we should setup a release validation test for this (had that already on my list)

@rolfverberg
rolfverberg merged commit 464b98e into main Oct 20, 2026
3 of 5 checks passed
@rolfverberg
rolfverberg deleted the 381-fix-the-linkage-of-the-independent-dims-in-the-edd-strain-analysis-workflow branch October 21, 2026 16:00
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.

Fix the linkage of the independent dims in the EDD strain analysis workflow

2 participants