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

Define position observations #430

Merged
merged 2 commits into from
May 16, 2024
Merged

Conversation

cookeac
Copy link
Collaborator

@cookeac cookeac commented Nov 16, 2023

Define position observation events (group and animal) that allow you to record that an animal (or group/mob) was seen (observed) at a given named position or geospatial location.

Resolves #383

Note: #383 talked about these as "location" events originally, but this could cause confusion with Location = Farm/Holding.
In this context we mean a barn, pen, field, paddock, or other named area, or a geospatial location (a polygon or a point such as a lat/long position).

Note: This PR references the GeoJSON geometry object defined at https://geojson.org/schema/Geometry.json
This currently causes an OAS3 parsing error, because the GeoJSON schema uses the $schema and #id keywords that are not supported in OpenAPI 3.0. They will be supported when we upgrade to OpenAPI 3.1 and the error should disappear.

Define position observation events (group and animal) that allow you to record that an animal (or group/mob) was seen (observed) at a given named position or geospatial location.

Resolves adewg#383
@cookeac cookeac mentioned this pull request Apr 21, 2024
Add the icarSortingSiteResource identifier (site) to position observations, so you can record a position observation to confirm that an animal has been sorted into that site.
@cookeac
Copy link
Collaborator Author

cookeac commented Apr 30, 2024

Updated position observations to support the site ID from sorting sites. See also #456.

"type": "string",
"description": "Identifier for a sorting site (icarSortingSiteResource) for this position."
},
"geometry": {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was a little bit worried how this property will be generated in the c# code, but seems that there is nothing special about it. Works fine.
image

Copy link
Collaborator

@AlexeyHardCode AlexeyHardCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes looks fine. Code generation works as expected.

@cookeac cookeac merged commit 68fa35b into adewg:Develop May 16, 2024
@cookeac cookeac deleted the location-observation-383 branch September 5, 2024 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants