addon streetsmart

Issue #797 resolved
Stolz created an issue

Hallo zusammen,

in actionsStreetSmart.js benötigen wir timeTravelVisible: true da wir mehrere Befahrungsjahrgänge vorhalten.

Zudem sollte der Marker auf den gefundenen Aufnahmepunkt gesetzt werden (bei RECORDING_CLICK und VIEW_CHANGE) . Aktuell bleibt er an der Klickposition stehen. Diese Position entspricht nicht dem angezeigten Cyclorama im Panoramaviewer.

Danke&VG

Dietmar Stolz

Comments (8)

  1. Dennis Sen

    Hallo Herr Stolz,

    würden Sie eine passende Portalkonfiguration zu diesem AddOn in dieses Ticket laden? Das würde die Bearbeitung deutlich erleichtern.

    Viele Grüße

  2. Stolz reporter

    Hallo Herr Sen,

    in Ergänzung an die letzte Mail sieht die config wie folgt aus:

    "streetSmart": {
    _"name": "360° Panorama",_  
    _"icon": "bi-camera-fill",_  
    _"streetsmartAPIVersion": "22.2",_  
    _"reactVersion": "16.13.0",_  
    _"styleId": "streetsmartMarker",_  
    _"initialWidth": 800,_  
    _**"cycloLayerID": "51",**_  
    _**"timeTravelVisible": true,**_  
    _**"toggle3DCursor": false,**_  
    _**"toggleAddressesVisible": false**_
    

    }

    In der rest-services.json sind keine Anpassungen erforderlich.

    VG

    Dietmar Stolz

  3. Dennis Sen

    Hallo Herr Stolz,

    damit komme ich leider nicht aus. Wenn ich dies unter portal/master ergänze, bekomme ich vom Tool die Fehlermeldung “Die Panorama-Ansicht kann nicht angezeigt werden. Details dazu finden Sie in der Entwickler-Konsole.“, wo wiederum steht “Cannot start Streetsmart-View. No service in rest-services found for serviceId streetsmart“.

    Eine Konfiguration finde ich in meinem Mailpostfach (wegen “in Ergänzung an die letzte Mail“) leider nicht. Oder wäre eine Konfiguration “im freien Internet” auch gar nicht lauffähig, weil hier lokale/versteckte Dienste benötigt werden?

    Falls doch, wäre es am einfachsten, wenn an Issues schlicht der komplette Portalordner angehängt wird. Auch scheinbar triviale Konfigurationsparameter wie Extent, Center, usw. müssen sonst immer erstmal eingestellt werden.

    Falls nicht, müsste ich “blind” programmieren. Dann könnte ich einen Branch herstellen, der ihre Programmiervorschläge konfigurierbar umsetzt, und würde dann bzgl. des Tests, ob das so funktioniert, wieder auf Sie zukommen.

    Viele Grüße

  4. Stolz reporter

    das hat ja bisher Frau Inka Dudek von der LGV bearbeitet.

    Werde mal versuchen den EMail-Verkehr an Sie weiter zu leiten.

  5. Dennis Sen

    Hallo Herr Stolz,

    vielen Dank für die Zulieferung. Ich habe die gewünschten Flags zur Konfiguration verbaut.

    • Für toggleAddressesVisible gab es in der Code-Vorlage keine Stelle. Ich habe das Flag unmittelbar nach toggle3DCursor eingesetzt, laut Doku von StreetSmart passt es dort hin. Ich konnte jedoch keinen Effekt feststellen, wenn das Flag auf true steht – wissen Sie dazu etwas?
    • Den übrigen Code habe ich wie gegeben übernommen, nur auf die Konfiguration zurückgreifend, und die Vorschläge zur cycloLayerID wie gegeben eingebaut.

    Einen PR zum Anschauen habe ich unter https://bitbucket.org/geowerkstatt-hamburg/addons/pull-requests/398/issue-797-addon-street-smart-parameters gestellt.

    Dabei ist mir aufgefallen, dass Sie ja eigentlich “fast schon fertig” waren und offenbar selbst auch programmieren. Wenn Sie mögen, können Sie auch gern direkt am PR mitarbeiten und auf dem zugehörigen Branch commiten. Die Rechte sollte der LGV vergeben können, falls es an diesen scheitert.

    Viele Grüße

  6. Log in to comment