Skip to content

feat: ndvi from stac#11

Open
michaelmattig wants to merge 11 commits intomainfrom
ndvi-stac
Open

feat: ndvi from stac#11
michaelmattig wants to merge 11 commits intomainfrom
ndvi-stac

Conversation

@michaelmattig
Copy link
Copy Markdown
Contributor

@michaelmattig michaelmattig commented Mar 20, 2026

No description provided.

@michaelmattig michaelmattig changed the title ndvi from stac feat: ndvi from stac Mar 20, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Mar 20, 2026 •
edited
Loading

Coverage Status

coverage: 80.578% (+0.7%) from 79.899% — ndvi-stac into main

@ChristianBeilschmidt
Copy link
Copy Markdown
Member

ChristianBeilschmidt commented Mar 20, 2026 •
edited
Loading

Habe ich einen gedanklichen Fehler, oder wäre die Config nicht

[geoengine]
base_url = "https://geoengine.biois.app.geoengine.io/api"

?

Der Fehler ist

2026-03-20T16:16:45.406573Z ERROR ogcapi_services::error: Generic error: Failed to perform OIDC login

Caused by:
error in response: status code 400 Bad Request

2026-03-20T16:16:45.406993Z ERROR tower_http::trace::on_failure: response failed, classification: Status code: 500 Internal Server Error, latency: 48 ms
at /home/beilschmidt/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/tower-http-0.6.8/src/trace/on_failure.rs:93
in tower_http::trace::make_span::request with method: POST, uri: /auth/accessTokenLogin?redirectUri=http%3A%2F%2Flocalhost%3A4200%2Fapp%2Fsignin, version: HTTP/1.1

@michaelmattig
Copy link
Copy Markdown
Contributor Author

michaelmattig commented Mar 24, 2026

Habe ich einen gedanklichen Fehler, oder wäre die Config nicht

[geoengine]
base_url = "https://geoengine.biois.app.geoengine.io/api"

?

Der Fehler ist

2026-03-20T16:16:45.406573Z ERROR ogcapi_services::error: Generic error: Failed to perform OIDC login

Caused by:
error in response: status code 400 Bad Request

2026-03-20T16:16:45.406993Z ERROR tower_http::trace::on_failure: response failed, classification: Status code: 500 Internal Server Error, latency: 48 ms
at /home/beilschmidt/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/tower-http-0.6.8/src/trace/on_failure.rs:93
in tower_http::trace::make_span::request with method: POST, uri: /auth/accessTokenLogin?redirectUri=http%3A%2F%2Flocalhost%3A4200%2Fapp%2Fsignin, version: HTTP/1.1

nö ist richtig. bei mir funktioniert es aber auch

@ChristianBeilschmidt
Copy link
Copy Markdown
Member

ChristianBeilschmidt commented Mar 25, 2026

Vielleicht kannst du meinen User noch mal aus der DB löschen? Irgendwas muss ja falsch sein.

ChristianBeilschmidt
ChristianBeilschmidt requested changes Apr 2, 2026
View reviewed changes
Comment thread backend/src/util.rs
Comment thread backend/src/util.rs
Comment thread backend/src/util.rs
ChristianBeilschmidt
ChristianBeilschmidt requested changes Apr 10, 2026
View reviewed changes
Comment thread backend/src/processes/ndvi.rs
Comment thread backend/src/processes/ndvi.rs
ChristianBeilschmidt
ChristianBeilschmidt requested changes Apr 14, 2026
View reviewed changes
Comment thread backend/Cargo.toml Outdated
github-code-quality[bot]
github-code-quality Bot found potential problems Apr 16, 2026
View reviewed changes
Comment thread api-client/typescript/models/HabitatDistanceProcessInputs.ts Dismissed
Comment thread api-client/typescript/models/HabitatDistanceProcessOutputs.ts Dismissed
Comment thread api-client/typescript/models/HabitatDistanceProcessParams.ts Dismissed
Comment thread api-client/typescript/types/ObjectParamAPI.ts Dismissed
Comment thread api-client/typescript/types/PromiseAPI.ts Dismissed
Comment thread api-client/typescript/types/ObservableAPI.ts Dismissed
ChristianBeilschmidt
ChristianBeilschmidt requested changes Apr 21, 2026
View reviewed changes
Comment thread backend/src/processes/ndvi.rs Outdated
ChristianBeilschmidt
ChristianBeilschmidt requested changes Apr 21, 2026
View reviewed changes
Comment thread frontend/src/app/result/result.component.ts
Comment thread frontend/src/app/result/result.component.ts
processId: this.processId(),
}),
defaultValue: {},
defaultValue: this.mockInputs,
Copy link
Copy Markdown
Member

@ChristianBeilschmidt ChristianBeilschmidt Apr 21, 2026

Choose a reason for hiding this comment

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

Wieso kann das nicht erst einmal leer sein?

Copy link
Copy Markdown
Contributor Author

@michaelmattig michaelmattig Apr 21, 2026

Choose a reason for hiding this comment

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

bisher war alles optional. jetzt gibt es die input-Outputs (der Punkt für den NDVI berechnet werden soll) immer. daher ist ein leeres Objekt jetzt kein gültiges Result mehr.

Copy link
Copy Markdown
Member

@ChristianBeilschmidt ChristianBeilschmidt Apr 29, 2026

Choose a reason for hiding this comment

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

Das mit dem mock-Placeholder ist das einzige, was noch stört. Irgendwie wäre das gut, wenn es erst einmal leer sein kann. Denn NDVi ist ja nicht das einzige Result. Da ist erst einmal nichts (nichts rendern) und dann ist irgendwann was, oder?

Comment thread frontend/src/app/result/result.component.ts Outdated
Comment thread frontend/src/app/result/result.component.ts Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChristianBeilschmidt ChristianBeilschmidt ChristianBeilschmidt requested changes

+1 more reviewer

@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@michaelmattig @coveralls @ChristianBeilschmidt