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 links including final project #626 #424

Merged
merged 10 commits into from
Dec 20, 2023
Merged

Add links including final project #626 #424

merged 10 commits into from
Dec 20, 2023

Conversation

eggvoice
Copy link
Collaborator

@eggvoice eggvoice commented Oct 2, 2023

No description provided.

@eggvoice
Copy link
Collaborator Author

eggvoice commented Oct 2, 2023

See issue #335 and #399

@eggvoice eggvoice changed the title Add link to NDVI analysis per # 335 and # 339 Add link to NDVI analysis per #335 and #399 Oct 2, 2023
@millie-spencer
Copy link
Collaborator

looks good to me!

@eggvoice eggvoice reopened this Oct 5, 2023
Copy link
Contributor

@ellenalamont17 ellenalamont17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice analysis. I might recommend adding a location in your header. It says the Caldor Fire but does not indicate where that is located. Only after I scroll down further do I see an indication that it is near Lake Tahoe. With earth data, everything is tied to location, Stating that location makes it easier for the reader to gain some context. Also, your plots have titles but no real interpretative descriptions. Maybe consider adding after each graph/immage.

Copy link
Collaborator

@taylor-obrien taylor-obrien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, Ed!

@eggvoice
Copy link
Collaborator Author

#444

@eggvoice
Copy link
Collaborator Author

eggvoice commented Dec 5, 2023

#568

@eggvoice eggvoice changed the title Add link to NDVI analysis per #335 and #399 Add links including final project #626 Dec 18, 2023
@eggvoice
Copy link
Collaborator Author

#626

@eggvoice
Copy link
Collaborator Author

@eculler can you merge this one too?? Thank you!!

@eculler eculler merged commit 341bf55 into main Dec 20, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants