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

Api2 first commit #131

Merged
merged 3 commits into from
Sep 18, 2023
Merged

Api2 first commit #131

merged 3 commits into from
Sep 18, 2023

Conversation

micowan
Copy link
Collaborator

@micowan micowan commented Sep 9, 2023

Moving from the old domain AIForOrcas, to the new domain OrcaHello.

The API represents a ground up rebuild, taking into account the new container design and data schema created to address some performance issues and additional requests by Scott and Dave. Specifically:

  1. Creation of a new property called "state". This is also the new partition key. When the ML process deposits a new metadata/sampling it should do so with a state of "Unreviewed".
  2. Removal of the "reviewed" and "SRKWFound" properties (combined into "state")
  3. Addition of a number of composite indexes to make locating and sorting records more efficient.
  4. Modification of existing endpoints and additional endpoints to address new requested data pulls
  5. Hydrophone information is now being pulled from external API provided by Scott and not hard coded so adding new ones should be much easier

Copy link
Member

@micya micya left a comment

Choose a reason for hiding this comment

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

I don't think anyone will review 200+ files, so approving on the condition that we smoke test the new web app before submission.

@micowan
Copy link
Collaborator Author

micowan commented Sep 14, 2023 via email

@micowan micowan closed this Sep 18, 2023
@micowan micowan reopened this Sep 18, 2023
@micowan micowan merged commit 72fe3d5 into main Sep 18, 2023
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