Skip to content

Model Monitors endpoints#55

Merged
sinnec merged 6 commits intomainfrom
24-monitoring-alerting-functionality
Dec 12, 2022
Merged

Model Monitors endpoints#55
sinnec merged 6 commits intomainfrom
24-monitoring-alerting-functionality

Conversation

@sinnec
Copy link
Copy Markdown
Contributor

@sinnec sinnec commented Dec 7, 2022

No description provided.

- Created basic endpoints for model monitors
- Adjusted schema for model monitors and alerts
- Some cleaup
@sinnec sinnec linked an issue Dec 7, 2022 that may be closed by this pull request
@sinnec sinnec marked this pull request as draft December 7, 2022 11:33
Comment thread src/api/v1/model_monitors.py Outdated
Comment thread src/schemas/modelMonitor.py Outdated
Comment thread src/schemas/modelMonitor.py
Comment thread .gitignore
@sinnec sinnec marked this pull request as ready for review December 9, 2022 11:21
@squaredev-io squaredev-io deleted a comment from gcharis Dec 9, 2022
Copy link
Copy Markdown
Member

@momegas momegas left a comment

Choose a reason for hiding this comment

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

This is an awesome first iteration.
Great Job 👍

Comment thread src/api/v1/model_monitors.py Outdated
Comment thread src/cron_tasks/monitoring_alerts.py
Comment thread src/cron_tasks/monitoring_alerts.py Outdated
@sinnec sinnec merged commit 97481f6 into main Dec 12, 2022
@gcharis gcharis deleted the 24-monitoring-alerting-functionality branch December 20, 2022 12:17
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.

Monitoring / alerting functionality

3 participants