Giter Club home page Giter Club logo

Comments (15)

remuslazar avatar remuslazar commented on July 24, 2024

@secuder also die Implementierung ist bei GE und OCM so:

  1. es werden immer nur die Steckertypen gefiltert, welche auch in den Settings hinterlegt sind. Heißt also, da kommen nur Ladestationen, wo mindestens eins von den Typen vorhanden ist.

  2. wenn man "Fastcharging" Filter Option aktiv ist, dann wird folgendes gemacht:

    • Wenn die "AC Fastcharging" Option aktiv ist, so werden zusätzlich Stationen gefiltert, wo die maximale Leistung über die eingestellte Schwelle ist (20 oder 43kW in den Settings)
    • Wenn die o.g. Option nicht aktiv ist, so werden zusätzlich nur die Steckertypen gefiltert, welche vom Typ DC Fastcharging getagged sind (aktuell nur Tesla, CCS und Chademo)

Code hier:

        var connectionTypeIDs = PlugFinderUserDefaults.OCMSelectedConnectionTypes

        if filter.onlyFastCharge {
            params["minpowerkw"] = Int(PlugFinderUserDefaults.fastChargingMinPowerKW)
            if !PlugFinderUserDefaults.acFastChargingCapable {
                connectionTypeIDs = connectionTypeIDs.filter { dcPlugTypes.contains($0) }
            }
        }
        (..)
        params["connectiontypeid"] = connectionTypeIDs.map{String($0)}.joined(separator: ",")

Wie du siehst, ist da kein Rocket Science, was gemacht wird. Das macht genau das, was ich oben beschrieben habe (bzw. sollte es das tun ;)

Wenn da nicht das gewünschte raus kommt, so muss man den Fehler bei OCM ggf. suchen.

Auch kann man mit der OCM API selber experimentieren, man braucht nur einen Browser dafür:

https://openchargemap.org/site/develop/api

Ich werde vermutlich keine Zeit finden, um mich damit intensiver zu befassen, bin aber gerne bereit, einen anderen Algorithmus zu implementieren, wenn man hier andere Vorschläge hat, die nachweislich besser sind.

Lasse das also noch offen, hat aber (bei mir) erst mal nicht höchste Priorität.. Ich selber nutze nur GE zur Zeit und dort konnte ich keine Anomalien feststellen

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

Was heißt denn zusätzlich gefiltert? Durch einen Filter können es per Definition des Filters nur weniger Treffer werden.

from evplugfinderpublic.

remuslazar avatar remuslazar commented on July 24, 2024

genau das meinte ich doch damit, da wird zusätzlich ein weiteres Filter-Kriterium angewandt. Das ist alles mit logisch UND verknüpft. "Filter A" UND "Filter B" eben. Ist das nicht verständlich? Ich kann es nur noch auf Englisch probieren:)

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

Die genannten 43kW AC Stationen werden nur im Szenario 2. oben gefunden. Das ist kein sinnvolles, vom User erwartetes Ergebnis. Die müssen, weil Typ 2 Stecker, in jedem der Szenarios gefunden werden, sonst stimmt da was nicht.

Auch nicht erwartet werden kann, dass im Szenario 3. Ladepunkte unter 22kW (Schnellladung ist in den Einstellungen als >=22kW definiert), die noch dazu DC Stecker sind (die ja nicht ausgewählt sind), angezeigt werden.

from evplugfinderpublic.

remuslazar avatar remuslazar commented on July 24, 2024

@secuder aber klar, im 1) wird nur nach kompatiblen Steckertypen gefiltert, da spielen die AC und Fastcharging Optionen doch gar keine Rolle... Wenn der User in den Settings Typ 2 ausgewählt hat, so werden die Stationen auch angezeigt. Die Logik habe ich doch versucht zu beschreiben, wenn das noch nicht verständlich ist, sorry:)

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

In meinen Augen gehst du bei den USE-CASES von der DC + langsam-AC Voraussetzung im Hinterkopf aus, die sich aber in der Beschreibung der Einstellungen und Filter nicht wiederfindet und damit für AC Fahrer falsche Ergebnisse (siehe oben) bringt.

