Low level GeoJson implementation
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
Piste de recherche
Commencez par examiner l’implémentation actuelle dans services-geojson, puis comparez la structure de ses modèles avec RFC 7946 et les conseils de performance Android référencés. Définissez comment la réécriture ou l’alternative mesurera l’empreinte mémoire et les performances pour environ 1 000 features ; le travail sera considéré comme terminé lorsque les modèles GeoJSON atteindront les objectifs de scalabilité demandés sans allocations ni abstractions inutiles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
This is a request for either a rewrite or alternative implementation of the current GeoJson module found here. We need a high performance implementation that has a minimal memory footprint. The reasoning behind that is that geojson models should be scalable (eg. downstream we query the map for 1000 of features, the path currently taken to create those and the intermediate objects created for doing that isn't optimal atm).
Couple of thoughts to improve this:
- use primitive instead of objects where possible
- no need for AutoValue, implementation is based on a specification that doesn't change (often)
- if geojson models are immutable, use arrays instead of List
- limit method invocations & don't allocate memory if you can avoid it
- don't over abstract concepts if they can be expressed more simple
More information on performance tips here
RFC of GeoJSON can be found here.
cc @mapbox/android
- Langage dominant
- Java
- Étoiles
- 438
- Forks
- 117
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de mapbox/mapbox-java
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
mapbox/mapbox-java#1640 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
mapbox/mapbox-java#1635 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
mapbox/mapbox-java#1619 ·
-
Mapbox-java incompatible with gson 2.13.0 and up - call to internal GSON classes that were renamed Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 48/100
mapbox/mapbox-java#1614 · 2 commentaires · 1 réaction ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
mapbox/mapbox-java#1612 · 1 réaction ·
Toutes les issues de mapbox/mapbox-java
Issues similaires
-
bug untriaged
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
opensearch-project/ml-commons#5094 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
-
emitter:client:csharp feature
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
affects/8.10 affects/8.9 component/clients kind/bug likelihood/mid severity/mid
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Two open-case totals on one screen: the Programs tile says 15,858 and the nav badge says 15,868 Ouvertebug frontend maui-pilot
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100