Farmmanager

[Lace123]

Hi,
wollte ein paar Änderungen am Farmmanager machen, habe gesehen, dass gerade aktiv daran gearbeitet wird. Daher liste ich hier mal meine Ideen auf.

1. Mir ist aufgefallen, dass mit den letzten Änderungen das C-farmen nicht mehr funktioniert. Er zieht 1 vom Klick-counter ab, schickt aber nichts ab und bringt auch keinen Fehler. (Bei alten Versionen ohne die letzten Änderungen geht es)
2. Es wäre nützlich eine zusätzliche Leiste zu haben, die die minimale Distanz zu den Dörfern angibt. So könnte man die nahe liegenden Farmen mit höherer Priorität abfarmen.
3. R-Tab einen Hacken einfügen, der es erlaubt zwischen absoluten und prozentualen Resten zu tauschen. Grund: Wenn man Dörfer farmt, die weiter weg sind, laufen die Einheiten lange zurück, bis man sie wieder losschicken kann. --> 30% der Lkav einbehalten, damit man wieder alle Dörfer abfarmen kann. (Hat auch noch ein paar andere effizienz implikationen, die ich hier nicht alle ausführen möchte)
4. Erlauben, alte Berichte einzulesen. Es funktioniert nicht, dass Workbench Berichte, die älter als der Farmeintrag sind der Farm zuzuweisen. D.h. wenn man in der Datenbank einen Bericht von gestern hat, aber die Farm heute neu hinzugefügt hat, weißt Workbench den Bericht nicht zu, denkt es hat keine Berichte für die Farm.
   Der Vorteil wäre es, dass man alle Farminformationen(auch BB-Dörfer) von einer alten Version in eine neuere importieren kann, also ohne großen aufwand die DS Version wechseln kann.
   Alternativ könnte man auch erlauben, dass die Funktion nach Farmen in Berichten suchen auch die Barbarendörfer beachtet. Derzeit muss man nach Barbarendörfern suchen, die werden hinzugefügt und dadurch kann man keine älteren Informationen zuweisen, sondern muss einen neuen Bericht einlesen.
