search

Clay-foundation / model

The Clay Foundation Model (in development)
https://clay-foundation.github.io/model/
Apache License 2.0
236 stars 25 forks source link

Fix missing env #259

Closed yeelauren closed 2 weeks ago

yeelauren commented 1 month ago

Hey friends! While testing locally, I noticed that the environment.yml file seems to be missing some packages for the notebooks in /docs. I've made my attempt at fixing the version numbers.

I also noticed that this repo is missing a reference to the submodule stacchip

https://github.com/Clay-foundation/model/blob/a61f9faf4cadfdbef76b0651ae0aa638ef396323/docs/clay-v1-wall-to-wall.ipynb#L56

To get this to run locally, I had to copy and move the directory around for stacchip.

yellowcap commented 1 month ago

Not sure if we should also update the conda-lock.yml file with this?

yellowcap commented 1 month ago

We removed the stacchip library from the dependencies, as it was only used in one notebook for one tiny function. The updated wall-to-wall notebook is now independent of stacchip.

I think with this also the geoarrow and boto3 dependencues are no longer necessary. @yeelauren when you have time, please test your workflow again using main or the branch and let us know if the dependency issues were resolved.

Btw, we are also updating other libraries on the env file in https://github.com/Clay-foundation/model/pull/264

yellowcap commented 2 weeks ago

I think this has been fixed in updating the env here https://github.com/Clay-foundation/model/pull/264 and removing stacchip from the wall-to-wall example here https://github.com/Clay-foundation/model/pull/263.

@yeelauren feel free to reopen if you still encounter these problems.