[Feature Request] Support `MultiPoint` geojson import

[baslat]

I have a geojson export from colota. It is of type MultiPoint. When I try to import into geopulse, I get the error pasted below, seemingly because it's not LineString or Point. Would it be possible to support MultiPoint geojson imports please?

ERROR [org.git.tes.geo.imp.ser.ImportJobService] (quarkus-virtual-thread-301468) Failed to validate import job 0637f99e-3353-4c89-af57-48cbd0f405f6: Invalid geojson format: Could not resolve type id 'MultiPoint' as a subtype of `org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonGeometry`: known type ids = [LineString, Point] (for POJO property 'geometry')
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 7, column: 17] (through reference chain: org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonFeature["geometry"]): java.lang.IllegalArgumentException: Invalid geojson format: Could not resolve type id 'MultiPoint' as a subtype of `org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonGeometry`: known type ids = [LineString, Point] (for POJO property 'geometry')
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 7, column: 17] (through reference chain: org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonFeature["geometry"])
        at org.github.tess1o.geopulse.importdata.service.BaseGpsImportStrategy.validateAndDetectDataTypes(BaseGpsImportStrategy.java:67)
        at org.github.tess1o.geopulse.importdata.service.GeoJsonImportStrategy_ClientProxy.validateAndDetectDataTypes(Unknown Source)
        at org.github.tess1o.geopulse.importdata.service.ImportDataService.validateAndDetectDataTypes(ImportDataService.java:45)
        at org.github.tess1o.geopulse.importdata.service.ImportDataService_ClientProxy.validateAndDetectDataTypes(Unknown Source)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService.processAvailableJobs(ImportJobService.java:149)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService.processImportJobs(ImportJobService.java:130)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService_ClientProxy.processImportJobs(Unknown Source)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService_ScheduledInvoker_processImportJobs_0f300e332ad99ca81116475ae6c2150ed42afafe.invokeBean(Unknown Source)
        at io.quarkus.scheduler.common.runtime.DefaultInvoker.invoke(DefaultInvoker.java:25)
        at io.quarkus.scheduler.common.runtime.DelegateInvoker.invokeDelegate(DelegateInvoker.java:29)
        at io.quarkus.scheduler.common.runtime.StatusEmitterInvoker.invoke(StatusEmitterInvoker.java:35)
        at io.quarkus.scheduler.common.runtime.DelegateInvoker.invokeDelegate(DelegateInvoker.java:29)
        at io.quarkus.scheduler.common.runtime.DelegateInvoker.invokeComplete(DelegateInvoker.java:36)
        at io.quarkus.scheduler.common.runtime.OffloadingInvoker$1$1.run(OffloadingInvoker.java:45)
        at io.quarkus.virtual.threads.ContextPreservingExecutorService$ContextPreservingRunnable.run(ContextPreservingExecutorService.java:45)
        at java.base@25.0.4/java.util.concurrent.ThreadPerTaskExecutor$TaskRunner.run(ThreadPerTaskExecutor.java:291)
        at java.base@25.0.4/java.lang.Thread.runWith(Thread.java:1487)
        at java.base@25.0.4/java.lang.VirtualThread.run(VirtualThread.java:460)
        at java.base@25.0.4/java.lang.VirtualThread$VThreadContinuation$1.run(VirtualThread.java:252)
