weecology / PortalData

Official Repo of the Portal Project Data
Other
46 stars 30 forks source link

Add code check for double-recaps in the same period #316

Closed bleds22e closed 5 years ago

bleds22e commented 5 years ago

We've run into the problem a couple times now where the code says we have a new tag that needs a star but that new tag is on a removal. Sometimes this is because that rodent was caught on the first night, tagged, and then caught on the second night in a removal plot.

It would be nice to have a line of code that tells us that information.

diazrenata commented 5 years ago

Do we want a data flag note1 = 19 for this?

And if so do we want to add it retroactively to past censuses? I'm fiddling around right now and there are ~800 instances of the same tag occurring more than once in a single period (not counting NA/0/-1 or negative period codes).

bleds22e commented 5 years ago

Hm. I don't think we really need to add a note1 code for it. I think the bigger issue at hand is having the QAQC code flag it if it comes up to remind us that it is a possibility if we're confused about why it doesn't have a code.

Ellen K. Bledsoe

Ph.D. Candidate, Interdisciplinary Ecology

School of Natural Resources and Environment

Dept. of Wildlife Ecology and Conservation

University of Florida

ellen.bledsoe@ufl.edu she/her/hers


From: Renata Diaz notifications@github.com Sent: Tuesday, May 14, 2019 2:09:11 PM To: weecology/PortalData Cc: Bledsoe,Ellen; Assign Subject: Re: [weecology/PortalData] Add code check for double-recaps in the same period (#316)

Do we want a data flag note1 = 19 for this?

And if so do we want to add it retroactively to past censuses? I'm fiddling around right now and there are ~800 instances of the same tag occurring twice in a single period (not counting NA/0/-1 or negative period codes).

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_weecology_PortalData_issues_316-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DADLRQLI3UDHNK7HHO4IHZDLPVL54PA5CNFSM4HGDOLKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVMKBOQ-23issuecomment-2D492347578&d=DwMCaQ&c=sJ6xIWYx-zLMB3EPkvcnVg&r=S0VcwTKeaPiJAtTkGWUZUyPBRXjEHRfvwINR3lSlytc&m=VgXheg_Ee72EgHdS2uKGYRvc2OrV7QVtHoKk8lbzMqk&s=IxFCT3Se0wXC6KTZ4PKPArNt4quIdwxvYSzQbc3TD5w&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ADLRQLMTDISVDSUXSJ6QLFTPVL54PANCNFSM4HGDOLKA&d=DwMCaQ&c=sJ6xIWYx-zLMB3EPkvcnVg&r=S0VcwTKeaPiJAtTkGWUZUyPBRXjEHRfvwINR3lSlytc&m=VgXheg_Ee72EgHdS2uKGYRvc2OrV7QVtHoKk8lbzMqk&s=q-NmQ5VdXS9rh98gRS5i_HSdKJsIx0V3MqM2NB9pXwU&e=.

diazrenata commented 5 years ago

🙌 that's significantly easier. #318 implements the change to the QA process.