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

hotfix: snafu from renaming ml_backends to processing_services #697

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

mihow
Copy link
Collaborator

@mihow mihow commented Jan 27, 2025

During the merging of #532 #684, something happened in which "ml_backends" was only partially renames to "processing_services" and we lost the precious "app.py" file. This prevented the ml_container from building and the tests from passing.

Some notes on naming:

  • "Processing Services" is how we will call the servers (or endpoints) that host an API for processing images from Antenna. These can now be added by the user and hosted themselves. They run one or more processing pipelines. The services may be synchronous or asynchronous, they may manage their own queue, they may be a cluster or a single server.
  • "ML Backend" is how we have been referring to the particular service for detecting and classifying moths with CNN models, like the one we have now.
  • So the "processing_services" folder will have a few of these examples. The docker-compose.yml will likely have multiple processing services running soon. The current one is called "ml_backend", this could change to "moth_identifiers" or something more specific.

Feel free to add your thoughts, sorry for the bad merge!!!

Copy link

netlify bot commented Jan 27, 2025

Deploy Preview for ami-dev canceled.

Name Link
🔨 Latest commit 54c6f3e
🔍 Latest deploy log https://app.netlify.com/sites/ami-dev/deploys/67979d07040dad0008c947c6

@mihow mihow merged commit 334cb29 into main Jan 27, 2025
6 checks passed
@mihow mihow deleted the fix/ml-backend-rename branch January 27, 2025 14:53
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.

1 participant