neuroevolution-ai / NeuroEvolution-CTRNN_new

MIT License

3 stars 3 forks source link

Add elman #49

Closed Dranero closed 3 years ago

Dranero commented 3 years ago

Hallo, ich habe folgendes gemacht:

Netze: IBrain

Verschiebung der Input-Vorbereitung in IBrain ILayeredBrain
Abstrakte Brain Klasse
verallgemeinert schichten basierte Netze Elman Netz
basiert auf ILayerdBrain Gru Netz
basiert auf ILayerdBrain LayeredLSTM
basiert auf ILayerdBrain

Brain Visualisierung

Anpassung der IBrain Klasse durch Erweiterung von Methoden
Implementierung von Elman Visualisierung

Bug Fixes

CTRNN Bug fix, output Matrix war falsch rum (diagonal gespiegelt). Das zog sich auch durch den ganzen Visualizer

Tests muss ich noch nachliefern, es wäre aber nett, wenn ihr schon mal über das Vorhandene drüber schauen könntet

Dranero commented 3 years ago

47 ist dabei

23 ist teilweise dabei

pdeubel commented 3 years ago

Ok blöd gelaufen ich wusste nicht dass du auch an dem normalisieren der Brain Inputs gearbeitet hast, da bei #47 keine Antwort mehr kam. Ich hab an was anderem gearbeitet und das hat sich mit dem normalisieren überschnitten -> d.h. ich hab das Refactoring der Normalisierungen auch implementiert. Kannst dich das nächste mal gerne selbst in dem Issue assignen dann weiß man dass du daran arbeitest 👍🏽 😄 Btw: Von mir aus kannst auch gern direkt im Repo arbeiten also eigene Branches pushen und auf denen arbeiten ohne das Repo auf deinen Account zu forken - wie es dir besser passt

Ich würde jetzt einfach deinen PR reviewen und meinen als Draft machen. Wenn deiner gemerged ist dann werd ich mich um die Code Konflikte kümmern

bjuergens commented 3 years ago

denk daran gelegentlich den master branch reinzumergen. Das geht mit pycharm sehr gut. Und je öfter man es macht, desto einfacher wird es. Wenn die branches erstmal weitgenug divergiert sind, ist es sehr sehr nervig die wieder zu mergen