improvements to the LULC workflow example - #742
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
View / edit / reply to this conversation on ReviewNB ncclementi commented on 2026-08-13T21:58:21Z Should we link into other docs on how to setup the cluster? |
|
View / edit / reply to this conversation on ReviewNB ncclementi commented on 2026-08-13T21:58:21Z Not sure I follow what do you mean by "stock" RAPIDS installation, in the first sentence. The whole paragraph makes emphasis on how a rapids installation is not enough, but we are not installing ll rapids, so let's change the narrative around what we are using and how it integrates with cudf, cuml, etc.
Also where is this being installed? Was this a local cluster? If someone needs to use run this let's say in aws or similar how they get the cluster going? We should link to some docs to help in this step, and clarify we run this locally if that's what we did.
I think this feels a bit disconnected. Like how would the user get to this point?
|
|
View / edit / reply to this conversation on ReviewNB ncclementi commented on 2026-08-13T21:58:22Z Do we have a guide to point to people on how to create this file? I noticed we do provide a geojson file, We should mentioned this as an example. |
|
View / edit / reply to this conversation on ReviewNB ncclementi commented on 2026-08-13T21:58:23Z nitpick: the color labels are very hard to read is it possible to have the box somewhere else to make sure we can read it? |
| dependencies: | ||
| - python=3.13 | ||
| - cuda-version>=13.0,<=13.2 | ||
| - rapids=26.08 |
There was a problem hiding this comment.
Do we want to install all rapids? since we are already installing bunch of other stuff. Do we need all rapids?
ncclementi
left a comment
ncclementi
left a comment
There was a problem hiding this comment.
@jayavenkatesh19 and I did a final path review together, this is good to go.
2 participants
Closes #641