OCM hat offenbar keine Priorität, aber das beschriebe Verhalten ist nachhaltig falsch.

Ich erarbeite gern einen Vorschlag (oder zumindest mit), aber das hat für mich nur Sinn, wenn du meiner Argumentation folgen magst. womöglich ist es auch nicht mit einer Umbenennung der Einstellungen getan.

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

(Verstehe ich nicht, warum ein 43kW AC Typ 2 in Szenario 1. nicht angezeigt wird - alle Voraussetzungen sind doch erfüllt!?)
(Gern auch auf Englisch)

from evplugfinderpublic.

remuslazar avatar remuslazar commented on July 24, 2024

@secuder also nochmals, damit wir die Sache abschließen können:

wenn die ACFastCharging Einstellung AKTIV ist (Zoe Fahrer), so wird im Prinzip nur nach Stationen, wo die Leistung über die eingestellte schwelle ist, gefiltert, wenn Fastcharging an ist.

Aus dem o.g. Code geht ja das hervor. if !PlugFinderUserDefaults.acFastChargingCapable {, also IF NOT, nur wenn NICHT ACFastCharging wird dann der 2. Filter noch angewandt.

Und wenn das immer noch nicht das gewünschte raus bringt, so ist der Fehler in der OCM API zu suchen. Das meinte ich doch damit.. Die Logik ist so in Ordnung.

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

Hm, tut mir leid, ich habe vor ziemlich vielen Jahren mal Pascal gelernt, das kann ich so nicht beurteilen.
Aus User Sicht sind die Ergebnisse nicht schlüssig, nicht nur wegen der 43kW AC, sondern auch wegen der dann plötzlich angezeigten 20kW DC, die durch die Steckerfilterung und durch die Bedingung >=22kW eigentlich ausgeschlossen sein sollten.

Zu den 43kW Typ 2: Hast du berücksichtigt, dass es in OCM offenbar zwei Typ 2-Arten (ID 25 und ID 1036) gibt? Wenn man nach Type 2 Anschlüssen filtert, dann müssen beide zugelassen sein.

from evplugfinderpublic.

remuslazar avatar remuslazar commented on July 24, 2024

@secuder nein, das ist dann wohl der Bug. Typ 1036 wird nicht berücksichtigt. Ist das Typ2 wo das Kabel dran ist, oder? i.d.R. ist ja Typ2 nur Buchse, Kabel muss der Fahrer dabei haben.

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

Genau. ID 25 ist offenbar eine Typ 2 Dose (Kabel mitbringen), ID 1036 ist ein Typ 2 Stecker (einfach direkt ans Auto anschliessen).

FormalName": "IEC 62196-2 Type 2",
"IsDiscontinued": false,
"IsObsolete": false,
"ID": 25,
"Title": "Mennekes (Type 2)"

"FormalName": null,
"IsDiscontinued": false,
"IsObsolete": false,
"ID": 1036,
"Title": "Mennekes (Type 2, Tethered Connector) "

(Quelle: http://api.openchargemap.io/v2/referencedata/)

from evplugfinderpublic.

remuslazar avatar remuslazar commented on July 24, 2024

@secuder hast du mich falsch verstanden, ich meinte, das ist der Bug in der App. Das werde ich sicher im nächsten Build korrigieren!

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

Bei dem DC Ladepunkt OCM-45012 verstehe ich nun immerhin zum Teil, warum er trotz <22kW angezeigt wird:
img_3926

Der ist Level 3 getaggt trotz einer Ladeleistung von nur 20kW.

Unklar ist mir aber trotzdem, warum der uberhaupt angezeigt wird, denn da gibt es keinen Typ 2 Stecker (und andere Stecker hatte ich in den Einstellungen nicht ausgewählt)

from evplugfinderpublic.

secuder avatar secuder commented on July 24, 2024

Habs gesehen und meinen Comment schon korrigiert :-) aber zu spät :-D

from evplugfinderpublic.

remuslazar avatar remuslazar commented on July 24, 2024

ist kein Beinbruch! Gute n8! :)

from evplugfinderpublic.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.