Skip to content

mask support is added#51

Open
MarjanAsgari wants to merge 6 commits intoNRCan:mainfrom
MarjanAsgari:support_masks
Open

mask support is added#51
MarjanAsgari wants to merge 6 commits intoNRCan:mainfrom
MarjanAsgari:support_masks

Conversation

@MarjanAsgari
Copy link
Contributor

No description provided.

@MarjanAsgari
Copy link
Contributor Author

@mpelchat04 Support for checking internal masks, reading masks, normalizing data, and setting nodata values to zero has been implemented in this branch.

However, the key requirement is that GDAL must be available in the runtime environment where this project is executed. Therefore, GDAL needs to be installed either in the Docker image of the project that calls the geo_inference or as a dependency of the geo_inference package itself.

If GDAL is not available, the code will not be able to detect internal masks and will emit a runtime warning instead.

@MarjanAsgari
Copy link
Contributor Author

@mpelchat04 Hi Mathieu, based on J-F’s suggestion, I removed the GDAL dependency and used Rasterio instead to check whether each dataset has mask flags. I tested this on several images, and the results match GDAL’s behavior. As a result, GDAL installation is no longer required, and the change is ready for review.

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