Transform coordinates to decimal format #11

Merged

g merged 5 commits from w2/photovoter_backend:decimal-coordinates into main

w2 commented

Owner

Convert coordinates from rational64u ("42/1, 18/1, 2914/100") EXIF uses to represent degrees, minutes, and seconds of a point to decimal format (42.30809)

General formula is
dec_degree = degreesNumerator / degreesDenominator
+ minutesNumerator / minutesDenominator / 60
+ secondsNumerator / secondsDenominator / 3600

Resources

https://en.wikipedia.org/wiki/Geographic_coordinate_conversion
https://gis.stackexchange.com/questions/136925/how-to-parse-exif-gps-information-to-lat-lng-decimal-numbers

w2 added 1 commit

d18786b000 add: transform coordinates to decimal form

w2 commented

Poster

Owner

fixes #7

w2 commented

Poster

Owner

@g
Should we take the reference too and if it's [W]est or [S]outh turn coordinates negative right here?

g commented

Owner

@g
Should we take the reference too and if it's [W]est or [S]outh turn coordinates negative right here?

Sounds good, let's do it

w2 added 1 commit

6bd14777a3 add: take into account reference cardinal direction

g reviewed

util/import_photos.py

													
					    # 5 sum up the result

					    # 6 convert to decimal

					    # 7 round to 5th decimal point

					    dec_coordinates = round(

g commented

Owner

R-style pipes would be functional here. Sadly Python doesn't have one. Yet comments make it clear.

g commented

Owner

https://git.iamonlyherefortheicecream.ml/DIWHY/photovoter_backend/issues/12#issue-12
needs a fix

g commented

Owner

https://git.iamonlyherefortheicecream.ml/DIWHY/photovoter_backend/issues/13#issue-14
is worth fixing as well

w2 added 1 commit

2816355886 del: redundant db columns

w2 commented

Poster

Owner

fixes #12

w2 added 1 commit

be6a1f4eb7 del: redundant db columns in update requests

w2 commented

Poster

Owner

fixes #12 better

I should probably start testing my commits, huh? You never know.

w2 added 1 commit

b601b106b4 ref: change coordinates db datatype

w2 commented

Poster

Owner

fixes #13

g approved these changes

g merged commit 4b5a20a109 into main 4 years ago

g referenced this issue from a commit

Merge pull request 'Transform coordinates to decimal format' (#11) from w2/photovoter_backend:decimal-coordinates into main

g commented

Owner

A little map with initial results

193 KiB

👍 1

w2 deleted branch decimal-coordinates

Reviewers

g approved these changes

The pull request has been merged as 4b5a20a109.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: DIWHY/photovoter_backend#11