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

Clean-RMNP-recaptured-data script ready for review #6

Open
GabySamaniego opened this issue Dec 19, 2023 · 3 comments
Open

Clean-RMNP-recaptured-data script ready for review #6

GabySamaniego opened this issue Dec 19, 2023 · 3 comments

Comments

@GabySamaniego
Copy link
Contributor

I finished reviewing and updating the code for cleaning the recaptured data. @ezylstra, I would appreciate it if you could look at it and let me know what you think, especially my comments on lines 407 to 411 and the TODO comments at the end of the script. Thanks!

@ezylstra
Copy link
Contributor

Sounds great! I'll plan on reviewing tomorrow

@ezylstra
Copy link
Contributor

@GabySamaniego, I just went carefully through the code and I think it all looks great! All the steps you had in there made sense to me, and you've managed to get all the information you'll need moving forward into just two columns (original and new). I'm impressed. All the TODO items look good too.

I'm sure questions may come up as you start merging information and band numbers with the banding dataset. Just let me know if I can help figure things out.

@GabySamaniego
Copy link
Contributor Author

Thank you @ezylstra! I'll continue working on the other scripts and let you know if I have any questions.

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