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

Tilføj funktionalitet til analyse af højdetidsserier #779

Merged
merged 30 commits into from
Oct 21, 2024

Conversation

krebslw
Copy link
Collaborator

@krebslw krebslw commented Oct 3, 2024

Dette PR tilføjer følgende underkommandoer til FIREs ts-kommandogruppe:

  • fire ts hts : udtræk en højdetidsserie
  • fire ts plot-hts : plot en højdetidsserie
  • fire ts analyse-hts : analysér en (eller flere) højdetidsserie(r)

Alle tre kommandoer har allerede-eksisterende pendanter til at udføre samme handlinger med GNSS-tidsserier.

Commitsne er opdelt som følger:

  • nødvendige tilføjelser og ændringer til tidsserie-API'et. Herunder et par refaktoriseringer for at generalisere noget af den eksisterende funktionalitet som var målrettet GNSS-tidsserier.
  • opret hjælpemoduler under ts-modulet
  • tilføj nye underkommandoer
  • tilføj mulighed i fire niv regn for at plotte nyberegnede koter, hvis de er knyttet til en tidsserie.
  • opret/rediger tests
  • alle nye kommandoer (inkl. info-, niv- og luk- underkommandoer fra forrige PR's) tilføjes til sphinx dokumentation.

Uddybende brugervejledning under docs/ vil komme i et senere PR.

Copy link
Collaborator

@kbevers kbevers left a comment

Choose a reason for hiding this comment

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

Første runde review. Jeg er kommet til fddba0d. Fortsættes i morgen.

Implementerer WLS (Weighted Least Squares)

Derudover fjernes klassens attribut "tidsserier", da det ledte til
en cirkulær reference
Afledte statistik-parametre blev beregnet i API'et, som så blev
brugt af CLI-laget til at rapportere videre til brugeren.

Generelt ved API'et dog ikke hvad brugeren gerne vil vide.
Derfor stiller API'et nu bare sine beregningsmetoder til rådighed,
som så kan bruges til at beregne diverse afledte statistikparametre
i CLI-laget hvor det er relevant.
Copy link
Collaborator

@kbevers kbevers left a comment

Choose a reason for hiding this comment

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

Det ser ud til at der er taget hånd om de forskellige kommentarer jeg havde, så hvis du ellers er klar er det bare merge

@krebslw
Copy link
Collaborator Author

krebslw commented Oct 21, 2024

Har tilføjet alt til sphinx dokumentation, foruden diverse smårettelser. Merger nu, så det er klart til at folk kan hente det ned inden workshop i overmorgen. Hvis der kommer rettelser på baggrund af feedback fra workshoppen, så må det komme med senere.

@krebslw krebslw merged commit bbc5007 into SDFIdk:master Oct 21, 2024
1 check passed
@krebslw krebslw deleted the analyse-hts branch November 19, 2024 13:14
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.

2 participants