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

STYLE: Standardize I/O Naming to Read/Write #753

Merged

Conversation

nyoungbq
Copy link
Contributor

Renamed all I/O filters to read/write as instructed. The documentation, test cases, filters, and algorithms were all renamed in the ComplexCore and OrientationAnalysis plugins. The ITK plugin was untouched aside from the human names being changed to make them propagate with similar filters. CMake file declarations reordered to conform to A-Z sorting, and small stylistic bug fixes I came across while working on this branch.

@nyoungbq nyoungbq requested a review from imikejackson October 24, 2023 16:48
@nyoungbq nyoungbq force-pushed the enh/standardize_io_naming branch 2 times, most recently from 67c379a to 9cfe9b2 Compare October 25, 2023 16:17
@imikejackson imikejackson force-pushed the enh/standardize_io_naming branch from 9cfe9b2 to 5e5cb65 Compare October 26, 2023 13:39
@nyoungbq nyoungbq force-pushed the enh/standardize_io_naming branch from 5e5cb65 to 1779c7c Compare October 30, 2023 15:23
@imikejackson imikejackson force-pushed the enh/standardize_io_naming branch from 1779c7c to 313c6ab Compare November 2, 2023 16:37
@imikejackson
Copy link
Contributor

I/O Renaming

ComplexCore

Old File Name New File Name Type
AbaqusHexahedronWriterFilter WriteAbaqusHexahedronFilter Filter
AbaqusHexahedronWriter WriteAbaqusHexahedron Algorithm
AvizoRectilinearCoordinateWriterFilter WriteAvizoRectilinearCoordinateFilter Filter
AvizoRectilinearCoordinateWriter WriteAvizoRectilinearCoordinate Algorithm
AvizoUniformCoordinateWriterFilter WriteAvizoUniformCoordinateFilter Filter
AvizoUniformCoordinateWriter WriteAvizoUniformCoordinate Algorithm
ExportDREAM3DFilter WriteDREAM3DFilter Filter
FeatureDataCSVWriterFilter WriteFeatureDataCSVFilter Filter
ImportDeformKeyFileV12Filter ReadDeformKeyFileV12Filter Filter
ImportDeformKeyFileV12 ReadDeformKeyFileV12 Algorithm
ImportVolumeGraphicsFileFilter ReadVolumeGraphicsFileFilter Filter
ImportVolumeGraphicsFile ReadVolumeGraphicsFile Algorithm
ImportBinaryCTNorthstarFilter ReadBinaryCTNorthstarFilter Filter
ImportBinaryCTNorthstar ReadBinaryCTNorthstar Algorithm
LosAlamosFFTWriterFilter WriteLosAlamosFFTFilter Filter
LosAlamosFFTWriter WriteLosAlamosFFT Algorithm
RawBinaryReaderFilter ReadRawBinaryFilter Filter
RawBinaryReader ReadRawBinary Algorithm
StlFileReaderFilter ReadStlFileFilter Filter
StlFileReader ReadStlFile Algorithm
VtkRectilinearGridWriterFilter WriteVtkRectilinearGridFilter Filter
VtkRectilinearGridWriter WriteVtkRectilinearGrid Algorithm

OrientationAnalysis

Old File Name New File Name Type
EnsembleInfoReaderFilter ReadEnsembleInfoFilter Filter
EnsembleInfoReader ReadEnsembleInfo Algorithm
ExportGBCDGMTFileFilter WriteGBCDGMTFileFilter Filter
ExportGBCDGMTFile WriteGBCDGMTFile Algorithm
ExportGBCDTriangleDataFilter WriteGBCDTriangleDataFilter Filter
ExportGBCDTriangleData WriteGBCDTriangleData Algorithm
ImportH5Data ReadH5Data Utility
ImportH5EspritDataFilter ReadH5EspritDataFilter Filter
ImportH5EspritData ReadH5EspritData Algorithm
ImportH5OimDataFilter ReadH5OimDataFilter Filter
ImportH5OimData ReadH5OimData Algorithm
INLWriterFilter WriteINLFileFilter Filter
INLWriter WriteINLFile Algorithm

@nyoungbq nyoungbq force-pushed the enh/standardize_io_naming branch from 313c6ab to cebb965 Compare November 9, 2023 21:58
@imikejackson imikejackson force-pushed the enh/standardize_io_naming branch from cebb965 to 330ede1 Compare November 10, 2023 20:18
nyoungbq and others added 19 commits November 13, 2023 15:13
This is due to the renaming of the filters which causes API changes.

Signed-off-by: Michael Jackson <[email protected]>
@imikejackson imikejackson force-pushed the enh/standardize_io_naming branch from c658b81 to 7ee5e7b Compare November 13, 2023 20:14
@imikejackson imikejackson enabled auto-merge (squash) November 13, 2023 20:17
@imikejackson imikejackson merged commit 8b7b55d into BlueQuartzSoftware:develop Nov 13, 2023
7 checks passed
@imikejackson imikejackson deleted the enh/standardize_io_naming branch November 14, 2023 02:10
imikejackson added a commit to imikejackson/simplnx that referenced this pull request Oct 20, 2024
* AbaqusHexahedronWriter -> WriteAbaqusHexahedron
* AvizoRectilinearCoordinateWriter -> WriteAvizoRectilinearCoordinate
* AvizoUniformCoordinateWriter -> WriteAvizoUniformCoordinate
* ComplexCore Algorithm renaming
* Complete complexcore io rename update
* Rename OrientationAnalysis i/o
* Sort cmake lists alphabetical
* Update ITK I/O human names
* Update version to 1.2.0

* Fixing various python related codes
* FindFeaturePhases: Change k_FeaturePhasesArrayName_Key value
* Python: Adjust release notes structure to separate versions.
* Rename the ImportHDFDatasetParameter to ReadHDF5DataSetParameter
* Rename the H5EbsdReaderParameter to ReadH5EbsdFileParameter
* Upate python codes to WriteDREAM3DFilter
* Fix runtime exception when parent file path is empty.

Signed-off-by: Michael Jackson <[email protected]>

---------

Signed-off-by: Michael Jackson <[email protected]>
Co-authored-by: Michael Jackson <[email protected]>
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