feat: validate data paths in SatelliteDataModule initialization #92
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.
Validate data paths in SatelliteDataModule initialization
This PR addresses issue #18 by adding validation of data paths during the initialization of the
SatelliteDataModuleclass.Changes
_validate_data_paths()method to check if all provided data paths exist__init__after setting thezarr_pathattributeTesting
The changes ensure that invalid data paths are caught early during initialization rather than when the dataloader methods are called. This provides better error messages and prevents potential issues from surfacing later in the training process.
Impact
This change improves the robustness of the code by failing fast when data paths are invalid, making it easier to debug configuration issues.