Skip to content
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 files via upload #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

eseigeldanf
Copy link
Contributor

index.ipynb - removed Colab chunk, formatted for consistency
index.Colab - made Colab notebook for users

index.ipynb - removed Colab chunk, formatted for consistency
index.Colab - made Colab notebook for users
@eseigeldanf eseigeldanf added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 21, 2025
@eseigeldanf
Copy link
Contributor Author

HI DONT APPROVE YET I don't have the Colab chunk in the Colab document

Replaced index-Colab so it DOES have the Colab chunk for realsies this time
@eseigeldanf
Copy link
Contributor Author

Ok I have added the Colab chunk to the new index-Colab file and now its only test fail is the normal exit error 1

@HaleySchuhl
Copy link
Contributor

Looking good so far. Thanks for tackling this, @eseigeldanf .

I noticed in the error that it's saying HTTP Error 404: Not Found and the error is triggered at this line img = pcv.io.open_url(url=" https://raw.githubusercontent.com/danforthcenter/plantcv-tutorial-color-correction/main/img/plant-image.jpg ").

It looks like the actual test image for this tutorial is called 1_B73_sand_C_2023-04-14_10_19_07.jpg so could you try updating the url path in that open_url line of code?

Updated formatting for both index and index-Colab
Changed the image URL in Colab to be current
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants