ocean-data-factory-sweden / kso

Notebooks to upload/download marine footage, connect to a citizen science project, train machine learning models and publish marine biological observations.
GNU General Public License v3.0
5 stars 12 forks source link

fix: update first two cells of notebooks #247

Closed victor-wildlife closed 1 year ago

victor-wildlife commented 1 year ago

I added a roadblock to restart the colab session if panoptes can't be imported, this avoids issues with the requests packages. I also got rid of the code to fix libmagic issues as it seems no longer needed, this was initially added to deal with panoptes issues. I combined the installation requirements into the first notebook cell and left the second cell to only load packages. I added a small description before the first cell for colab users

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Diewertje11 commented 1 year ago

We talked about the sys.path and we decided to change it to

sys.path.insert(0, os.path.abspath(os.path.join(os.getcwd(), "..")))

This still needs to be changed.

victor-wildlife commented 1 year ago

@jannesgg all good on your end to merge these changes?