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

Delete old birdflow_crs object #203

Open
ethanplunkett opened this issue Jan 13, 2025 · 2 comments
Open

Delete old birdflow_crs object #203

ethanplunkett opened this issue Jan 13, 2025 · 2 comments

Comments

@ethanplunkett
Copy link
Contributor

birdflow_crs defines the projection used in the original BirdFlow Shiny application. It is no longer particularly meaningful.

chenyangkang added a commit to chenyangkang/BirdFlowR that referenced this issue Jan 13, 2025
@chenyangkang
Copy link

Is this preprocess_species function still needed? As it uses the birdflow_crs

preprocess_species <- function(species = NULL,

@ethanplunkett
Copy link
Contributor Author

Yes. preprocess_species() is the first step in fitting any BirdFlow model so we definitely need it; but preprocess_species() doesn't use birdflow_crs. One of its tests does though. The line that has to change is in test-preprocess_species . You could either define that projection locally within that test or use a different projected CRS in that test - EPSG:5069 might work: terra::crs("EPSG:5069").

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

No branches or pull requests

2 participants