matthiasos/venusos-modbustcp

venusos-modbustcp's Issues

Register should be displayed in the table

Range in ModbusOperation

Die Range der Rückgabe muss bei der ModbusOperation berücksichtigt werden.
Also das Ergebnis an die range anpassen:
Wenn range -32000 bis 32000 und der Wert 5000 ist, muss noch 32000 drauf addiert werden oder?

Ip adresse variable über Eingabe fenster

GUI für Backend

einfach GUI für Backend erstellen

• jgoodies einbinden
• einfache Oberfläche:
  -- Droppdown für Operation
  -- read/write
  -- JTextArea für Rückgabe

Weboberfläche! Gut und einfach zu bedienen

alle funktionalitäten mit programmierbausteinen
modular

Special Unit Case for units

The value is correct, but after that the whole unit gets displayed i.e. "Absorption 0=Absorption;1=...."
Maybe show it in paratheses

Rename wolfsbau to ModbusTCP

Search Fields for Operations and Devices

Search Fields for Operations and Devices, where you can type and the Combobox will be filtered on the fly

When no Excel list is provided the programm should search for a list in the same folder

When no Excel list is provided the programm should search for a list in the same folder
With Regex so the version doesnt matter.
If multible files are found, use the one with the highest version number

GUI Einheiten anzeigen

Uint INT Double und Scale errechnen die Range von einzelne Registern

Delete doesnt work when multible rows are the same

When there are Several rows with the same data, the delete doesnt function correctly.
Sometimes the "-" is missing and the last element cant be deleted.

Probably is because all row are the same and it cant be distinguished between them

Modbus server abfrage delay einbauen?!

Capacity hat als Einheit Degrees celsius

Capacity hat als Einheit Degrees celsius, das hört sich nicht richtig an.
Register 318

DBusUnit als Java Objekt umsetzen

DBusUnit nicht mehr als Enum sondern als Java Objekt umsetzen.
Dann per Vererbung immer die toString Methode überschreiben.
Dann braucht man keine extra Behandlung für alle flag 0,1,2 etc Werte.

Remove Http Get Request to Inverter

Active Input Einheit hat für zwei Werte "AC Input"

Active Input Einheit bei Register 29 "Active Input" hat für 0 und 1 jeweils "AC Input" als Wert.
Soll eines davon vllt eher DC oder so sein? Oder Output?

GUI favoriten verschiebbar machen

ModbusResultIntTest als Parametrized test machen

https://github.com/junit-team/junit4/wiki/Parameterized-tests

Weiter Modbus Operationen umsetzen

Modbus Operationen mit folgenden Registern sollen umgesetzt werden laut der Excel Liste:
D:\ownCloud\DE_WO_Wolfsbau Speicher\3_Technical_offer\Modbus PLC\CCGX-Modbus-TCP-register-list-2.12 OOs.xlsx

• 3, 6, 9, 12, 21, 22, 23, 26, 27, 29, 30, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 56 -> #22
• 259, 261, 262, 263, 264, 266, 318, 319, 774, 775, 776, 777, 784, 789, 790 #28
• 800, 806, 807, 808, 809, 810, 811, 812, 813, 817, 818, 819, 820, 821, 822, 840, 841, 842, 843, 844, 850, 851, 855, 860, 866, 2600, 2601, 2602, 2603, 2604, 2605, 2606, 2607, 2608, 2609, 2616, 2617, 2618, 2619, 2620, 2621, 3304 -> #29

Make GUI look better

Use other look and feel or a additional library

Loading a different Excel File doesnt work

java -jar ModbusTCP.jar CCGX-Modbus-TCP-register-list-3.00.xlsx Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: Datei [CCGX-Modbus-TCP-register-list-3.00.xlsx] nicht gefunden!

String bei ModbusRead

Datentyp in der Excel zeigt an wieviele Register man auslesen muss:
string[7]

Dann einfach die Werte in Character umwandeln

