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

Rework how fields in the bounding box JSON are represented #45

Open
kevinsbarnard opened this issue Nov 15, 2023 · 1 comment
Open

Rework how fields in the bounding box JSON are represented #45

kevinsbarnard opened this issue Nov 15, 2023 · 1 comment
Assignees
Labels
core Changes to the application core enhancement New feature or request

Comments

@kevinsbarnard
Copy link
Collaborator

The bounding box JSON has taken on several metadata fields in addition to the box dimensions (and potential linked image reference). Among others, these include:

  • observer
  • verifier
  • generator

The presence of these metadata in the JSON is unideal for a few reasons. Primarily, the JSON schema is not formally standardized or described, so there is ambiguity in the population and interpretation of these values. In addition, this schema tends to change over time to suit new needs and would benefit from being separated from the core x, y, width, height, and image_reference_uuid fields that describe the bounding box and its source imagery.

We should rework how we are handling these fields. This is part of a larger discussion of how best to structure the bounding box/track information within VARS.

@kevinsbarnard kevinsbarnard added the enhancement New feature or request label Nov 15, 2023
@kevinsbarnard kevinsbarnard added the core Changes to the application core label Nov 16, 2023
@kevinsbarnard
Copy link
Collaborator Author

Related to mbari-org/annosaurus#14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Changes to the application core enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants