Skip to content
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

[Fixes #246,247] Introducing concept of Assets and 3D tiles in GeoNode #254

Merged
merged 49 commits into from
Jul 3, 2024
Merged

[Fixes #246,247] Introducing concept of Assets and 3D tiles in GeoNode #254

merged 49 commits into from
Jul 3, 2024

Conversation

mattiagiupponi
Copy link
Contributor

No description provided.

etj and others added 30 commits April 24, 2024 16:10
@etj
Changes to support Assets
3256464
@mattiagiupponi
Merge branch 'master' of github.com:GeoNode/geonode-importer into geo…
872aacb 
…node_12124_assets
@mattiagiupponi
Fix asset build
e8713cb
@mattiagiupponi
Fix asset build
e9d8ced
@mattiagiupponi
Merge branch 'master' of github.com:GeoNode/geonode-importer into geo…
da28282 
…node_12124_assets
@mattiagiupponi
Remove original file at upload end
98833fa
@mattiagiupponi
Merge with master and fix tests
bcf1741
@mattiagiupponi
Merge branch 'master' of github.com:GeoNode/geonode-importer into geo…
09bf865 
…node_12124_assets
@mattiagiupponi
Let importer create the asset
caa7200
@mattiagiupponi
Let importer create the asset
437346a
@etj
Fixes
50d2a08
@mattiagiupponi
Add test coverage for asset-importer
c2b4f2c
@mattiagiupponi
Merge branch 'assets_data_retriever' of github.com:GeoNode/geonode-im…
8ec0eeb 
…porter into assets_data_retriever
@mattiagiupponi
Add test coverage for asset-importer
749f085
@mattiagiupponi
Add test coverage for asset-importer
4e37f9d
@mattiagiupponi
Add test coverage for asset-importer
e1344a9
@mattiagiupponi
Fix test coverage
c9c5251
@mattiagiupponi
Fix test coverage
b2de28a
@mattiagiupponi
Fix test coverage
647241b
@mattiagiupponi
First implementation of 3dtiles handler
d916572
@mattiagiupponi
First implementation of 3dtiles handler
a2c42e6
@mattiagiupponi
First implementation of 3dtiles handler
a61aac7
@mattiagiupponi
First implementation of 3dtiles handler
b6180fc
@mattiagiupponi
Add test coverage for 3dtiles
b12ddfa
@mattiagiupponi
test coverage
d097dfb
@mattiagiupponi
test coverage
bede3ea
@mattiagiupponi
test coverage
1becb43
@mattiagiupponi
set resource_type as dataset
686690d
@mattiagiupponi
Fix asset location for 3dtiles handler
950ce93
@mattiagiupponi
Fix runtest
dc214d2
mattiagiupponi and others added 19 commits June 6, 2024 12:22
@mattiagiupponi
Fix runtest
c7641f0
@mattiagiupponi
use region for cacluate bbox in 3dtiles
b611c3d
@mattiagiupponi
Add test coverage and refactor with black
d4ce079
@mattiagiupponi
Add test coverage and refactor with black
6bb056a
@mattiagiupponi
Add test coverage and refactor with black
2661110
@mattiagiupponi
fix path stem
8dbc004
@mattiagiupponi
Assets data retriever (#244)
83cd097 
* [Fixes #242] CRS parsing is not correctly handled for CSV files
* Let importer create the asset
* Add test coverage for asset-importer
@mattiagiupponi
[Fixes #247] Introducing assets in geonode-importer
39aa21b
@mattiagiupponi
Merge with ISSUE_247
41c70df
@mattiagiupponi
[Fixes #247] Introducing assets in geonode-importer
253581b
@mattiagiupponi
[Fixes #246] Introducing 3dtiles in geonode-importer
095fc89
@mattiagiupponi
[Fixes #247] Remove typos
b8536c1
@mattiagiupponi
Merge branch 'ISSUE_247' of github.com:GeoNode/geonode-importer into …
dbcc738 
…ISSUE_246
@mattiagiupponi
[Fixes #246] Add function for calcultating bbox from boundingVolume
cf51cdd
@mattiagiupponi
Add ID for base handlers
5c6596a
@mattiagiupponi
Merge branch 'ISSUE_247' of github.com:GeoNode/geonode-importer into …
959834c 
…ISSUE_246
@mattiagiupponi
[Fixes #246] Fix transform for 3dtiles handling
103342e
@mattiagiupponi
[Fixes #246] Fix setting of bbox
3a8f213
@Gpetrak
[Fixes #246] Implementation of boundingBox.sphere parsing for 3D Tile…
de0efef 
…s - Issue #79 (#253)

* boundingBox.sphere parser for 3D Tiles
* cleaning the code in tests for the sphere_to_wgs84 function
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 3, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  importer
  celery_tasks.py
  orchestrator.py 70-71
  importer/api
  views.py 188
  importer/handlers
  base.py 64, 71
  utils.py
  importer/handlers/common
  vector.py
  importer/handlers/geojson
  handler.py 76-77
  importer/handlers/tiles3d
  exceptions.py
  handler.py 139, 148-199, 223, 251-254, 275, 297
  utils.py 139, 218
Project Total  

This report was generated by python-coverage-comment-action

@mattiagiupponi mattiagiupponi changed the title Issue 246 [Fixes #246,247] Introducing concept of Assets and 3D tiles in GeoNode Jul 3, 2024
This was linked to issues Jul 3, 2024
Create handler for the 3Dtiles #246
Closed
Introducing assets in geonode importer #247
Closed
@mattiagiupponi mattiagiupponi marked this pull request as ready for review July 3, 2024 14:52
@mattiagiupponi mattiagiupponi requested a review from giohappy July 3, 2024 14:52
@giohappy giohappy merged commit 12fc9f4 into master Jul 3, 2024
2 checks passed
@mattiagiupponi mattiagiupponi deleted the ISSUE_246 branch August 1, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@giohappy giohappy giohappy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Introducing assets in geonode importer Create handler for the 3Dtiles
4 participants
@mattiagiupponi @giohappy @etj @Gpetrak