5. Wenn der Angriff Verluste aufweist, die Farm sperren/oder den Wall schätzen (letzteres ist wohl zu aufwendig. Die könnten mit einem gelben Schloss gesperrt werden, dass nur Angriffe mit Rammen erlaubt, wenn derWall Stufe 2 oder höher, oder der Bericht Verluste hatte.
   Das würde es erlauben, die Farmen einmal zuspähen und dann ohne Späher weiterzufamen.
6. Das Farmen mit gewissen Gruppen erluben. Denke die Idee kam schnonmal auf.
7. Ein Feld bei C-Farmen einfügen, dass eine manuelle Obergrenze hinzufügt. Z.b. 2h, so dass WB maximal so viele Einheiten schickt, dass einen Produktion von 2 h abgefarmt wird.
   Grund: So kann man einen für sich sinnvollen Betrag einfügen und WB schickt keine 100 Lkav auf ein kleines Dorf, nur weil es lange nicht mehr abgefarmt wurde. Die Annahme, dass sich Rohstoffe ansammeln ist absolut unrealistisch. Die wichtigste Änderung, die es sogar effizienter machen kann WB zu benutzen anstelle von build-in Farmmanager. Habe ich bereits getestet.
8. Mehrfachangriffe erlauben. Alos, wenn man einen Angriff losschickt, die theoretische Menge direkt um den Betrag, der geplündert würde erniedrigen und es ermöglichen auf die gleiche Farm nocheinmal zu laufen. Mit dem minimal farm angaben, kann man verhindern, dass man auf Dörfer läuft, bei denen sich das Farmen noch nicht lohnt.
   Grund: Bei weit entfernten farmen kommt man nur alle 4 Stunden vorbei. Aber Leute werden da aktiv farmen. Wenn man alle 2 h einen Angriff darauf schickt, der die Produktion von den 2h abholt, kann man seine farm effizienz enorm steigern.
9. Das Rammen mitschicken funktioniert nur bei c-farmen, aber nicht bei A/B. Wäre aber nützlich, weil man dann bei C nur Lkav einträgt und in A z.b. 100 Axt, damit man wenig Verluste beim Wall hat und die dann mit Rammen losschicken kann.
10. Einen Indikator einfügen, der einen darauf hinweist, ob das Dorf ein Spieler oder ein BAba Dorf ist. Eventuell ein Button, der Babas ausblendet. Habe ab und zu einenn Spieler hinzugefügt, und den wieder zufinden, wenn er entfernt werden muss, weil er wieder aktiv wurde ist ganz schön aufwendig.

Sorry für die vielen Rechtschreibfehler, habe dass alles einfach schnell runtergetippt.

[extremeCrazyCoder]

Erstmal da sind einige gute Ideen dabei und einiges sollte schon implementiert sein.
Aktiv dran rum basteln tue ich grad nicht also wenn du willst kannst du gern mit dem coden beginnen.
Ich arbeite eher immer so Schwung weise am Ende vom Tage commite ich allerdings alle Changes. Ich würde dich daher einfach bitten, dass du dir die aktuelle Version aus meinem repro holst und dann deine Changes entweder zu mir pusht oder ins Main repro damit keine Konflikte entstehen.

1. Das ist mir noch nicht aufgefallen, nachdem ich den aber selten bis nie nutze ist das auch nicht verwunderlich. Wenn du willst kann ich mir das mal ansehen oder du siehst es dir an :)
2. Bei A, B, C kann man sowas ja schon angeben oder meinst du die sollte in der Tabelle sein?
3. Nachdem die Truppen jetzt vom neuen Standardangriff Parser eingelesen werden solltest du hier die gleichen Möglichkeiten haben wie bei diesen auch. (gilt für A, B, C, R) also ist eine Prozent Angabe kein Problem (man beachte allerdings "%" Angaben bei Standardangriffen #49 was noch nicht in der Offiziellen version drinnen ist)
4. das wäre irgendwie gegen denn sinn des Features, allerdings kannst du neu gefundene Farmen ja mit einem extrem alten wert initialisieren, was das einlesen von irgendeinem Bericht am Anfang ermöglichen würde.
   Weiters könntest du den Export / Import der Farm-Manager settings hinzufügen
5. Das Feature würde ich irgendwie zum ein / ausschalten machen sonst könnte das andere User stören.
   Das Wall schätzen sollte nicht so schwer sein (solange der inc nicht komplett rot ist), da müsstest du einfach das DSSim Modul nutzen (Wie das geht kannst du dir beim SOS analyzer ansehen). Die ganzen infos wie z.B.: Luck, Truppenstärken sind ja vorhanden man muss sie nur nutzen :=)
6. Ich hatte da leider keine Zeit dafür wenn du lust hast kannst du ja Farmmanager Funktion "kürzeste Distanz" #35 gleich mit machen :D
7. Die Idee der Erfolgsquote ging ja schon in diese Richtung, so eine Implementierung könnte das Farmen allerdings wirklich verbessern.
8. Wird schwer zu implementieren sein, weil jetzt alles darauf ausgelegt ist, dass immer nur 1x Truppen gesendet werden. Du kannst dich aber daran versuchen
9. Warum nicht (klarerweise mit Checkbox)
10. Kann ich verstehen dass das verwirrend sein kann, du solltest aber schauen, dass die Tabelle nicht zu groß wird

[Lace123]

Kenne mich hier noch nicht so aus.^^ Muss erst noch herausfinden, wie das alles funktioniert mit dem zusammenfügen usw. bin kein Programmiergnie. :-P

Habe aber gerade sowieso nichts zum zusammenfügen, weil ich die Änderungen in einer alten Version gemacht habe, da ich den Fehler mit c-farmen in der neuen Version nicht gefunden habe. (Finde den ganzen Code etwas unübersichtlich-- Unübersichtlichkeit liegt aber nicht an den neuen Änderungen)

