EinEinfach / CaSSAndRA

Cascaded sunray server and rover application
MIT License
29 stars 17 forks source link

Pathplanner: zu viele Irokesen #143

Open marvin78 opened 4 months ago

marvin78 commented 4 months ago

Ich weiß, dass du nicht mehr gerne am Pathplanner fuckelst aber das hier stört mich und dürfte, aus meiner Sicht, auch so nicht sein:

image

Da fehlt zu viel und es bleibt hier immer eine ganze Menge stehen: So müsste das aussehen (lieber zu viel, als zu wenig):

image

Wenn du Zeit hast, kannst du dir das ja ggf. nochmal ansehen. Ich habe das auf meiner Map in vielen Bereichen. Du sagst immer, es interessiert nicht, wie die Mähbahnen innerhalb des Perimeters sind. Das stimmt in solchen Fällen eben nur dann, wenn man sicher sein kann, dass alles abgefahren wird und nichts stehen bleibt (und eben der Perimeter nicht ständig für Dinge misbraucht wird, für die er nicht da ist). Hier konnte man in der Sunray App immer schön nachkorrigieren. Da das in Cassandra nicht möglich ist, muss der Pathplanner besser werden (aus meiner Sicht).

max951 commented 4 months ago

+1 ist bei mir leider auch so

EinEinfach commented 4 months ago

Ja, muss ich mir genauer anschauen

themanfrommoon commented 4 months ago

+1 Me, too

Es stört ja nicht, wenn der Rover 2x über eine Stelle drüber fährt, aber keinmal ist einmal zu wenig.

EinEinfach commented 4 months ago

Das Thema ist echt knifflig, habe tatsächlich noch keine simple Lösung für das Problem.

Als Übergangslösung schlage ich euch vor distance to border = border laps im Pathplanner auszuwählen

marvin78 commented 4 months ago

Du meinst bei distance to border=2 auch border laps=2. So ist es hier eingestellt. Das klappt also nicht., Man könnte 3 border Laps nehmen aber bei unserem komplexen Garten dauert das Mähen einer Fläche gut doppelt so lange. Distance to border zu verringern ergibt bei Alfred keinen Sinn. Dann fährt er gegen alles und jeden.

Es gibt ja auch solche Stellen. Da kann man schwierig mit Workarounds arbeiten. In Sunray habe ich mir das einfach zurechtgezogen:

image

EinEinfach commented 4 months ago

Den Rechteck links kriege ich denke ich weg, den rechten Rechteck bekommst du mit distance to border = border laps weg. Bei dem mittleren Rechteck bin ich mir nicht sicher was da passiert

marvin78 commented 4 months ago

Was meinst du mit distance to border = border laps? Ich habe distance to border = 2 und border laps = 2 eingestellt und das ist das Ergebnis. Hier auch:

image

EinEinfach commented 4 months ago

Ah sorry, ich meine border laps = distance to border + 1

marvin78 commented 4 months ago

Ja. Das ist ein Problem. Die Idee hatte ich auch. Unser Garten ist jedoch zu komplex dafür. Die Border-Laps sind das, was ohnehin schon prozentual (im Verhältnis zur Fahrdauer) am längsten dauert und die meiste Energie aus dem Akku zieht. Noch eine Runde mehr pro Task ist hier nicht sinnvoll. Ich habe keine Ahnung davon, wie komplex das ist, das im Pathplanner zu lösen. Ich würde mir bloß wünschen, dass es nicht aus dem Fokus verschwindet. :)

Manchmal kann man es mit anderen Winkeln lösen. Aber auch das kommt nicht der Effizienzu zu Gute.

EinEinfach commented 4 months ago

Wie gesagt, erstmal als workaround gedacht. Ich habe das Problem identifiziert, aber habe noch keine 100%e Lösung, ein Teil der Löcher kann ich schon stopfen, aber bei weitem nicht alle.

Es ist jetzt als issue dokumentiert, und geht nicht verloren

marvin78 commented 4 months ago

Gut, Danke. Wir haben einen Garten mit vielen Ecken und Kanten und ich habe aktuell 20 Stellen, an denen große Irokesen stehen bleiben (und noch viele kleine Ecken). Wenn das Gras nicht so schnell wächst, wie aktuell, dann sieht man es nicht ganz so schlimm (und man kann es eben lösen, in dem man täglich den Winkel wechselt) aber aktuell sieht das echt besch*** aus, da das Gras hier mehrere cm am Tag wächst und es auch nicht überall sinnvoll ist, den Winkel zu wechseln.

Allerdings muss ich auch sagen, dass Alfred aktuell mit leicht modifizierter 318 Firmware und vor allem danke Cassandra, einen echt guten Job macht. Von daher ist das alles jammern auf hohem Niveau. Das ist ggf. auch das Problem: Setzt man den Standard hoch, werden auch die Ansprüche immer höher ;)

EinEinfach commented 4 months ago

Ich kann schon mal das Update pushen, wo zumindest ein Teil der Irokesen verschwinden soll

marvin78 commented 4 months ago

Kannst dir auch Zeit lassen. Ich kann das ggf. sowieso erst heute Abend testen. Alfred mäht gerade.

marvin78 commented 4 months ago

Jetzt sieht das an der einen Stelle so aus:

image

Das darf da so nicht sein. Da fährt er mehrfach in die Ecke mit Mauer und Stufe. Insgesamt erscheint da viel mehr Durcheinander.

EinEinfach commented 4 months ago

Gleiche Einstellungen wie vorhin?

marvin78 commented 4 months ago

Jap. Gleiche Einstellungen, wie vorhin. Interessant ist, dass es aber offenbar gar nicht an der neuen Version liegt. Ich hatte wieder zurück gerüstet auf den vorherigen Commit und er macht den gleichen Quatsch (auch mit derm ganzen Durcheinander hin und her). Nur der Original Task (jap, gleiche Einstellungen - der existiert schon lange). sieht ordentlich aus (eben mit den Haken an den ein oder anderen Stellen).