mapbox / mapbox/mapbox-java

Implementation difference between geojson js vs geojson java

Ouverte
#1,077 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

GeoJson Turf
Langage dominant
Java
Étoiles
438
Forks
117
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

This issue is raised from PR #1075 .

  • In the js implementation of geojson, the properties value of a Feature can be empty, but in the java implementation, an empty properties field is ignored:

https://github.com/mapbox/mapbox-java/blob/2130ea708c72ef8b98aeb02102554d7dc32606a0/services-geojson/src/main/java/com/mapbox/geojson/Feature.java#L280

  • When converting js version of geojson to the java version, the precision of coordinates has been reduced, i.e. from 102.227783203125 to 102.2277832. This issue was introduced in #938 . It seems the precision is trimed due to performance consideration.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par Feature.java au niveau de la gestion liée de properties et examinez PR #1075 et issue #938 pour les changements de comportement. Comparez la sortie Java avec le comportement GeoJSON de JavaScript pour les properties vides et la précision des coordonnées ; le travail est terminé lorsque les deux différences sont corrigées et couvertes par des tests pertinents.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
data
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.