Zu 9. (Rammen farmen) habe den Fehler gefunden. Die Checkboxen gibt es ja schon. Aber die rammen wurden im Troophelper bei getTroopsForCarriage gesetzt, das wird aber nur für c-farmen verwendet. Man müsste es in farminformation bei farmfarm einfügen, damit es bei A/B auch funktioniert.

Zu 2. (Min Disatanz beim farmen) Verstehe nicht, welches Feature du damit meinst^^. Sehe nur die Einstellung zu max und min Laufzeit. Hier nochmal ausführlicher, auf was ich hinauswollte:
Die Liste der Farmen wird von oben nach unten abgearbeitet. Dabei steht sie Sortiert nach dem Koordinaten system von oben links nach unten rechts. Wenn man genau die hälfte der Truppe hat, um das angegebene Gebiet abzufarmen, dann schickt er alle Truppen auf die linke Seite, obwohl es nähere farmen auf der rechten Seite gibt.
Das kann man umgehen, indem man die max Laufzeit so einstellt, dass es genau passt, aber dafür muss man selber einschätzen wie weit man laufen kann. Daher der Einfall mit der min Distanz, dass die Tabelle nach min Distanz abgearbeitet wird.

11. Neue Idee: Eine Katta-möglichkeit einstellen. Man kann Barbarendörfer runterkattern, damit sie die Rohstoffgebäude weiter ausbauen.
    Grund: Wenn die Kaserne auf 0 ist und HG unter 3, dann ist die Chance, dass die Babas Rohstoffgebäude ausbauen bei 50%. (Außerdem wird der Wall seltener ausgebaut, was die Verluste beim farmen verringert)
    Außerdem gibt es ein Ausbaulimit bei 500 Punkten. Durch das Kattan, werden unnötige Gebäude abgerissen und macht die Farmen auf diese Art lukrativer. Darüberhinaus hat man die meiste Zeit sowieso nichts mit den Kattas zu tun, während die Lkavs farmen.
    Wäre sogar komplementär zum ingame-Farmmanager.
    Umsetzung: Man muss die Mmöglichkeit haben, dass man die Gebäudeart, die man kattan will angibt, damit beim Übertragen immer auf das selbe gebäude geschoßen wird. Ansonsten ist das unmöglich manuell immer einzustellen, auf was man schießt. Zumdem sollte man ein Text einfügen, der erklärt, dass bevor man abschickt den Standard angriff auf das gewünschte gebäude zu setzten.
    Außer es gibt eine Möglichkeit das Kattaziel gleich mitzuübertragen (glaube ich nicht).
    Der Rest wäre synonym zu Rammen mitschicken.

[extremeCrazyCoder]

zu 2: ok
zu 9: Ich würde das nicht Fehler nennen, aber ich habs mir nicht genug angeschaut um sagen zu können ob das einer ist
zu 11:
Das ist nicht möglich. Ich kann zwar sagen, dass katas bei einem Angriff dabei sein sollen ich kann aber nicht festlegen auf welches Gebäude sie gehen (sonst ist die Idee nett)

sieh dich einfach mal ein bisschen im Code um. Ich arbeite derzeit an der Befehlsübersicht (intern) & dem Taktikplaner.

[Lace123]

C-farmen gefixed // correct speed by used units (not necessary as they are sorted by runtime!?
// ... but won't hurt anyway)
speed = Math.max(speed, units.getSpeed());
Hat den Bug ausgelöst xD wie ironisch.

Rammen können auch mit A/B tab geschickt werden
Einen Fehler behoben, bei dem auch weniger als die min. Truppen geschickt wurden.

Habe einen Katta-tab eingefügt, bei dem man ein Ziel auswählen kann, dass dann automatisch runtergekattert wird.

Würde dann noch die Erfoglsquote berücksichtigen bei A/B entfernen. Da werden fixe Truppen losgeschickt, dass bringt bei denen gar nichts. Als nächstes schaue ich mir auch noch das mit der Effizienz Obergrenze und farmen mit Gruppen an.

[Lace123]

