Skip to content

Make Geokoordinaten/Laengen+Breitegrade nullable; Add NoerdlicheBreite, OestlicheLaenge, Zone, Nordwert and Ostwert #597

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
Merged
merged 8 commits into from
Nov 19, 2024
Merged

Make Geokoordinaten/Laengen+Breitegrade nullable; Add NoerdlicheBreite, OestlicheLaenge, Zone, Nordwert and Ostwert #597

merged 8 commits into from
Nov 19, 2024

Conversation

@FreddyFox892
Copy link
Contributor

No description provided.

@FreddyFox892 FreddyFox892 linked an issue Nov 18, 2024 that may be closed by this pull request
New COM/Geokoordinate fields #596
Closed
hf-kklein
hf-kklein reviewed Nov 18, 2024
View reviewed changes
Copy link
Collaborator

@hf-kklein hf-kklein left a comment

Choose a reason for hiding this comment

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

Bist du sicher , dass man (nördliche Länge+östliche Breite) und (Breiten- +und Längengrad) nicht ineinander umrechnen kann? Also dass das eigentlich verschiedene Darstellungsformen der selben werte sind ?

BO4E/COM/Geokoordinaten.cs Outdated
public decimal? Laengengrad { get; set; }

/// <summary> Gibt die Östliche Koordinate eines entsprechenden Ortes an.</summary>
[JsonProperty(PropertyName = "oestlichelaenge")]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Kannst du bitte die json property namens konsequent loser camel case machen?

BO4E/COM/Geokoordinaten.cs Outdated
public decimal? Laengengrad { get; set; }

/// <summary> Gibt die Östliche Koordinate eines entsprechenden Ortes an.</summary>
[JsonProperty(PropertyName = "oestlichelaenge")]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Kannst du bitte die json property namens konsequent loser camel case machen?

BO4E/COM/Geokoordinaten.cs Outdated
[JsonProperty(PropertyName = "noerdlichebreite")]
[JsonPropertyName("noerdlichebreite")]
[ProtoMember(6)]
public decimal? noerdlichebreite { get; set; }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Und die property Name konsequent UpperCamelCase

BO4E/COM/Geokoordinaten.cs Outdated
[JsonProperty(PropertyName = "ostwert")]
[JsonPropertyName("ostwert")]
[ProtoMember(9)]
public string? oestwert { get; set; }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ost, nicht oest.

@FreddyFox892
Copy link
Contributor Author

Guten Morgen @hf-kklein , ja man kann das alles von Längen und Breiten grad ineinander umrechnen, aber muss dann dann in das utm system umrechenen
Aber dafür bräuchte es dann ein entsprechendes Mapping, was, so wie ich das grad bei Chati mal angefragt hab, schon hinreichend komplex werden würde
Daher die frage ob es nicht eher sinnmacht die neunen 6 Werte für das UTM koordinaten system einfach zu speichern als alles in länge und breite um zu rechnen

@hf-kklein
Copy link
Collaborator

Daher die frage ob es nicht eher sinnmacht die neunen 6 Werte für das UTM koordinaten system einfach zu speichern als alles in länge und breite um zu rechnen

gut. dann machen wir das pragmatisch erstmal so. kommentierst du dann im docstring, dass man entweder die einen oder die anderen Wertepaare angeben muss und machst klar, dass wir da keine Umrechnung durchführen, auch wenn man die als user vllt erwarten würde?

Ansonsten halt noch die Property-Names gerade ziehen.

hf-kklein
hf-kklein reviewed Nov 19, 2024
View reviewed changes
BO4E/COM/Geokoordinaten.cs Outdated
/// Diese Komponente liefert die Geokoordinaten für einen Ort.
/// Hierbei kann es sich um Breiten- und Längengrad oder um UTM-Koordinaten handeln. Diese lassen sich zwar ineinander
/// umrechnen, jedoch sind die UTM-Koordinaten genauer, benötigen aber auch mehr Werte.
/// Ein Mapping ist nicht vorgesehen, da die UTM-Koordinaten in der Regel nur von Fachleuten verwendet werden.
Copy link
Collaborator

Choose a reason for hiding this comment

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

sind wir etwa keine fachleute :D?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Irgendwo Ja und irgendwo nein ^^
78b9757

@hf-kklein hf-kklein changed the title Update GeoKoordinate Make Geokoordinaten/Laengen+Breitegrade nullable; Add NoerdlicheBreite, OestlicheLaenge, Zone, Nordwert and Ostwert Nov 19, 2024
hf-kklein
hf-kklein reviewed Nov 19, 2024
View reviewed changes
hf-kklein
hf-kklein approved these changes Nov 19, 2024
View reviewed changes
Copy link
Collaborator

@hf-kklein hf-kklein left a comment

Choose a reason for hiding this comment

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

passt für mich.

@FreddyFox892 FreddyFox892 merged commit 2b3e39b into main Nov 19, 2024
6 checks passed
@FreddyFox892 FreddyFox892 deleted the FST/GeoKordinatenUpdate branch November 19, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hf-kklein hf-kklein hf-kklein approved these changes

Assignees

@FreddyFox892 FreddyFox892

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

New COM/Geokoordinate fields

3 participants

@FreddyFox892 @hf-kklein