Fix info message + report Pekel threshold modification used in post-processing#59
Merged
ytanguy merged 2 commits intoremove_eoscalefrom Mar 12, 2026
Merged
Fix info message + report Pekel threshold modification used in post-processing#59ytanguy merged 2 commits intoremove_eoscalefrom
ytanguy merged 2 commits intoremove_eoscalefrom
Conversation
Closed
ytanguy
added a commit
that referenced
this pull request
Apr 21, 2026
* Fix bug - new mask_valid_indices array to enumerate valid segments (WIP : still some debug messages to remove) * Fix bug - new mask_valid_indices array to enumerate valid segments (WIP : still some debug messages to remove) * New Cython function to produce valid indices mask, apply in the clustering steps * NDVI/NDWI are limited to -1/1 interval : correct rare issues when some pixels have a negative value * Fix : NDVI/NDWI values are -1000/1000 and not -1 / 1 * Fix NDVI/NDWI, and take into account NODATA to compute number of segment to compute * remove eoscale on first stage valid stack * add eomultiprocessing component * rm eoscale from ndvi & ndwi functions * rm eoscale from texture functions & adapt docstrings * rm out directories from slurp manager & spread profiles along workflow * add logger info step * split eoscale utils & profile utils * fix texture profile forom int16 to uint16 * fix key cloud mask use case * add concatenate samples in utils & rm from eoscales/profiles utils * add mp_n_to_m_scalars function * watermask to slurp exec & manager mp framework * update mask variable names * fix concatenate samples function & concat samples * add rgba bands to rf pred * fix pekel filter in post_process * Add NDWI threshold use cases * Add handling of Aux files * Add some docstrings * Add some docstrings * Fix type mismatch : remove [ ], otherwise, a call to prepare with existing valid stack but new NDxI files fail... (#55) * First Urbanmask version & Slurp executor aux files handling * Fix info message + report Pekel threshold modification used in post-processing (#59) * Fix info message + report Pekel threshold modification used in post-processing * Fix documentation #57 * update logs urbanmask * integ modifs in watermask, logs & aux inputs handling * update logs in urbanmask * write tif in all urban use cases & diff valid stack & OG if no water/veg Masks * reduce McCabe Complexity through orchestration pattern * phr to vhr * swap vhr phr watermask * first version shadowmask & stack masks with eomultiprocessing component * remove eoscale imports * add docstrings * phr to vhr * adding support to n bands images in slurp mp executor * handle random input order in n bands images (h, w, c), (c, w, h),... (#67) * handle tile_max_size args * Revert accidental change in veg mask * handle tile_max_size in remaining masks * Remove eoscale vegmask (#69) * first vegetation mask version * veg mask handle tile_max size * debug segmentation processing in vegetationmask with dev of func mp_n_to_m_images_with_mapping * fix stats in vegetationmask, first correct version * adapt some docstrings in veg mask * refactor mp_n_to_m_scalars * before_clean.tif optionnal if debug * report #44 in veg mask * removed all eoscale references * --version option added * fix --version argparse * first test version for mnh with urbanmask * add test with mnh * logging in veg mask & improve code style w/ pylint * Update file paths in config_test_mnh.json * Update file paths in main_MNH_config.json * Add tqdm to package dependencies * replace display infos function * format w/ Black compliance --------- Co-authored-by: Yannick TANGUY <yannick.tanguy@cnes.fr> Co-authored-by: Yannick TANGUY <45934446+ytanguy@users.noreply.github.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Fix #40 and also fix info message.