Caused by: com.fasterxml.jackson.databind.exc.InvalidTypeIdException: Could not resolve type id 'MultiPoint' as a subtype of `org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonGeometry`: known type ids = [LineString, Point] (for POJO property 'geometry')
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 7, column: 17] (through reference chain: org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonFeature["geometry"])
        at com.fasterxml.jackson.databind.DeserializationContext.invalidTypeIdException(DeserializationContext.java:2088)
        at com.fasterxml.jackson.databind.DeserializationContext.handleUnknownTypeId(DeserializationContext.java:1637)
        at com.fasterxml.jackson.databind.jsontype.impl.TypeDeserializerBase._handleUnknownTypeId(TypeDeserializerBase.java:299)
        at com.fasterxml.jackson.databind.jsontype.impl.TypeDeserializerBase._findDeserializer(TypeDeserializerBase.java:164)
        at com.fasterxml.jackson.databind.jsontype.impl.AsPropertyTypeDeserializer._deserializeTypedForId(AsPropertyTypeDeserializer.java:150)
        at com.fasterxml.jackson.databind.jsontype.impl.AsPropertyTypeDeserializer.deserializeTypedFromObject(AsPropertyTypeDeserializer.java:135)
        at com.fasterxml.jackson.databind.deser.AbstractDeserializer.deserializeWithType(AbstractDeserializer.java:262)
        at com.fasterxml.jackson.databind.deser.impl.MethodProperty.deserializeAndSet(MethodProperty.java:138)
        at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:302)
        at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:169)
        at com.fasterxml.jackson.databind.deser.DefaultDeserializationContext.readRootValue(DefaultDeserializationContext.java:342)
        at com.fasterxml.jackson.databind.ObjectMapper._readValue(ObjectMapper.java:5023)
        at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3104)
        at org.github.tess1o.geopulse.gps.integrations.geojson.StreamingGeoJsonParser.parseFeatureArray(StreamingGeoJsonParser.java:115)
        at org.github.tess1o.geopulse.gps.integrations.geojson.StreamingGeoJsonParser.parseFeatures(StreamingGeoJsonParser.java:88)
        at org.github.tess1o.geopulse.importdata.service.GeoJsonImportStrategy.validateFormatSpecificData(GeoJsonImportStrategy.java:57)
        at org.github.tess1o.geopulse.importdata.service.BaseGpsImportStrategy.validateAndDetectDataTypes(BaseGpsImportStrategy.java:47)
        ... 18 more

[tess1o]

I was not aware about MultiPoint when I did this feature. Let me check if it can be added.

[tess1o]

Yes, it will be added in the next release.

[tess1o]

Hello,

The feature has been implemented in release1.37.1 - https://github.com/tess1o/geopulse/releases/tag/v1.37.1

[baslat]

Thanks for looking into it! I tried and I get a different error (below). I also found the PR that changed Colota's export from normal geojson to MultiPoint. Attached is an export from colota I am confortable sharing, although note I had to rename it from colota.geojson to colota.geojson.json to I could upload it to this comment (I don't think that will make a difference though).

colota.geojson.json

ERROR [org.git.tes.geo.imp.ser.ImportJobService] (quarkus-virtual-thread-1117) Failed to validate import job 3c84a3eb-ef85-482c-ba5d-9109a412070b: Invalid geojson format: Cannot deserialize value of type `java.lang.Double` from Array value (token `JsonToken.START_ARRAY`)
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1401, column: 21] (through reference chain: org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonFeature["properties"]->org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonProperties["accuracy"]): java.lang.IllegalArgumentException: Invalid geojson format: Cannot deserialize value of type `java.lang.Double` from Array value (token `JsonToken.START_ARRAY`)
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1401, column: 21] (through reference chain: org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonFeature["properties"]->org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonProperties["accuracy"])
        at org.github.tess1o.geopulse.importdata.service.BaseGpsImportStrategy.validateAndDetectDataTypes(BaseGpsImportStrategy.java:67)
        at org.github.tess1o.geopulse.importdata.service.GeoJsonImportStrategy_ClientProxy.validateAndDetectDataTypes(Unknown Source)
        at org.github.tess1o.geopulse.importdata.service.ImportDataService.validateAndDetectDataTypes(ImportDataService.java:45)
        at org.github.tess1o.geopulse.importdata.service.ImportDataService_ClientProxy.validateAndDetectDataTypes(Unknown Source)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService.processAvailableJobs(ImportJobService.java:149)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService.processImportJobs(ImportJobService.java:130)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService_ClientProxy.processImportJobs(Unknown Source)
        at org.github.tess1o.geopulse.importdata.service.ImportJobService_ScheduledInvoker_processImportJobs_0f300e332ad99ca81116475ae6c2150ed42afafe.invokeBean(Unknown Source)
        at io.quarkus.scheduler.common.runtime.DefaultInvoker.invoke(DefaultInvoker.java:25)
        at io.quarkus.scheduler.common.runtime.DelegateInvoker.invokeDelegate(DelegateInvoker.java:29)
        at io.quarkus.scheduler.common.runtime.StatusEmitterInvoker.invoke(StatusEmitterInvoker.java:35)
        at io.quarkus.scheduler.common.runtime.DelegateInvoker.invokeDelegate(DelegateInvoker.java:29)
        at io.quarkus.scheduler.common.runtime.DelegateInvoker.invokeComplete(DelegateInvoker.java:36)
        at io.quarkus.scheduler.common.runtime.OffloadingInvoker$1$1.run(OffloadingInvoker.java:45)
        at io.quarkus.virtual.threads.ContextPreservingExecutorService$ContextPreservingRunnable.run(ContextPreservingExecutorService.java:45)
        at java.base@25.0.4/java.util.concurrent.ThreadPerTaskExecutor$TaskRunner.run(ThreadPerTaskExecutor.java:291)
        at java.base@25.0.4/java.lang.Thread.runWith(Thread.java:1487)
        at java.base@25.0.4/java.lang.VirtualThread.run(VirtualThread.java:460)
        at java.base@25.0.4/java.lang.VirtualThread$VThreadContinuation$1.run(VirtualThread.java:252)
