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

Project_Feedback_gbream #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

gbream
Copy link

@gbream gbream commented Nov 18, 2019

Nice job on the first draft. I find your topic to be extremely interesting as I have some background in biogeography and climate change. Having an interactive tool that can clearly communicate the spread of invasive species and infer climate change as a driver of that spread would be very useful in educating people about the issue.

@@ -5,10 +5,10 @@ subtitle: An interactive tool for understanding the distribution of invasive ins
output: html_document
---

# Introduction
# Introduction
Copy link
Author

Choose a reason for hiding this comment

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

This introduction does a good job of laying the groundwork for the project and clearly states the question you want to explore in your analysis. I do wonder, though, how you will be integrating the temperature increase mentioned in that question.

@@ -60,7 +60,7 @@ SWWdf<-occ2df(SWW)
Spocc downloads data via "occ" and converts to a dataframe via "occ2df".

# Results
My figures are not currently displayed, a placeholder image is below. The final project will have a slider built in Shiny that allows the user to control the timeframe of the data displayed.
My figures are not currently displayed, a placeholder image is below. The final project will have a slider built in Shiny that allows the user to control the timeframe of the data displayed.
Copy link
Author

Choose a reason for hiding this comment

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

The placeholder map shows what looks like distributions of various invasive species as well as the date they became established. Will temperature change over time be a layer or option for display on the final? I ask because your central question/hypothesis mentions a correlation with mean temperature rise.

@@ -82,4 +82,4 @@ This map will be wrapped into a larger Shiny fluidPage so that occurences can be

What have you learned? Are there any broader implications?

# References
# References
Copy link
Author

Choose a reason for hiding this comment

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

Add references

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.

1 participant