Ich brauche mal eine zweite Meinung. MinHaul scheint ein Relikt zu sein. Es wird nur in der Berechnung für C verwendet, nicht aber für A/B.
Allerdings würde es andersherum mehr Sinn machen. Bei C gibt man sowieso eine Untergrenze bei den Einheiten an. Bei C ist dann entweder die Farmgrenze oder die Einheitengrenze überflüssig (Je nachdem, welche höher ist wirkt)
Aktion: Würde es ändern, dass minHaul bei A/B die Farmen beschränkt und würde es bei C durch die Farmobergrenze ersetzen.
Was sind die Meinungen? Übersehe ich was?

[extremeCrazyCoder]

Also so weit ich das sehe hat min Haul die Funktion, dass es definiert ob noch ressourcen da waren im BB-Dorf oder nicht (anhand der gespähten Ressourcen) über den sinn lässt sich streiten. Du kannst das mit dem beschränken der Farmen gern so implementieren.

Wie das mit dem Kata tab funktioniert musst du mir nochmal erklären

[Lace123]

Man kann ingame ein Standardziel für Katapulte festlegen. Das legt man ingame dann zum Beispiel auf das Hauptgebäude und wählt in der WB das Hauptgebäude aus. Dann schickt die WB auf farmen genau so viele Katapulte, dass das Gebäude auf 0 bzw. 1 zerstört wird. Dadurch erhöht sich die Profitabilität vom farmen. Wird eher selten gemacht, weil der Aufwand relativ groß ist. Mit WB ist das aber ein eher geringer Aufwand.

[extremeCrazyCoder]

ah ok :) das kann funktionieren

[Lace123]

Zwei Kleinigkeiten gibt es noch, die ich optimieren könnte..

1. Das Kattan auf kleine Farmen beschränken (<600P), um keine großen Babas unnötig klein zuschießen
2. Die Katapulte so splitten, dass immer eine Stufe zerstört wird. Das wissen einige nicht, aber so kann man mehr Schaden verursachen, als alle Katas auf einmal zu schicken. (Könnte mit einer Checkbox implementiert werden?)
3. Dass Kataangriffe nicht als Farmangriff zählen, so dass der Status nicht geändert wird, und man währenddessen die Farm weiter normal mit Lkavs abfarmen kann.

Ich habe mir gerade eben das Feature "Farmen in Berichten suchen" angeschaut. Es sollte auch Barbarendörfer erkennen. Macht es aber nicht. Habe einen logger nach jedem if/else Schritt eingebaut. Die Barbarendörfer landen aber nirgends. Scheint ein Problem mit den Reports von Barbarendörfern zu sein. Irgendwas stimmt auf jedenfall nicht.
Außerdem erkennt er keine Berichte, die bereits als Farm im Farmmanager eingetragen waren, auch wenn man sie aus der Farmliste löscht. Keine Ahnung, ob die Fehler miteinander verbunden sind.
Das Problematische ist, dass man so schwerlich die WB Version wechseln kann, weil man alle Berichte nochmal einlesen muss, was ein unnötig großer Zeitaufwand ist.
Vielleicht willst du dir mal anschauen, was da los ist. Denke ich bleibe erst mal beim Farmmanager.

[extremeCrazyCoder]

Vor der nächsten release werde ich mir alles was du noch an Bugs über lässt ansehen :)

[Lace123]

Hi,
Ich bin inzwischen an zwei Stellen an Probleme gestoßen mit TroopAmountDynamic. Die Funktion getAmountForUnit gibt entweder das zurück, was der min Betrag in den Konfigurationen ist, oder das was im Dorf steht, je nachdem was weniger ist. Die Konfigurationen sind aber eine Untergrenze.

1. Schickt er immer Farmtruppen ab, selbst wenn nur 1 Axtkämpder da ist.

2. Bei allowedFarmUnits sind immer welche vorhanden, selbst wenn nur eine einzelne Farmeinheit im Dorf steht. Was an mehreren Stellen zu Schwierigkeiten führt.

Ist das so gewollt und nicht ganz richtig implementiert im Farmmanager? Code im Farmmanager oder bei TroopAmountDynamic abändern?

