feat: climate risk from cmip cordex by jdroenner · Pull Request #18 · geo-engine/BioIS

jdroenner · 2026-07-08T22:55:23Z

No description provided.

coveralls · 2026-07-08T23:04:48Z

Coverage is 85.756% — cmip_cordex into main. No base build found for main.

ChristianBeilschmidt

Konnte das aus Gründen (Karte) nicht anwenden. Aber hier schon mal meine Kommentare.

ChristianBeilschmidt · 2026-07-09T06:17:56Z

      volumeMounts:
        - name: pgdata
-          mountPath: /var/lib/postgresql/data
+          mountPath: /var/lib/postgresql/18/docker


Warum ist das anders, obwohl sich sonst nichts an der Container-Config geändert hat?

docker-library/postgres#1259

ChristianBeilschmidt · 2026-07-09T06:24:59Z

+
+/// Calculates climate-risk indicators from CORDEX/CMIP5 climate data for a given point and time window.
+#[derive(Debug, Clone)]
+pub struct ClimateCordex5Process;


Vielleicht mehr User-facing benennen. Was macht der User damit?

Cordex sagt ja mehr über die Quelldaten.

ich hab es umbenannt

Ok, am besten auch noch Modul umbenennen.

ChristianBeilschmidt · 2026-07-09T06:41:50Z

