Skip to content

Adapt Sql Coordinates #294

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 59 commits into from
Apr 22, 2025
Merged

Adapt Sql Coordinates #294

merged 59 commits into from
Apr 22, 2025

Conversation

danielfeismann
Copy link
Member

resolves #287

@danielfeismann
Use WKBElement Geography for Coordinates
c0ef9c3
@danielfeismann
Merge branch 'main' into df(#287-WKBElement-Geography-for-Coordinates
126c341 
# Conflicts:
#	poetry.lock
@danielfeismann
poetry
f1bcf2c
@danielfeismann
rollback get lat and lon of coordinate
b212581
@danielfeismann
make WeatherValue.time noon optional
e36e8f3
@danielfeismann
adapt lon and lat
44cb898
@danielfeismann
adapt coordinate sql model and adapt tests
ec6c396
@danielfeismann
remove unnecessary things, fmt etc.
0f4f0e0
@danielfeismann
docker compose compatibility
10b47d0
@danielfeismann
skip tests if docker is required but not available
956b564
@danielfeismann
adapt ci
76a1268
@danielfeismann
ci
1daedc9
@danielfeismann
start docker before
5b69a4b
@danielfeismann
start docker
233e6ae
@danielfeismann
try docker desktop
7fb9dcd
@danielfeismann
some more ci tests
7b19549
@danielfeismann
even more ci try
1115862
@danielfeismann
even more ci tries
e1eca76
@danielfeismann
even more more ci tries
6c624fb
@danielfeismann
next try
c51c1f8
@danielfeismann
try another approach
0655873
@danielfeismann
Merge branch 'main' into df(#287-WKBElement-Geography-for-Coordinates
da008e8 
# Conflicts:
#	poetry.lock
@danielfeismann
fmt
7ee0955
@danielfeismann
remove pytest marks
ba22087
@danielfeismann
Merge branch 'main' into df(#287-WKBElement-Geography-for-Coordinates
35cd753 
# Conflicts:
#	poetry.lock
@danielfeismann
poetry
3e9bf5b
@danielfeismann
test additional marks
43297ba
@danielfeismann
remove some marks
5df6fe9
@danielfeismann
rollback fixture conftest
5b8c9a3
@danielfeismann
at mark and try check os again
27e4e22
@danielfeismann danielfeismann mentioned this pull request Mar 24, 2025
Closed
@danielfeismann danielfeismann self-assigned this Mar 24, 2025
@danielfeismann danielfeismann added the code quality Improve code quality label Mar 24, 2025
@danielfeismann danielfeismann marked this pull request as ready for review March 24, 2025 15:14
@danielfeismann danielfeismann requested a review from t-ober March 24, 2025 15:14
@danielfeismann danielfeismann marked this pull request as draft March 24, 2025 16:05
@danielfeismann danielfeismann marked this pull request as ready for review March 25, 2025 17:01
t-ober
t-ober approved these changes Apr 22, 2025
View reviewed changes
Copy link
Contributor

@t-ober t-ober left a comment

Choose a reason for hiding this comment

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

LGTM

@t-ober t-ober merged commit 457541e into main Apr 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t-ober t-ober t-ober approved these changes

Assignees

@danielfeismann danielfeismann

Labels
code quality Improve code quality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Adapt Sql Coordinates
2 participants
@danielfeismann @t-ober