-
Notifications
You must be signed in to change notification settings - Fork 11
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
Finished Sofast HDF5 IO #31
Merged
+468
−400
Merged
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
82e1e1d
Removed unused output hdf file
braden6521 0cbd455
Migrated surface_params to directly use Surface2Dxxx. Saving not impl…
braden6521 fa166af
Added HDF5 IO abstract classes to hdf5_tools
braden6521 2f7de79
Added save/load methods to Surface2D... classes
braden6521 a12b184
Single facet stable
braden6521 1ffe71d
Aded missing documentation to process_multifacet_geometry
braden6521 ace0620
Migrated SofastFringe to use logging and Surface2Dxx classes only.
braden6521 5b3745c
Reorganized imports in SofastFringe examples
braden6521 d71af33
Changed names of sofastFringe examples
braden6521 38ca87f
Moved view camera distortion ex to correct folder
braden6521 6ed65ba
Updated SofastFringe integration tests
braden6521 57a239f
Added matplotlib import
braden6521 697f3fc
Updated test_SlopeSolver to use Surface2D classes
braden6521 027e66e
Added docstring
braden6521 f050465
Updated documentation in process_optics_geometry
braden6521 19afc0c
Renamed ParamsSlopeSolver to ParamsSlopeSolverAbstract
braden6521 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
File renamed without changes.
This file contains 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 file contains 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 file contains 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
Binary file removed
BIN
-346 KB
.../calibration/test/data/output/CalibrationScreenPosition/screen_distortion_data_100_100.h5
Binary file not shown.
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's typically better to do this step outside a function, such as in the
if __name__ == "__main__":
control, because lt.logger() should only be called once per process.That being said, I think it's ok to leave it here, since the example_driver() function essentially is the main process.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, I'll make sure to do that in the future.
I'll leave it here for the time being for the reason you mentioned. Also the save_dir needs to be set up before the log, so I'm fine with leaving it in this file.