+    let wfl_ids =
+        try_join_all(variable_workflows.map(|wfl| register_workflow_handler(configuration, wfl)))
+            .await;
+
+    let wfl_ids = match wfl_ids {
+        Ok(id) => id,
+        Err(e) => {
+            if let Some(error) = error_response(&e) {
+                anyhow::bail!("Failed to register a workflow `{e}`: {error:?}");
+            }
+            anyhow::bail!("Failed to register a workflow `{e}`");
+        }
+    };
+
+    let wfl_id_str = wfl_ids
+        .iter()
+        .map(|workflow_id| workflow_id.id.to_string())
+        .collect::<Vec<_>>();
+
+    for (wf_id, v) in wfl_id_str.iter().zip(var_props.iter()) {


Die Variablen-Namen werden in einem Monat kryptisch sein. Ist nicht so gebräuchlich, also ausschreiben?

Kryptische Variablennamen sind ausgeschrieben (z.B. model statt m, workflow statt wfl).

ChristianBeilschmidt · 2026-07-09T06:43:56Z

@@ -0,0 +1 @@
+./backend/rust-toolchain.toml


Coverage am besten > 80% lassen.

ChristianBeilschmidt

Ich konnte jetzt erst einmal nur Code-only anschauen. Müssen noch mal wegen der Daten sprechen.

ChristianBeilschmidt · 2026-08-07T14:25:01Z

+    /// Name of a row property that carries the cell color (hex) for this column.
+    #[serde(default, skip_serializing_if = "Option::is_none")]
+    pub color_field: Option<String>,
+    /// Name of a row property that carries a ready-to-display label for this column.
+    /// When set, the frontend renders it instead of the raw value.
+    #[serde(default, skip_serializing_if = "Option::is_none")]
+    pub label_field: Option<String>,


Check: Sind das validate table schema Felder (https://datapackage.org/standard/table-schema/)

Erledigt. Alle verwendeten Feld-Typen sind gültige Table-Schema-Typen (string/number/integer/boolean). Die nicht standardkonformen list/itemType wurden durch type: "array" mit items-Deskriptor ersetzt.

ChristianBeilschmidt · 2026-08-07T14:26:10Z

    String,
    Number,
    Integer,
+    Percentage,


Es gibt keinen Percentage-Typ bei TableSchema: https://datapackage.org/standard/table-schema/#field-types

Percentage ist jetzt nur noch ein Units-Helfer und taucht nicht mehr als Table-Schema-Feld-Typ auf. Die Prozent-Anzeige (z.B. "+10 days (+20 %)") läuft über die biois-Extension (labelField/colorField).

ChristianBeilschmidt · 2026-08-07T14:28:31Z

+
+impl Default for YearRange {
+    fn default() -> Self {
+        YearRange(20)


Ich denke nicht, dass das ein guter Default ist.
Wenn das bei dem einen Prozess sinnvoll ist, dann vielleicht den Default dort definieren.

Der Default wurde aus units.rs entfernt und ist jetzt prozesslokal definiert (default_year_range() in climate_risk).

ChristianBeilschmidt · 2026-08-07T14:29:18Z

    }
 }

+/// Length of a climate window in years (e.g., 5-30 years).


Suggested change

/// Length of a climate window in years (e.g., 5-30 years).

/// Length of a time window in years (e.g., 5 years).

Hat ja nicht immer was mit Klima zu tun und man kann kein von bis angeben.

Docstring wie vorgeschlagen angepasst: Length of a time window in years (e.g., 5 years).

ChristianBeilschmidt · 2026-08-07T14:32:58Z

+    pub region: Option<CordexRegion>,
+}
+
+#[derive(Deserialize, Serialize, Debug, JsonSchema, ToSchema, Clone)]


brauchst du bei den Ausgaben immer auch Deserialize?

Für die Ausgaben wird nur noch Serialize benötigt; Deserialize ist auf die Eingabetypen beschränkt.

ChristianBeilschmidt · 2026-08-07T15:20:16Z

+                </td>
+              }
+            }
+            @case (ColumnType.Percentage) {


Den Typ gibt es ja nicht in TableSchema

Erledigt - der Feld-Typ heißt jetzt array (mit items-Deskriptor) statt list, konform zu Table Schema.

ChristianBeilschmidt · 2026-08-07T15:27:48Z

+                  <mat-chip class="cell-content">
+                    <span
+                      class="color-dot"
+                      [style.background-color]="element[column.colorField]"


Irgendwie fühlt es sich falsch an, eine Spalte mit Farbwerten zu haben.
Evtl. finden wir andere Felder für Metadaten oder müssen eine Extension machen (https://datapackage.org/standard/extensions/).

Die Farb-/Label-Werte sind jetzt hiddenFields und über eine Table-Schema-Extension (biois.display mit kind/labelField/colorField) an die Datenfelder gebunden. Keine sichtbare Farbwert-Spalte mehr.

ChristianBeilschmidt · 2026-08-07T15:28:55Z

+          title:
+            typeof innerValue === 'object' &&
+            innerValue !== null &&
+            'name' in innerValue &&
+            typeof innerValue.name === 'string'
+              ? innerValue.name
+              : this.fieldName(key),


Woh, was ist hier passiert? Evtl. eine Funktion mit Doc?

Der Code ist in benannte Funktionen mit Doc-Kommentaren zerlegt (u.a. dataResourceTitle, fixDataValue).

ChristianBeilschmidt · 2026-08-07T15:31:09Z

+
+/// Calculates climate-risk indicators from CORDEX/CMIP5 climate data for a given point and time window.
+#[derive(Debug, Clone)]
+pub struct ClimateCordex5Process;


Ok, am besten auch noch Modul umbenennen.

jdroenner · 2026-08-13T00:13:28Z

Alle Review-Kommentare wurden adressiert. Zusammenfassung der Änderungen:

Backend (climate_risk):

Modul aufgeteilt in compute.rs / types.rs / workflow.rs / mod.rs
Eingaben werden am Prozessrand typisiert via parse_inputs (mit Fehlerkontext) konvertiert; keine rohen JSON-Werte mehr
expect aus dem User-Input-Pfad entfernt (Result-Propagation)
ProbabilityClass als Enum modelliert, Schwellwerte aus return_period_years() (1:20_000 bis 1:5); 365.25 als DAYS_PER_JULIAN_YEAR
Überlappende Regionen: nearest_containing wählt die nächstgelegene enthaltene Bounding Box (mit Test)
Tests: alle it_...-Konvention, approx-Asserts, Props-Tests konsolidiert
Anzeige (Label/Farbe) über biois-Table-Schema-Extension mit hiddenFields statt sichtbarer Farbspalten

Table Schema (shared):

Nicht standardkonforme Typen list/itemType ersetzt durch array + items-Deskriptor (gültige Table-Schema-Felder)
Percentage nur noch Units-Helfer, kein Feld-Typ

Frontend:

array-Typ-Unterstützung, hiddenFields-Filterung, resolveItemsSchema/resolveSchemaRef-Refactoring, JSDoc ergänzt

Prüfungen: cargo test --lib 146 passed (11 bekannte bb8-Timeouts der DB), Clippy/Fmt clean, Frontend 95 Tests + Lint/Prettier clean. @ChristianBeilschmidt gerne erneut reviewen.

jdroenner changed the title ~~feat: climeate rist from cmip cordex~~ feat: climate risk from cmip cordex Jul 8, 2026

ChristianBeilschmidt requested changes Jul 17, 2026

View reviewed changes

ChristianBeilschmidt requested changes Aug 7, 2026

View reviewed changes

Integrate CMIP/CORDEX climate-risk analysis

a355028

jdroenner force-pushed the cmip_cordex branch from 4809bbb to a355028 Compare August 13, 2026 14:15

		@@ -0,0 +1 @@
		./backend/rust-toolchain.toml No newline at end of file

	/// Length of a climate window in years (e.g., 5-30 years).
	/// Length of a time window in years (e.g., 5 years).

Uh oh!

Conversation

jdroenner commented Jul 8, 2026

Uh oh!

coveralls commented Jul 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ChristianBeilschmidt left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ChristianBeilschmidt left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

jdroenner commented Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

coveralls commented Jul 8, 2026 •

edited

Loading