CulverLab / CulverLab/sparcd-exploration

Tagger: correcting an estimated capture time leaves the ESTIMATED marker in place

Open
#254 0 comments 0 reactions 1 assignee Claimed by @Chris-Schnaufer View on GitHub
app:uploader
Dominant language
TypeScript
Stars
6
Forks
3
Avg merge
2d 13h
Merged PRs (30d)
106

Description

After #237, a file whose camera wrote no time uploads with `timestamp_issues` true in deployments.csv and a `[TIMESTAMP:]` marker in its media.csv comments. Fixing that time in the tagger rewrites the timestamp column only. The marker stays, so the file reopens with the ESTIMATED badge on a hand-corrected time, and `timestamp_issues` never flips back once every flagged file is corrected.

Where: `packages/camtrap/src/index.ts` `mergeMedia` writes col 4 and leaves col 10 alone.

Fix: when a correction lands on a flagged row, rewrite the marker to `manual` (or drop it), and recompute `timestamp_issues` when no flagged rows remain. Whether a hand-corrected time should still count as an issue is a team call, see item 1 in #237.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.