Caused by: com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot deserialize value of type `java.lang.Double` from Array value (token `JsonToken.START_ARRAY`)
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1401, column: 21] (through reference chain: org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonFeature["properties"]->org.github.tess1o.geopulse.gps.integrations.geojson.model.GeoJsonProperties["accuracy"])
        at com.fasterxml.jackson.databind.DeserializationContext.reportInputMismatch(DeserializationContext.java:1814)
        at com.fasterxml.jackson.databind.DeserializationContext.handleUnexpectedToken(DeserializationContext.java:1588)
        at com.fasterxml.jackson.databind.deser.std.StdDeserializer._deserializeFromArray(StdDeserializer.java:221)
        at com.fasterxml.jackson.databind.deser.std.NumberDeserializers$DoubleDeserializer._parseDouble(NumberDeserializers.java:743)
        at com.fasterxml.jackson.databind.deser.std.NumberDeserializers$DoubleDeserializer.deserialize(NumberDeserializers.java:704)
        at com.fasterxml.jackson.databind.deser.std.NumberDeserializers$DoubleDeserializer.deserialize(NumberDeserializers.java:683)
        at com.fasterxml.jackson.databind.deser.impl.MethodProperty.deserializeAndSet(MethodProperty.java:129)
        at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:302)
        at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:169)
        at com.fasterxml.jackson.databind.deser.impl.MethodProperty.deserializeAndSet(MethodProperty.java:129)
        at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:302)
        at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:169)
        at com.fasterxml.jackson.databind.deser.DefaultDeserializationContext.readRootValue(DefaultDeserializationContext.java:342)
        at com.fasterxml.jackson.databind.ObjectMapper._readValue(ObjectMapper.java:5023)
        at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3104)
        at org.github.tess1o.geopulse.gps.integrations.geojson.StreamingGeoJsonParser.parseFeatureArray(StreamingGeoJsonParser.java:118)
        at org.github.tess1o.geopulse.gps.integrations.geojson.StreamingGeoJsonParser.parseFeatures(StreamingGeoJsonParser.java:91)
        at org.github.tess1o.geopulse.importdata.service.GeoJsonImportStrategy.validateFormatSpecificData(GeoJsonImportStrategy.java:57)
        at org.github.tess1o.geopulse.importdata.service.BaseGpsImportStrategy.validateAndDetectDataTypes(BaseGpsImportStrategy.java:47)
        ... 18 more

[tess1o]

Thanks for sharing the original GeoJson file. This part is per design specification of GeoJson MultiPoint:

{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "geometry": {
        "type": "MultiPoint",
        "coordinates": [
          [13.781120419502258, 46.48409390449524],
          [13.78147379443997, 46.4848262722412],
          [13.78142625790963, 46.484828782115414],
          [13.780906583886887, 46.48478652994054],
          [13.780506737654873, 46.484691555010116],
          [13.780433016474241, 46.484565571071904],
          [13.779867165, 46.48433303],
          [13.780078452540957, 46.484158941657164]
        ]
      },

However the part from properties looks like Colota specific implementation. They provide an array of properties (like accuracy, battery) and each element of that array corresponds to a GPS coordinate.

I will adjust my parser to support this format.