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

Move to deployments → media → observation hierarchy #279

Merged
merged 4 commits into from
Feb 13, 2023

Conversation

peterdesmet
Copy link
Member

@peterdesmet peterdesmet commented Feb 7, 2023

This PR is a first step at updating the Camtrap DP structure, discussed in #203 (comment). It creates a linear hierarchy:

deployments → media → observations

It does so by:

  • Removing the potentially conflicting foreign key deploymentID from observations (join should be made via media)
  • Removing the redundant field timestamp from observations (info can be obtained from media)
  • Removing mediaID from observations. Note that this currently restricts Camtrap DP to event-based observations only, but that will be resolved in a next update where we rename observations to event-observations and add a media-observations table.
  • Updating the definitions of the media and observations table
  • Renaming sequenceID to eventID in both the media and observation table
  • Renaming sequenceInterval to eventInterval in the profile
  • Renaming the controlled value sequence to event in classificationLevel in the profile
  • Renaming sequence to event in definitions where relevant. The only occurrence is now in media.eventID:

Unique identifier of the event the media file belongs to. Events contain one or more media files (e.g. a single image or video or a sequence of successive images or videos) and are defined by package.project.eventInterval.

  • Updating the example dataset accordingly
  • Minor (unrelated) change to homepage and adding sitemap

The PR bumps the version from 0.5 to 0.6.

- Remove `observations.deploymentID` from schema and example
- Remove `observations.timestamp` from schema and example
- Remove `observations.mediaID` from schema and example
- Rename `media.sequenceID` to `media.eventID` in schema and example
- Rename `observations.sequenceID` to `observations.eventID` in schema and example
@peterdesmet
Copy link
Member Author

Rather than merging this to main, I have created a 0.6 branch, that will include all upcoming changes for the release candidate.

@peterdesmet peterdesmet merged commit 9839b4f into 0.6 Feb 13, 2023
@peterdesmet peterdesmet deleted the new-structure branch February 13, 2023 10:14
@peterdesmet peterdesmet mentioned this pull request Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants