Illegal argument in isolate message

smiesnyrobert commented 1 year ago

Hi, can you please help me with this error?

Iam using this versions:

routing_client_dart: ^0.4.0 flutter_osm_plugin: ^0.53.3

i have this code:

void getRoute() {

    final manager = MyOSRMManager.custom(server: "myCustomServerUrl");

    List<LngLat> waypoints = [
      LngLat(
          lng: widget.userDetail.order!.waypointList[0].longitude,
          lat: widget.userDetail.order!.waypointList[0].latitude),
      LngLat(
          lng: widget.userDetail.order!.waypointList[1].longitude,
          lat: widget.userDetail.order!.waypointList[1].latitude),
    ];

    manager
        .getRoad(
          waypoints: waypoints,
          geometries: Geometries.polyline,
          steps: true,
        )
        .then((road) => {
              controller.drawRoadManually(
                  road.polyline!
                      .map((e) => GeoPoint(latitude: e.lat, longitude: e.lng))
                      .toList(),
                  RoadOption(
                    roadWidth: 10,
                    roadColor: Colors.blue,
                    zoomInto: true,
                  ))
            });
  }

and i get this error

E/flutter (25057): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Invalid argument(s): Illegal argument in isolate message: (object extends NativeWrapper - Library:'dart:io' Class: _SecureFilterImpl@13069316)
E/flutter (25057): #0      Isolate._spawnFunction (dart:isolate-patch/isolate_patch.dart:399:25)
E/flutter (25057): #1      Isolate.spawn (dart:isolate-patch/isolate_patch.dart:379:7)
E/flutter (25057): #2      Isolate.run (dart:isolate:238:15)
E/flutter (25057): #3      compute (package:flutter/src/foundation/_isolates_io.dart:17:18)
E/flutter (25057): #4      OSRMManager.getRoad (package:routing_client_dart/src/osrm_manager.dart:71:21)
E/flutter (25057): <asynchronous suspension>

Can u please help? Thank you

liodali commented 1 year ago

are you calling getRoute in another isolate ?

smiesnyrobert commented 1 year ago

I have just normal button in widget and call it after button click

glonaldo70 commented 1 year ago

same problem

liodali commented 1 year ago

upgrade to new version for osrm dart client 0.4.2 contain some fixes

glonaldo70 commented 1 year ago

it what i'm using 0.4.2 . have i to add also flutter_osm_plugin in my dependencies?

liodali commented 1 year ago

if you're use another map no need for flutter osm plugin can provide me more code to understand more the issue

glonaldo70 commented 1 year ago

DetailSite( _selectedSite!, onSite: () async { Position mPosition = await _determinePosition(); print( " MY LOCATION IS: ${mPosition.longitude} + ${mPosition.latitude}"); List waypoints = [ LngLat( lng: mPosition.longitude, lat: mPosition.latitude), LngLat( lng: _selectedSite!.longitude!, lat: _selectedSite!.latitude!), ]; _road = await manager.getRoad( waypoints: waypoints, geometries: Geometries.polyline, steps: true, language: Languages.en, roadType: RoadType.car, overview: Overview.full); setState(() {}); }, )

liodali commented 1 year ago

this callback onSite is called inside isolate or just future ?

glonaldo70 commented 1 year ago

Just future void

smiesnyrobert commented 1 year ago

Ive created clean new project and here is the whole code. I get same error.

import 'package:flutter/material.dart';
import 'package:routing_client_dart/routing_client_dart.dart';
import 'package:testprojekt/MyOSRMManager.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(),
      floatingActionButton: FloatingActionButton(
        onPressed: getRoute,
        child: const Icon(Icons.add),
      ),
    );
  }

  void getRoute() {
    final manager = MyOSRMManager.custom(server: "xxxx");

    List<LngLat> waypoints = [
      LngLat(lng: 13.388860, lat: 52.517037),
      LngLat(lng: 13.397634, lat: 52.529407),
      LngLat(lng: 13.428555, lat: 52.523219),
    ];
    manager
        .getRoad(
          waypoints: waypoints,
          geometries: Geometries.polyline,
          steps: true,
        )
        .then((road) => {
              print("success"),
            });
  }
}

pubspec.yml

name: testprojekt
description: A new Flutter project.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

version: 1.0.0+1

environment:
  sdk: '>=2.19.6 <3.0.0'

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.2
  routing_client_dart: ^0.4.2

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_lints: ^2.0.0

flutter:
  uses-material-design: true

liodali commented 1 year ago

if you can that you intercept the response usinf dev tools and share it with me if it possible

smiesnyrobert commented 1 year ago

which response u mean? http response from server?

liodali commented 1 year ago

yeah

smiesnyrobert commented 1 year ago

liodali commented 1 year ago

Oh sorry for that I found the issue ,it's my bad , I tried to improve current implementation and I forget that compute accept only top function and I moved parseRoad to mixin I will fix the issue and I will publish new version sorry again and thnx for your report just note I test in web and its working fine xD thats why I doubt at beginning

liodali commented 1 year ago

new version has being published 0.4.3

smiesnyrobert commented 1 year ago

thanks for fix, good job

liodali / OSM-Routing-Client-Dart

Illegal argument in isolate message #16