Geopackage support #180

Closed
spyhunter99 opened this Issue Oct 27, 2015 · 8 comments

Comments

Projects
None yet
2 participants
@spyhunter99
Collaborator

spyhunter99 commented Oct 27, 2015

add support for geopackage storage formats
http://www.geopackage.org/spec/

@spyhunter99

This comment has been minimized.

Show comment
Hide comment
@spyhunter99

spyhunter99 Oct 28, 2015

Collaborator

found an android friendly provider that provides tiles via google maps tile provider
https://github.com/ngageoint/geopackage-android

Collaborator

spyhunter99 commented Oct 28, 2015

found an android friendly provider that provides tiles via google maps tile provider
https://github.com/ngageoint/geopackage-android

@spyhunter99

This comment has been minimized.

Show comment
Hide comment
@spyhunter99

spyhunter99 Oct 30, 2015

Collaborator

quick update, i was able to get this to work, however the performance was terrible when using it synchronously. when loading the tiles asynchronously, tiles just don't show up and i'm not entirely sure why.

Collaborator

spyhunter99 commented Oct 30, 2015

quick update, i was able to get this to work, however the performance was terrible when using it synchronously. when loading the tiles asynchronously, tiles just don't show up and i'm not entirely sure why.

@spyhunter99

This comment has been minimized.

Show comment
Hide comment
@spyhunter99

spyhunter99 Dec 8, 2015

Collaborator

I wrote a tool which enables you to preconvert geopack files into osmdroid sqlite databases. This more or less meets the objective. It can be found here: https://github.com/spyhunter99/geopkg4osmdroid

Collaborator

spyhunter99 commented Dec 8, 2015

I wrote a tool which enables you to preconvert geopack files into osmdroid sqlite databases. This more or less meets the objective. It can be found here: https://github.com/spyhunter99/geopkg4osmdroid

@spyhunter99 spyhunter99 closed this Dec 8, 2015

@legrostdg

This comment has been minimized.

Show comment
Hide comment
@legrostdg

legrostdg Jan 4, 2017

It would be cool to support geopackage directly in osmdroid: it is a standard and is supported by gdal, which permits conversion from a lot of formats. Maybe the performance of geopackage-android has improved during the last year?

It would be cool to support geopackage directly in osmdroid: it is a standard and is supported by gdal, which permits conversion from a lot of formats. Maybe the performance of geopackage-android has improved during the last year?

@spyhunter99

This comment has been minimized.

Show comment
Hide comment
@spyhunter99

spyhunter99 Jan 4, 2017

Collaborator
Collaborator

spyhunter99 commented Jan 4, 2017

@spyhunter99 spyhunter99 reopened this Jan 4, 2017

@spyhunter99

This comment has been minimized.

Show comment
Hide comment
@spyhunter99

spyhunter99 Jan 5, 2017

Collaborator

@legrostdg you sir, are in luck. I still have the on device code. I'll see if it can be updated and the performance issues fixed. Unless you want to give it a whirl? I'm a bit busy at the moment on other tasks

Collaborator

spyhunter99 commented Jan 5, 2017

@legrostdg you sir, are in luck. I still have the on device code. I'll see if it can be updated and the performance issues fixed. Unless you want to give it a whirl? I'm a bit busy at the moment on other tasks

@spyhunter99

This comment has been minimized.

Show comment
Hide comment
@spyhunter99

spyhunter99 Jan 5, 2017

Collaborator

i think the reason why it wasn't originally merged has the version at the time had a lot of dependencies on google libraries, which i don't like and it sort makes understanding the licensing, complicated.

i also wanted to just included it with the samples osmdroid app, however the minsdk is 8 for osmdroid. the NGA stuff enforces a minsdk of 14. Enforcing that for the sample app would significantly reduce our test coverage unfortunately but it may be worth it. Maybe we can split up the sample app somehow to have a version for travis with a low minsdk but publish to the play store a version with everything in it.

ngageoint/geopackage-android#18

Collaborator

spyhunter99 commented Jan 5, 2017

i think the reason why it wasn't originally merged has the version at the time had a lot of dependencies on google libraries, which i don't like and it sort makes understanding the licensing, complicated.

i also wanted to just included it with the samples osmdroid app, however the minsdk is 8 for osmdroid. the NGA stuff enforces a minsdk of 14. Enforcing that for the sample app would significantly reduce our test coverage unfortunately but it may be worth it. Maybe we can split up the sample app somehow to have a version for travis with a low minsdk but publish to the play store a version with everything in it.

ngageoint/geopackage-android#18

@legrostdg

This comment has been minimized.

Show comment
Hide comment
@legrostdg

legrostdg Jan 5, 2017

@spyhunter99 unfortunately I reported this as the user, not as a coder, as I don't speak java... I would be happy to assist if it could be coded in haskell or python!

I think this worth increasing the minsdk but you are the one who decides :-).

Thanks a lot for taking time on this issue!

@spyhunter99 unfortunately I reported this as the user, not as a coder, as I don't speak java... I would be happy to assist if it could be coded in haskell or python!

I think this worth increasing the minsdk but you are the one who decides :-).

Thanks a lot for taking time on this issue!

spyhunter99 added a commit that referenced this issue Jan 5, 2017

spyhunter99 added a commit that referenced this issue Jan 6, 2017

spyhunter99 added a commit that referenced this issue Jan 13, 2017

@spyhunter99 spyhunter99 referenced this issue in ngageoint/geopackage-android Jan 13, 2017

Closed

OsmDroid variant #18

@spyhunter99 spyhunter99 added this to the v5.6.3 milestone Jan 13, 2017

spyhunter99 added a commit that referenced this issue Jan 13, 2017

spyhunter99 added a commit that referenced this issue Jan 14, 2017

#180 auto zoom to bounds of the selected tile source for both geopack…
…kage and mapsforge.


#527 found a simplier solution

spyhunter99 added a commit that referenced this issue Jan 14, 2017

spyhunter99 added a commit that referenced this issue Jan 14, 2017

spyhunter99 added a commit that referenced this issue Jan 15, 2017

#527 #180 attempt at fixing the build. root cause was related to the…
… forge sample, memory leaks, unclosed database connections for sqlcache. Memory dump is related to harmony
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment