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

First Draft Review #4

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

First Draft Review #4

wants to merge 1 commit into from

Conversation

adammwilson
Copy link

Making some progress. Some suggestions:

  1. don't worry about shiny - it will make the analysis much more complicated. If you really want to show a timeseries, you could probably do it with grouplayers in leaflet as shown here. So you would bin them (by decade, maybe?) and then you could turn the markers on and off together)
  2. I'd like you to add some more spatial analysis of some kind. At a minimum, this could be a convex polygon around each species (showing the approximate range). Do you have plans for how you will do the LULC or Temperature analysis? This is a complex question - so please be careful to keep it focused on illustrating a particular analysis well rather than roughly exploring many things.

Making some progress.  Some suggestions:
1) don't worry about shiny - it will make the analysis much more complicated.  If you really want to show a timeseries, you could probably do it with grouplayers in leaflet as shown [here](https://rstudio.github.io/leaflet/showhide.html).  So you would bin them (by decade, maybe?) and then you could turn the markers on and off together)
2) I'd like you to add some more spatial analysis of some kind.  At a minimum, this could be a convex polygon around each species (showing the approximate range).
Invasive species are organisms that cause ecological or economic harm in environments in which they are not native. As globalization has spread and the world has grown exponentially smaller, invasive species have become one of the greatest challenges facing environmental managers today. Coupled with constant influx of imported goods is the ever-looming spectre of climate change. Biodiversity loss caused by climate change may leave room in an ecosystem in which the invader has no natural predators, causing species hierarchies to shift, and the invasive species to outcompete native species already vulnerable due to climate induced stress. While there are numerous other factors involved in the spread of invasive species, this map seeks to ask the question: **Is the spread of invasive species correlated with an increase in mean temperature? Land usage or land cover?** This project aims to build an interactive tool that allows users to trace the spread of numerous invasive species in order to further understand the factors that allow invasive species to spread.

# Materials and methods
# Materials and methods
Copy link
Author

Choose a reason for hiding this comment

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

Once you finalize them, fill this in with more details about your specific methods.

@@ -38,7 +38,7 @@ library(shinydashboard)
knitr::opts_chunk$set(cache=TRUE) # cache the results for quick compiling
```

## Download and clean all required data
## Download and clean all required data
Copy link
Author

Choose a reason for hiding this comment

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

you can put in a vector of species names and make a single call. e.g.

species=c("sp1","sp2")
spdf=occ(species, has_coords=TRUE, limit=1000000)%>%occ2df()

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

# Results
# Results
Copy link
Author

Choose a reason for hiding this comment

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

see my notes in the pull request - I would encourage you to not add a shiny app until you have your analysis worked out. I suspect that will take most of your time.

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