Ecoscope 197

[davidovichmarcos]

Adding tests to execute notebooks with papermill on the CI

[cisaacstern]

@davidovichmarcos thanks so much for getting this so close, overall looks great.

I did take the somewhat "extreme" stance of setting this review as "Request changes" because there are a few things that I think we should clean up here before merge.

In addition to the comments below, I think we should rename _tests.yml to something like notebook-tests.yml which is (a) more descriptive; and (b) doesn't start with underscore (because it is running directly, not being inherited).

Let me know if I can clarify anything. Again, amazing work, this will be a huge leap forward here! 👏

[atmorling]

Thanks for sorting this out @davidovichmarcos 🙏
The only thing I'd add in addition to the credentials configuration is a preference to to make this more aligned with the existing workflows which have an environment.yaml / micromamba setup, rather than requirements.txt

As I was writing this I realised we probably want to do away with both of these approaches and do something simpler per the CI in ecoscope-workflows. Have created #247 to track that and leaving this here just to complete the loop. No action required here from you @davidovichmarcos

[cisaacstern]

Amazing!!!

Now that we have mapping deps coming from ecoscope directly, I don't think we need folium and branca (see below).

[cisaacstern]

🚀 👏