search

dpseidel / stmove

Build reproducible reports for spatial-temporal animal movement analysis
https://www.danaseidel.com/stmove/
GNU General Public License v3.0
3 stars 0 forks source link

AddIn does not respect a named vector for `seas` #12

Closed dpseidel closed 5 years ago

dpseidel commented 5 years ago

Currently if the seas argument is an unnamed vector the user get's this error:

Error in traj$seas[yday(traj$date) == seas[i]] <- names(seas)[i] : 
  replacement has length zero

which is pretty undecipherable. There should be improved handling such that if there are no names, the names default to seas1, seas2, etc..

dpseidel commented 5 years ago

this is the same error that gets called from the add-in but there, there needs to be additional handling of input -- right now everything is handled like a character string

dpseidel commented 5 years ago

I have fixed the original motivating bugs, but the addin still does not respect a named vector. No longer critical.

dpseidel commented 5 years ago

This is no longer an issue -- I'm not respecting named vectors in seas anywhere anymore