Order device list

Order device list by unit id

Progressbar when Reading from Modbus

Progressbar when Reading from Modbus

Choose a License

Korrekte Umrechnung von Registerwert auf Messwert bei Operationen mit vorzeichenrelevanten Ergebnissen.

Wie ist die korrekte Umrechnung von Registerwert auf Messwert.
Bei Operationen die kein Vorzeichen haben muss man nur den Wert mittels des Skalierungsfaktors anpassen.

Bei Operationen mit Vorzeichen (zB GRID_L1_POWER: Register 2600) wird es etwas komplizierter.
Der mM nach korrekte Lösungsweg ist:
Messwert = Registerwert - MAX_Modbus (=65535) oder Werte in der Range der Operation (also Min = -32768, Max = 32767 und das betragsweise addiert ist auch 65535) - 1.
Also zB bei GRID_L1_POWER Messwert = 44444 - 65535 - 1 = -21092

Wenn man es sich vereinfach auf zwei Zahlenstrahls vorstellt, ist es zu verstehen, siehe Anhang.

Deleting multiple rows at once doesnt work

only one row gets deleted

Write negativ integers should be possible

Writing negative integers should be possible, but "-" can't be entered in the text field

Convert code to English

Read Devices/Operations from the Excel file provided

Creating java objects for every Device/Operation is not maintainable.
The Excel File should be read and the Devices/Operations created during Runtime.

ModbusResult nicht generic sonder Vererbung verwenden

Im modbusresult muss man immer mit instance of prüfen welcher Datentyp enthalten ist.
Besser wäre Vererbung zu verwenden statt generics dann müssen die konkreten Klasse abstrakte Methoden Überscheiben.

Favoriten liste für Abfrage erstelllen

Um schnellere Abfragen zu gewährleisten bitte ein Favoritenliste hinzufügbar machen

AbstractModbusTCPClient Dependecy Injection verwenden

In AbstractModbusTCPClient Dependecy Injection verwenden, javax package.

Fehlerbehandlung, wenn Gerät nicht zur Operation passt

com.ghgande.j2mod.modbus.ModbusSlaveException: Gateway Path Unavailable
at com.ghgande.j2mod.modbus.io.ModbusTCPTransaction.execute(ModbusTCPTransaction.java:154)
at com.ghgande.j2mod.modbus.facade.AbstractModbusMaster.readMultipleRegisters(AbstractModbusMaster.java:236)
at de.ossi.wolfsbau.modbus.ModbusTCPReader.readOperationFromDeviceInternal(ModbusTCPReader.java:23)
at de.ossi.wolfsbau.modbus.ModbusTCPReader.readInternal(ModbusTCPReader.java:17)
at de.ossi.wolfsbau.modbus.ModbusReaderConnectionHandler.readOperationFromDevice(ModbusReaderConnectionHandler.java:28)

Aktuell kommt eine Exception wenn das Gerät nicht zur Operation passt.

VPN kompatibiltät prüfen

32668 oder 32768?

32668 oder 32768? Range der Werte

Mehr Tests für den Modbus Teil

Mehr Tests für den Modbus Teil erstellen!

Oberflächen design anschaubar machen!

Doku Lesen Modbus regsiter und werte bereich

Beschreibung Modus Werte Basis.pdf
Hier meine Doku und beschreibung

Modbus Excel hochladen

Modbus Excel hochladen

Change Project Name (again)

Please Change Project and JAR name to VenusOsModbusTCP

SLF4J Error in der Console

SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

Classes should not access their own subclasses during initialization (java:S2390)

SQ Issue java:S2390 in class DBusUnit :

When a parent class references a member of a subclass during its own initialization, the results might not be what you expect because the child class might not have been initialized yet. This could create what is known as an "initialisation cycle", or even a deadlock in some extreme cases.

To make things worse, these issues are very hard to diagnose so it is highly recommended you avoid creating this kind of dependencies.