Skip to content

feat: add project for street level imagery #980

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
ofr1tz merged 56 commits into from
Dec 12, 2024
Merged

feat: add project for street level imagery #980

ofr1tz merged 56 commits into from
Dec 12, 2024

Conversation

@Gigaszi
Copy link
Collaborator

@Gigaszi Gigaszi commented Nov 18, 2024 •
edited by ofr1tz
Loading

Includes

  • Add new project type street for street level imagery
  • Add functionality to extract Images and Metadata from Mapillary
  • Add functionality to spatial filter images

Post commands

sudo -u postgres psql -p 5432 -U mapswipe_workers -h 127.0.0.1 -d mapswipe < ./postgres/scripts/v2_to_v3/08_change_geom_type_for_tasks.sql

ofr1tz and others added 27 commits October 29, 2024 15:04
@ofr1tz
feat(street): add street project draft fixture
6779347
@Gigaszi
feat: add new street project
40cdc66
@Gigaszi
feat: add tutorial to street project
33bc5ec
@Gigaszi
refactor: move process_mapillary.py to utils
ec59ee0
@Gigaszi
feat: add functions to process mapillary data
78f28df
@Gigaszi
refactor: removed unused imports and classes
3a7ac21
@Gigaszi
feat: add function to convert input geojson to polygon
8e5c55c
@Gigaszi
tests: add unittests for process_mapillary
dd4e220
@Gigaszi
fix: wrong indention and patch in test_process_mapillary
b274af3
@Gigaszi
feat: add filtering and handling of mapillary response
7ec92ee
@Gigaszi
tests: add tests for filtering and handling of mapillary response
97e552c
@Gigaszi
refactor: use mapillary incredentials in variables
53dfda5
@Gigaszi
refactor: reformat file process_mapillary.py
481030c
@Gigaszi
feat: add geometry to StreetTask and refactoring
1187561
@Gigaszi
tests(street): use simpler geometry and add filter parameters to test…
0bef2e0 
…ing json
@Gigaszi
refactor: tests for mapillary processing and street project
789b027
@Gigaszi
fix: add missing parameter
c1f1a79
@Gigaszi
tests(street): add integration test for street project
a12a803
@Gigaszi
tests(street): add street.json for testing of street project
2fb3bc7
@Gigaszi
fix: test after refactoring
14ff237
@Gigaszi
feat: add functions for spatial sampling and tests for it
80ecf28
@Gigaszi
feat: use filtering in street project
984d821
@Gigaszi
feat: add spatial sampling to street project
4ab528d
@Gigaszi
feat: add size restriction for too many images
f7fc5f3
@Gigaszi
breaking: change geom column in postgres to allow all geometry types
f11f75e
@Gigaszi
feat: save location of image as geometry
98f82c9
@Gigaszi
feat: use spatial filter on downloaded images
6a1d15e
Hagellach37
Hagellach37 reviewed Nov 18, 2024
View reviewed changes
Gigaszi and others added 19 commits November 28, 2024 15:51
@Gigaszi
fix: tests for removed failing rows
9503a7d
@ofr1tz
style: fix flake8 errors and isort
c0016e2
@ofr1tz
style: isort
2f3d523
@ofr1tz
build: add requirements
6ad174f
@ofr1tz
build: add dummy mapillary key to github workflow
315d239
@ofr1tz
fix: use os.getenv instead of os.environ
2b88cf7
@ofr1tz
test: rename class and correct comments
24c7eec
@Gigaszi
fix: remove left overs of failed rows
7f46e80
@Gigaszi
Merge branch 'feature/street' of https://github.com/mapswipe/python-m…
652e885 
…apswipe-workers into feature/street
@Gigaszi
fix: unittests for tile download
bfa0818
@Gigaszi
fix: add condition to raise valueerror if no features are found in aoi
816b1e6
@ofr1tz
fix: use mapillary api key secret in github workflow
f287b3a
@ofr1tz
fix: use square brackets with os.environ
020178d
@ofr1tz
fix: add mapillary key to docker compose
dcaab3d
@ofr1tz
fix: allow tasks with point geom in postgres
8128488
@Gigaszi
fix: change geometry type in tasks table in initdb
74064f8
@Gigaszi
Merge branch 'feature/street' of https://github.com/mapswipe/python-m…
f712340 
…apswipe-workers into feature/street
@Gigaszi
fix: change to all geometries in tasks table in setup db
729a4be
@Gigaszi
fix: use lower case in setup db
994f398
@ofr1tz ofr1tz added dependencies Pull requests that update a dependency file python Pull requests that update Python code labels Dec 3, 2024
thenav56
thenav56 reviewed Dec 11, 2024
View reviewed changes
@ofr1tz ofr1tz merged commit 6bc241a into dev Dec 12, 2024
1 check passed
@ofr1tz ofr1tz deleted the feature/street branch December 12, 2024 10:15
@ofr1tz ofr1tz mentioned this pull request Feb 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Hagellach37 Hagellach37 Hagellach37 left review comments

@thenav56 thenav56 thenav56 approved these changes

@ofr1tz ofr1tz ofr1tz approved these changes

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file python Pull requests that update Python code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Gigaszi @Hagellach37 @thenav56 @ofr1tz