-
Notifications
You must be signed in to change notification settings - Fork 3
Random forest #215
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
base: main
Are you sure you want to change the base?
Random forest #215
Conversation
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.
minor comments/questions.
""" | ||
|
||
# Parameters | ||
window_size = 33 |
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.
are these indeed fixed for these network; or should they be a tunable parameter?
avg_b12 = b12.mean() | ||
|
||
# Calculate NDFI | ||
ndfi = ((b12 / avg_b12) - (b08 / avg_b08)) / (b08 / avg_b08) |
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.
is this one unavailble in the indiches package?
# Merge the two feature cubes | ||
inference_cube = s2_feature_cube.merge_cubes(s1_feature_cube) | ||
|
||
# link to the trained model: this model has an expiry date |
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.
at a standard model on S3 to be used; can check with core team the preferred location
"type": "text/html", | ||
"title": "OpenEO Web Editor", | ||
"href": "https://editor.openeo.org/?wizard=UDP&wizard~process=random_forest_firemapping&wizard~processUrl=https://raw.githubusercontent.com/ESA-APEx/apex_algorithms/8ace24cf9853989b964e65246249e1379833be9d/algorithm_catalog/vito/random_forest_firemapping/openeo_udp/random_forest_firemapping.json&server=https://openeo.vito.be" | ||
} |
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.
does it run with all job options or does it need an icnreased memory usage?
indices = compute_indices(cf_cube, indices_list) | ||
|
||
# calculate texture features | ||
features_udf = openeo.UDF.from_file("features.py") |
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.
can you give features.py a more descriptive name?
Testing of the RF pipeline to monitor its working.
Currently, the PR is at VITO.