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

pt 1072 missing in raster dataset #76

Open
mkclapp opened this issue May 25, 2022 · 4 comments
Open

pt 1072 missing in raster dataset #76

mkclapp opened this issue May 25, 2022 · 4 comments

Comments

@mkclapp
Copy link
Collaborator

mkclapp commented May 25, 2022

This point is in the Silver cluster and seems to be missing from the dataset that summarizes the raster files in the google drive link https://drive.google.com/drive/folders/1vNSUlaz34OA4uYNDjOkT_lToY60y0BPo. We do have point count data for this point.

@mkclapp
Copy link
Collaborator Author

mkclapp commented May 25, 2022

[] diagnose root of issue in raster data
[] if data for that point exists, edit BACI_setup.R to include it in the y array/data list

@ddkapan
Copy link
Collaborator

ddkapan commented Aug 17, 2022

@mkclapp The new code just merged to the repo fixes this. We now have point 1072 + metadata from plots and satellite to describe it. So I suggest you follow your second step immediately above and then close this issue. Thanks!

@mkclapp
Copy link
Collaborator Author

mkclapp commented Aug 17, 2022

Ok great! If we don't want the point count BACI data on this repo, this issue can be closed immediately and the outstanding merge request #77 could be merged. Otherwise that merge request will need to be rejected and I will start again on a new branch of main...

@ddkapan
Copy link
Collaborator

ddkapan commented Aug 23, 2022

I'm thinking it would be good to keep everything together until we have a dedicated ML ARU codebase. I'm happy to discuss, but IMO Let's not spend time splitting the repo at the same time as writing the new paper, without checking for dependencies, unless this a very simple fix, it sounds like it might cause some downstream problems.

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