Skip to content

ENH: Dream3dImportParameter supports additional import options.#1230

Merged
imikejackson merged 17 commits intoBlueQuartzSoftware:developfrom
joeykleingers:topic/Dream3dImportParameterEnh
Mar 26, 2025
Merged

ENH: Dream3dImportParameter supports additional import options.#1230
imikejackson merged 17 commits intoBlueQuartzSoftware:developfrom
joeykleingers:topic/Dream3dImportParameterEnh

Conversation

@joeykleingers
Copy link
Contributor

  • Added a new member variable enumeration to the ImportData struct called PathImportPolicy, which has options "IncludeList", "ExcludeList", and "All". If IncludeList is chosen, the data paths are interpreted as paths to objects to import. If ExcludeList is chosen, the data paths are interpreted as paths to objects to NOT import. If All is chosen, the data paths are ignored and everything is imported.

  • Included a new constructor that is required to be used when creating a new ImportData instance.

  • Updated the ReadDREAM3DFileFilter to be able to take in paths to singular objects without also requiring all the parent paths too. For example, it is now possible to set the DataPaths to simply "ImageGeom/CellData/Confidence Index" without having to also include "ImageGeom" and "ImageGeom/CellData". This will be useful for anyone trying to execute this filter without the GUI, especially on the Python side.

@joeykleingers joeykleingers added the enhancement New feature or request label Mar 6, 2025
@joeykleingers joeykleingers requested a review from JDuffeyBQ March 6, 2025 22:13
@joeykleingers joeykleingers force-pushed the topic/Dream3dImportParameterEnh branch from 828b4e4 to 093921a Compare March 6, 2025 22:20
@imikejackson imikejackson force-pushed the topic/Dream3dImportParameterEnh branch from 093921a to 07d434d Compare March 7, 2025 13:36
@joeykleingers joeykleingers force-pushed the topic/Dream3dImportParameterEnh branch 3 times, most recently from ce5be47 to 3c0fbbd Compare March 9, 2025 01:47
@imikejackson imikejackson linked an issue Mar 11, 2025 that may be closed by this pull request
@imikejackson imikejackson force-pushed the topic/Dream3dImportParameterEnh branch 3 times, most recently from 0d2021a to bf0353d Compare March 12, 2025 15:11
@joeykleingers joeykleingers force-pushed the topic/Dream3dImportParameterEnh branch 3 times, most recently from c6f0a1f to 274f25b Compare March 18, 2025 13:42
@imikejackson imikejackson force-pushed the topic/Dream3dImportParameterEnh branch 2 times, most recently from 60d6a33 to b6d377b Compare March 19, 2025 12:22
@joeykleingers joeykleingers force-pushed the topic/Dream3dImportParameterEnh branch from d832477 to 6f47ce8 Compare March 19, 2025 14:44
@joeykleingers joeykleingers requested a review from JDuffeyBQ March 19, 2025 14:44
@joeykleingers joeykleingers force-pushed the topic/Dream3dImportParameterEnh branch 2 times, most recently from d4ec7af to 2d42181 Compare March 20, 2025 16:59
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
…aths.

This change allows the filter to handle "Geometry/CellData/Fit" by itself
instead of having to also input "Geometry" and "Geometry/CellData".

This change will mainly benefit anyone running this filter without the GUI.

Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
joeykleingers and others added 9 commits March 23, 2025 22:39
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
Co-authored-by: Jared Duffey <jared.duffey@bluequartz.net>
Co-authored-by: Jared Duffey <jared.duffey@bluequartz.net>
Co-authored-by: Jared Duffey <jared.duffey@bluequartz.net>
Co-authored-by: Jared Duffey <jared.duffey@bluequartz.net>
Co-authored-by: Jared Duffey <jared.duffey@bluequartz.net>
Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
@joeykleingers joeykleingers force-pushed the topic/Dream3dImportParameterEnh branch from 2d42181 to 59f5e57 Compare March 24, 2025 02:39
@imikejackson imikejackson merged commit 90d09a4 into BlueQuartzSoftware:develop Mar 26, 2025
8 of 9 checks passed
@imikejackson imikejackson deleted the topic/Dream3dImportParameterEnh branch March 26, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding more options to Dream3dImportParameter::ImportData

3 participants