Skip to content

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

Open

Description

@GabySamaniego

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!

Activity

  1. ezylstra commented on Dec 19, 2023

    @ezylstra
    Contributor

    Sounds great! I'll plan on reviewing tomorrow

  2. ezylstra commented on Dec 21, 2023

    @ezylstra
    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.

  3. GabySamaniego commented on Dec 21, 2023

    @GabySamaniego
    ContributorAuthor

    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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions