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.

Sign up for GitHub

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

Jump to bottom

test background task to calculate hourly averages #18

Closed
silvioheinze opened this issue Oct 1, 2024 · 5 comments · Fixed by #30
Closed

test background task to calculate hourly averages #18

silvioheinze opened this issue Oct 1, 2024 · 5 comments · Fixed by #30
Assignees
@n11ik
Labels
P3 Normal priority, to be addressed in regular development cycles. test For changes related to testing, like adding new tests or fixing existing ones.
Milestone
v0.3.x

Comments

@silvioheinze
Copy link
Contributor

No description provided.

@silvioheinze silvioheinze added P3 Normal priority, to be addressed in regular development cycles. test For changes related to testing, like adding new tests or fixing existing ones. labels Oct 1, 2024
@silvioheinze silvioheinze modified the milestones: v0.2.x, v0.3.x Oct 1, 2024
@n11ik
Copy link
Contributor

n11ik commented Nov 7, 2024

@silvioheinze

Ich würde für die hourly averages keinen Background-Task erstellen, weil das meiner Meinung nach zu redundanten Daten führt. Stattdessen können wir Materialized Views verwenden, was den Vorteil hat: weniger Redundanzen und es ist einfacher zu adaptieren.

@silvioheinze
Copy link
Contributor Author

Passt. Magst du das mal implementieren?

@n11ik
Copy link
Contributor

n11ik commented Nov 7, 2024

Ja mache ich.

n11ik pushed a commit that referenced this issue Nov 7, 2024
#18 added alembic migration to add hourly_avg view
5d1fccc
@n11ik
Copy link
Contributor

n11ik commented Nov 7, 2024

Ich habe jetzt mal einen normalen view erstellt, weil die postgresql datenbank von sich aus keine IMVs unterstützt, falls es später zu performance Problemen kommen sollte können wir andenken z.B. diese erweiterung zu verwenden.

Ich glaube das ein normaler view für unsere momentanen Datenmängen völlig ausreichend ist.

@silvioheinze
Soll ich auch gleich ein get auf der api einrichten, damit man die mittelwerte abfragen kann, sonst würde ich das issue schließen.

@silvioheinze
Copy link
Contributor Author

Ah, super, das kannte ich noch garnicht!

Schau dir mal die Geosphere API an (https://dataset.api.hub.geosphere.at/v1/openapi-docs) und überlege, ob es einen neuen Endpunkt braucht dafür oder ob es in einen vorhandenen (z.B. /station/historical) integriert werden kann.

n11ik pushed a commit that referenced this issue Nov 8, 2024
#18 added precision option hourly avg to /station/historical
073c2a6
@n11ik n11ik closed this as completed Nov 8, 2024
@n11ik n11ik linked a pull request Nov 8, 2024 that will close this issue
18 test background task to calculate hourly averages #30
Merged
n11ik pushed a commit that referenced this issue Nov 8, 2024
#18 exclude view from alembic
0b3f899
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@n11ik n11ik

Labels
P3 Normal priority, to be addressed in regular development cycles. test For changes related to testing, like adding new tests or fixing existing ones.
Projects
None yet
Milestone
v0.3.x
Development

Successfully merging a pull request may close this issue.

18 test background task to calculate hourly averages luftdaten-at/luftdaten-api
2 participants
@silvioheinze @n11ik