-
-
Notifications
You must be signed in to change notification settings - Fork 384
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
Add actinia notebook #573
Add actinia notebook #573
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.
Would it be possible to add extent=region
to r.import
? Then only the needed parts are imported (Not tested yet)
I've been using |
@neteler Thank you for working on this. Is this PR ready to reviewed and merged? Or are you still working on it? |
The current blockers are actinia-org/actinia-core#482 and actinia-org/actinia-core#491 which we are working on. This PR should be fine but cannot be used yet. That said, we also face a problem when testing this PR against a locally patched actinia server:
The reason for this keyerror seems to be tests used in leafmap which use and fail because https://titiler.xyz can not work with a local server (seems to be used around here). But that's certainly somewhat unrelated. We are working on the completion of the |
Btw: we can increment the number of this notebook if you need 83 for another one! |
No worries. I don't have new notebooks to add at the moment |
The cloud based geoprocessing platform actinia is able to ingest and analyse large volumes of geodata. This PR proposes a notebook showcasing the processing of a hillshade map from an external data source (SRTM map). We use the https://actinia-org.github.io/actinia-python-client/.
865d4d7
to
bb2520e
Compare
Nonetheless the definition of Help welcome!
|
The notebook error is a werid one. I can't reproduce it locally. You can comment it out temporarily. For the colorbar, I would suggest using the m = leafmap.Map()
m.add_basemap("OpenTopoMap")
m.add_colormap(
'terrain',
label="Elevation",
width=4,
height=0.3,
orientation='horizontal',
vmin=0,
vmax=4000,
)
m |
I just committed changes to ignore the notebook that causes the werid error. It should pass the tests now. |
Thanks for the doc fix/workaround. TODO to self: sort out remaining actinia glitch. |
This has now been addressed in From my point of view ready to merge. Thanks for your patience. |
It looks great! I made some minor changes, including improving the installation method because wget does not work on Windows, and clean up the notebook. Thank you very much for your time and efforts on this! Excited to see this happen. |
Right, good point and thanks for the fix.
Thank you for inviting us to contribute this notebook and for all your support. Also thanks to @mmacata, @anikaweinmann and @linakrisztian for the needed updates in actinia-core and the actinia-python-client! |
The notebook is available on the website now: https://leafmap.org/notebooks/87_actinia |
The cloud based geoprocessing platform actinia is able to ingest and analyse large volumes of geodata. This PR proposes a notebook showcasing the processing of a hillshade map from an external data source (SRTM map).
To interface with actinia, the https://actinia-org.github.io/actinia-python-client/ is used.
Fixes #348
Requires actinia-org/actinia-core#482