Außerdem würde ich gerne das schicken von Kattas und Rammen von den Farmangriffen separieren, so dass man Dörfer auf die Kattas laufen immer noch abfarmen kann. Bei Rammen war das bisher ja kein Problem, weil man ansonsten Verluste hätte, aber Katas sind zum optimieren der Farmen da. Das lange sperren der Farm, obwohl verlustfreies farmen möglich ist, ist da eher hinderlich. Würde die Zeile "letztes Farmergebnis" dafür entfernen.**
**(zeigt an, ob die Farm beim letzten mal überschüßige Rohstoffe im Lager hatte. Überflüßig, weil C-farmen automatisch die Ressis mit einrechnet und die Erfolgchance ein wesentlich weitsichtigeres Maß zum feststellen der Profitabilität der Farmen ist).

Wenn ich schon dabei bin einen Roman zu schreiben vielleicht noch kurz deine Meinung: Bin gerade dabei einen Algorithmus zu bauen, der eine sinnvolle Farmreihenfolge festlegt.
Ist da die Median Distanz von Farmen zu den Farmdörfern oder die minimale Distanz zu den Farmdörfern besser. Habe schon eine eigene Meinung, würde aber gerne eine unvoreingenommene zweite lesen.

[extremeCrazyCoder]

Der erste Punkt ist so gewollt und sollte nicht in der TroopAmountDynamic Klasse geändert werden, weil wir sonst an anderen stellen Probleme bekommen.
Der 2. Punkt ist eine Funktion des Farmmanagers, daher kannst du den natürlich problemlos verändern. Ich muss jetzt aber zugeben, dass ich den sinn von der Funktion nicht wirklich verstanden habe^^ (warum die bei A und B verwendet wird) die schaut ja nur nach, ob mehr als 0 Truppen da sind und gibt dann an bei welchen Einheiten das zutrifft. (Bei C verstehe ich den Sinn ja noch da die LZ davon abhängt)
Du könntest ja z.B. in TroopAmountDynamic eine Funktion einbauen mit der du solche Elemente vergleichen kannst.

Das mit den Katas klingt sinnvoll, ich würde aber sagen, nachdem du die Funktion selber eingebaut hast weißt du vermutlich besser als ich was sie machen sollte und musst mich daher nicht Fragen ob das ok ist :)

zum Algorithmus: Ist jetzt etwas schwierig für mich da eine Aussage zu machen, weil ich meist auf No-Farm Welten spiele ich werde es aber dennoch mal Probieren (kannst dir dann ja noch eine 3. oder 4. Meinung einholen :D )

Ich würde sagen, dass man am Effizientesten Farmt, wenn der median aller Distanzen (von den Farmtrupps die verschickt werden) am kleinsten ist. Meine Idee dahinter ist, dass es nicht so schlimm ist wenn ein Trupp relativ weit reisen muss und dafür alle anderen kürzer laufen können.
Ich würde dir aber dazu raten, dass du es testest, weil ich jetzt nur nach Gefühl geantwortet habe und wenig praktische Erfahrungen damit hab.

Ach ja was mir noch aufgefallen ist: Du hast beim Klonen des Repros einen Fehler gemacht. https://github.com/Lace123/ds-workbench ist kein Fork von Torridity's oder meinem Repro, daher wirst du deine Änderungen nicht per Pull-Request an uns weitergeben können. Hast du eh oben rechts auf Fork gedrückt?

P.S. Jetzt hab ich dir auch einen Roman zurückgeschrieben ich hoffe das stört dich nicht^^

[Lace123]

Hast du was an
FarmInformation
DSWorkbenchFarmManager
FarmManager
TroopHelper
geändert?
Wenn ja ist das wohl u aufwendig alles zu übertragen. Dann sind das wohl private Änderungen für mich^^

[extremeCrazyCoder]

nein hab ich nicht, wobei es meistens auch kein Problem wäre wenn ich was geändert hätte.
Ich kann meine Changes derzeit allerdings nicht comiten, weil die noch nicht stabil laufen :D