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

Josef final #6

Open
wants to merge 174 commits into
base: master
Choose a base branch
from

Conversation

DocJosef
Copy link

@DocJosef DocJosef commented Jul 1, 2024

Several Changes:
Changed file references to models (with s) in the UI files, in the docker compose
added volumes to my UI in docker-compose
dropped the fastapi_app
removed export of port 8000 from the UI form UI.Dockerfile and docker-compose
added an image
removed obsolete files

The UI opens now on my Mac and shows all pages.

probinb and others added 30 commits June 12, 2024 15:50
UI/Streamlit:
-Adapted UI to configure the number of required
core features
-Implemented a simple predict against an api call
using the acquired features
-Added a Reset Button to put everything on None
-Added a Use Default button, to put everything on
default again

fastapi:
- Simple authentication under /login
admin, password
- Endpoints for Health (/) and later monitoring (/request_status)
- Endpoint to put predict_request

Dockerfile
use
docker build -t accidents_ui:latest .
docker run -p 8000:8000 -p 8501:8501 accidents_ui:latest

open localhost:8000/streamlit to open UI
use admin, password to log in
fixed an issue with uploading the data to the db to make it faster
Model api merged into master branch
evan-stromatias and others added 30 commits July 7, 2024 17:17
Tested on Docker. Did run
Merged with Evans PR
This should be the final state then
…. To send email alerts the 'alert_email' variable must be set to point to an email throggh the airflow ui
…ates

Updated the readme.md, updated my tabs, removed the old Volumes/data/…
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