Hi Thijs,

can you be more specific of what kind of changes you did in the example? Could it be that you’re sampling from a single EOPatch, which is only covered in majority with 3 different classes (some class(es) represent only very small fraction of patch area)?

Otherwise PointSampligTaks does have an option to sample the same number of pixels for each class. The parameter is called even_sampling is described here. But beware, that if for example only 1000 pixels belong to class A and if you request minimum 20000 pixels to be sampled for each class then all pixels from class A will be sampled in average 20 times. This will limit model’s ability to generalize…

Hey Anze,

Yes you are correct, I am sampling form one single EO Patch, which is covered in majority with 3 different classes as you said.

Im not sure how to use the even_sampling task in this case, could you give an example? Should I use PointRasterSampler instead of PointSamplingTask or is there a way I can give even_sampling as input to PointSamplingTask?

Thanks for your help.

I believe you can simply set even_sampling=True in the constructor of PointSamplingTask. It will be passed to PointRasterSampler.

I have the feeling that I am missing something. So first I define the PointSamplingTask as:

spatial_sampling = PointSamplingTask(

    n_samples=n_samples, 

    ref_mask_feature='LULC_ERODED', 

    ref_labels=ref_labels, 

    sample_features=[  # tag fields to sample

        (FeatureType.DATA, 'FEATURES'),

        (FeatureType.MASK_TIMELESS, 'LULC_ERODED')

    ], even_sampling=True) 

How can I then call the PointRasterSampler? With:

PointRasterSampler( spatial_sampling, even_sampling = True) ?

Perhaps I’m misunderstanding your question. If that’s the case, pleaser let me know.

One of the arguments of PointSamplingTask are **sampling_params, which can according to the documentation be: Any other parameter used by PointRasterSampler class. The PointSamplingTask creates in its execute method an instance of PointRasterSampler with **sampling_params passed to it.

Dear Anze,

Thanks for your help, it did indeed sample my pixels evenly in the available classes, but it brought me to a different issue answered in this post: Performing ErosionTask on MASK_TIMELESS.LULC with very few valid pixels

Since I have very few pixels for certain classes, they got eroded in the erosion step. Even though the erosion step by default is set to 1 pixel.

But now I have a different issue, because I tried to only erode the major classes in my patch, but that resulted in a map of only those 3 major classes. In the reply to William you mentioned giving different strengths for Erosion to different classes, how to do this? This is not clear to me through the documentation.

Thanks again for the ongoing help,

Thijs

Hi Thijs,

Let’s assume that you wish to erode a timeless mask feature with name “TRUTH”, which has 4 classes labeled 1,2,3,4 and 0 is the no-data value. Classes 1 and 2 can be eroded with disk with radius 3, class 3 with disk with radius 1 and keep class 4 as is, for example, as:

erode_12 = ErosionTask((FeatureType.MASK_TIMELESS, 'TRUTH'),  disk_radius=3, erode_labels=a1,2])

erode_3  = ErosionTask((FeatureType.MASK_TIMELESS, 'TRUTH'),  disk_radius=1, erode_labels=a3])

In summary, for each morphological operation (small/large erosion) you need to create a task, which however acts only on subset of classes/labels specified via erode_labels argument.

Best regards,

Anze

Hi Anze,

This procedure is clear to me. However it overwrites my existing mask made in erode_12 made in step one. Lets take the example classes from the tutorial:

erode_1 = ErosionTask((FeatureType.MASK_TIMELESS, ‘LULC’, ‘LULC_ERODED’), disk_radius=3, erode_labels=_0,1,2,9])

erode_2 = ErosionTask((FeatureType.MASK_TIMELESS, ‘LULC’, ‘LULC_ERODED’), disk_radius=1, erode_labels=15,8])

With erode_1 I erode my majority covered classes, and with erode_2 my smaller coverage classes. Unfortunately, I end up with a LULC_ERODED mask with only the results from erode_2.

I have tried only working in the ‘LULC’ timeless mask feature as per your example where you only have the ‘TRUTH’ timeless mask feature but this just overrode my LULC class with the same result.

Why did I loose my erode_labels=I0,1,2,9] in the result ?

Regards,

Thijs

Hi Thijs,

OK, now I understand the problem. It looks like the ErosionTask keeps only labels that are being eroded. The rest are set to no_data_value. IMO, this is a bug and not a feature.

For the time being you can use this custom task (until the ErosionTask is fixed):

class ClassFilterTask(EOTask):   

    """

    Run class specific morphological operation.

    """

    def __init__(self, lulc_feature, lulc_values, morph_operation, struct_elem=None):

        self.lulc_feature_type, self.lulc_feature_name = next(iter(self._parse_features(lulc_feature)))

        self.lulc_values=lulc_values

        

        if isinstance(morph_operation, MorphologicalOperations):

            self.morph_operation = MorphologicalOperations.get_operation(morph_operation)

        else:

            self.morph_operation = morph_operation

        self.struct_elem = struct_elem

    

    def execute(self, eopatch):

        lulc = eopatch[self.lulc_feature_type][self.lulc_feature_name].copy()

        

        for lulc_value in self.lulc_values:

            lulc_mod = self.morph_operation((lulc==lulc_value).squeeze(), self.struct_elem) * lulc_value

            lulc_mod = lulc_mod[...,np.newaxis]

            lulc[lulc==lulc_value]=lulc_mod[lulc==lulc_value]

        

        eopatch.add_feature(self.lulc_feature_type, self.lulc_feature_name, lulc)



        return eopatch

which can be then used as

from eolearn.ml_tools import MorphologicalFilterTask, MorphologicalOperations, MorphologicalStructFactory



erode_1 = ClassFilterTask((FeatureType.MASK_TIMELESS, 'LULC'),[1,2,9],

                          MorphologicalOperations.EROSION,struct_elem=MorphologicalStructFactory.get_disk(3))

erode_2 = ClassFilterTask((FeatureType.MASK_TIMELESS, 'LULC'),[5,8],

                          MorphologicalOperations.EROSION,struct_elem=MorphologicalStructFactory.get_disk(1))

Please note that you shouldn’t include no_data_value (0) to labels to be eroded.

I hope this will work for you.

Dear Anze,

This perfectly described a fix for the issue that I had, thanks for the outstanding support!

Regards,

Thijs