Merge pull request #4 from ModbusScope/update

Update (port to ModbusSim)

Author: jgeudens

.coderabbit.yaml

@@ -9,6 +9,7 @@ reviews:
   review_status: true
   collapse_walkthrough: false
   changed_files_summary: false
+  sequence_diagrams: false
   estimate_code_review_effort: false
   auto_review:
     enabled: true

ModbusSim.png

@@ -1,7 +1,14 @@
-# Build status
-[![Build status](https://ci.appveyor.com/api/projects/status/v7ysjn9c2koy1tb8?svg=true)](https://ci.appveyor.com/project/jgeudens/modbustestslave)
+<img src="./misc/icon/icon-256x256.png" alt="modbussim logo" title="ModbusSim" align="right" height="150" />
 
-# What is ModbusTestSlave
-ModbusTestSlave is an application that can be used to test a modbus master. The application consists of 2 main set of modules. A GUI-less part that contains all logic that can also be used in automated tests. The second part is a GUI that can be used for manual testing.
+Build status: [![Build](https://github.com/ModbusScope/ModbusSim/actions/workflows/ccpp.yml/badge.svg?branch=master)](https://github.com/ModbusScope/ModbusSim/actions/workflows/ccpp.yml)
 
-![ModbusTestSlave demo](ModbusTestSlave.png)
+# What is ModbusSim
+ModbusSim is a Modbus slave test application for validating Modbus master implementations.
+
+Key features:
+
+- Implements multiple configurable holding registers (values can be automatically incremented).
+- Generates a sine-wave signal as register values.
+- Can inject configurable communication errors to exercise master error handling.
+
+![ModbusSim demo](ModbusSim.png)

ModbusTestSlave.png

@@ -3,7 +3,7 @@ set -ex
 
 # Copy assets
 cp misc/ModbusSim.desktop release/bin/linux/
-cp misc/icon/icon-64x64.png release/bin/linux/ModbusSim.png
+cp misc/icon/icon-256x256.png release/bin/linux/ModbusSim.png
 cd release/bin/linux/
 
 # now, build AppImage using linuxdeploy and linuxdeploy-plugin-qt

README.md

@@ -2,13 +2,11 @@
 
 #include "ui_mainwindow.h"
 
+#include <QDebug>
 #include <QModbusTcpServer>
 #include <QUrl>
-#include <QDebug>
 
-MainWindow::MainWindow(QWidget *parent) :
-    QMainWindow(parent),
-    _pUi(new Ui::MainWindow)
+MainWindow::MainWindow(QWidget* parent) : QMainWindow(parent), _pUi(new Ui::MainWindow)
 {
     _pUi->setupUi(this);
 
@@ -65,10 +63,8 @@ MainWindow::MainWindow(QWidget *parent) :
     /** Setup registerView **/
     _pUi->tblRegData->setModel(_pRegisterDataModel);
     _pUi->tblRegData->verticalHeader()->show();
-    _pUi->tblRegData->horizontalHeader()->show(); // Not sure yet (show or not to show?)
-
-    /* Don't stretch columns, resize to contents */
-    _pUi->tblRegData->horizontalHeader()->setSectionResizeMode(QHeaderView::ResizeToContents);
+    _pUi->tblRegData->horizontalHeader()->show();
+    _pUi->tblRegData->horizontalHeader()->setSectionResizeMode(QHeaderView::Stretch);
 
     QList<uint> registerList = QList<uint>() << 0 << 1 << 2 << 3 << 4;
     _pSlaveData->setRegisterState(registerList, true);
@@ -80,7 +76,7 @@ MainWindow::MainWindow(QWidget *parent) :
     _pSineGraph->setPeriod(10000);
 
     QString windowCaption;
-    windowCaption = QString("ModbusTestSim");
+    windowCaption = QString("ModbusSim");
 
     setWindowTitle(windowCaption);
 }

misc/icon/Icons8_flat_sim_card.svg

@@ -1,14 +1,14 @@
 #ifndef MAINWINDOW_H
 #define MAINWINDOW_H
 
-#include <QMainWindow>
 #include <QButtonGroup>
+#include <QMainWindow>
 
-#include "testslavemodbus.h"
 #include "testslavedata.h"
+#include "testslavemodbus.h"
 
-#include "registerdatamodel.h"
 #include "incgraph.h"
+#include "registerdatamodel.h"
 #include "sinegraph.h"
 
 namespace Ui {
@@ -20,7 +20,7 @@ class MainWindow : public QMainWindow
     Q_OBJECT
 
 public:
-    explicit MainWindow(QWidget *parent = nullptr);
+    explicit MainWindow(QWidget* parent = nullptr);
     ~MainWindow();
 
 private slots:
@@ -34,8 +34,7 @@ private slots:
     void handleRequestProcessed();
 
 private:
-
-    Ui::MainWindow *_pUi;
+    Ui::MainWindow* _pUi;
 
     bool _bAutoInc;
 
@@ -47,10 +46,10 @@ private slots:
     QButtonGroup _exceptionGroup;
     QButtonGroup _errorRecurrenceGroup;
 
-    TestSlaveModbus *_pSlaveModbus;
-    TestSlaveData *_pSlaveData;
+    TestSlaveModbus* _pSlaveModbus;
+    TestSlaveData* _pSlaveData;
 
-    RegisterDataModel *_pRegisterDataModel;
+    RegisterDataModel* _pRegisterDataModel;
 };
 
 #endif // MAINWINDOW_H