Merge pull request #80 from robamu-org/mueller/v3.0.0rc1

v3.0.0rc1

.github/workflows/package.yml

@@ -5,16 +5,17 @@ on: [push]
 jobs:
   build:
 
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
     strategy:
       matrix:
+        os: [ubuntu-latest, windows-latest]
         python-version: ['3.8', '3.9', '3.10']
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
 
     - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v4
       with:
         python-version: ${{ matrix.python-version }}
 
@@ -32,11 +33,11 @@ jobs:
         flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
     - name: Run tests and generate coverage data
       run: |
-        python3 -m pip install coverage pytest
+        python3 -m pip install coverage pytest pyfakefs
         coverage run -m pytest
 
     - name: Upload coverage to Codecov
-      uses: codecov/codecov-action@v1
+      uses: codecov/codecov-action@v3
       with:
         flags: unittests
         name: codecov-umbrella

.gitignore

@@ -12,11 +12,14 @@ build
 *.egg-info
 
 /examples/log
-/src/log
-/src/tests/log
+/log
+/tests/log
+/tmtccmd/log
 
 .idea
 venv
 
+/misc/.xdp-*
+
 # CodeCov
 .coverage

CHANGELOG.md

@@ -7,6 +7,23 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/).
 
 ## [unreleased]
 
+## [v3.0.0rc1] 03.07.2022
+
+- Overhaul of application architecture
+- Significant simplification of various modules to increase testability
+- Test coverage increased
+- Reduced number of modules significantly by moving code into the repective `__init__` files where
+  possible
+- GUI improved, added separate TM listening button
+- Documentation improved
+- New logo
+- Simplified general package structure, remove `src` folder and have `tmtccmd` package and `tests`
+  package in repo root
+- First CFDP handler components
+- Reduce usage of globals. The end goal is to remove them altogether
+- Reduce overall number of spawned threads
+- Added Sequence Count handling modules
+
 ## [v2.2.2]
 
 - Improve internal structure of sequential sender receiver object

MANIFEST.in

@@ -1,12 +1,15 @@
 include LICENSE
 include NOTICE
+include CHANGELOG.md
 
-include docs/*.rst
-include docs/logo_tmtccmd.png
-include docs/conf.py
-include docs/Makefile
-include docs/*.md
+graft docs
+prune docs/_build
 
-include tests/*
+graft tests
+prune tests/log
 
-include pycharm/*
+include requirements.txt
+
+recursive-exclude * *.pyc *.pyo
+
+include misc/logo-tiny.png

README.md

@@ -1,50 +1,70 @@
-<p align="center"> <img src="docs/logo_tmtccmd_smaller.png" width="40%"> </p>
+<p align="center"> <img src="misc/logo.png" width="50%"> </p>
 
-TMTC Commander Core [![Documentation Status](https://readthedocs.org/projects/tmtccmd/badge/?version=latest)](https://tmtccmd.readthedocs.io/en/latest/?badge=latest)
+TMTC Commander [![Documentation Status](https://readthedocs.org/projects/tmtccmd/badge/?version=latest)](https://tmtccmd.readthedocs.io/en/latest/?badge=latest)
 [![package](https://github.com/spacefisch/tmtccmd/actions/workflows/package.yml/badge.svg)](https://github.com/spacefisch/tmtccmd/actions/workflows/package.yml)
 [![codecov](https://codecov.io/gh/robamu-org/tmtccmd/branch/develop/graph/badge.svg?token=BVOE3A4WE4)](https://codecov.io/gh/robamu-org/tmtccmd)
 [![PyPI version](https://badge.fury.io/py/tmtccmd.svg)](https://badge.fury.io/py/tmtccmd)
 ====
 
 ## Overview
 
-- Documentation: https://tmtccmd.readthedocs.io/en/latest/
-- Project Homepage: https://github.com/robamu-org/tmtccmd
+- [Documentation](https://tmtccmd.readthedocs.io/en/latest/)
+- [Project Homepage](https://github.com/robamu-org/tmtccmd)
 
-This commander application was first developed by KSat for the 
-[SOURCE](https://www.ksat-stuttgart.de/en/our-missions/source/) project to test the on-board 
-software but has evolved into a more generic tool for satellite developers to perform TMTC 
-(Telemetry and Telecommand) handling and testing via different communication interfaces. 
-Currently, only the PUS standard is implemented as a packet standard. This tool can be used either 
-as a command line tool or as a GUI tool. The GUI features require a PyQt5 installation.
+This is a small Python framework for satellite developers to perform TMTC
+(Telemetry and Telecommand) handling and testing via different communication interfaces.
+This tool can be used either as a command line tool or as a GUI tool. The GUI features require a
+PyQt5 installation. This package also has dedicated support to send and receive ECSS PUS packets
+or other generic CCSDS packets.
 
-This client currently supports the following communication interfaces:
+The TMTC commander also includes some telemetry handling components and telecommand packaging
+helpers. Some of those components are tailored towards usage with the
+[Flight Software Framework (FSFW)](https://egit.irs.uni-stuttgart.de/fsfw/fsfw/).
+
+## Features
+
+- Special support for `Packet Utilisation Standard (PUS)`_ packets and `CCSDS Space Packets`_.
+  This library uses the `spacepackets`_ library for most packet implementations.
+- Support for both CLI and GUI usage
+- Flexibility in the way to specify telecommands to send and how to handle incoming telemetry.
+  This is done by requiring the user to specify callbacks for both TC specification and TM handling.
+- One-Queue Mode for simple command sequences and Multi-Queue for more complex command sequences
+- Listener mode to only listen to incoming telemetry
+- Basic logger components which can be used to store sent Telecommands and incoming Telemetry
+  in files
+- Some components are tailored towards usage with the
+  `Flight Software Framework (FSFW) <https://egit.irs.uni-stuttgart.de/fsfw/fsfw/>`_.
+
+This has a communication interface abstraction which allows to exchange TMTC through different
+channels. The framework currently supports the following communication interfaces:
 
 1. TCP/IP with UDP and TCP
 2. Serial Communication using fixed frames or a simple ASCII based transport layer
 3. QEMU, using a virtual serial interface
 
-The TMTC commander also includes some telemetry handling components and telecommand packaging
-helpers. Some of those components are tailored towards usage with the
-[Flight Software Framework (FSFW)](https://egit.irs.uni-stuttgart.de/fsfw/fsfw/).
+It is also possible to supply custom interfaces.
 
 ## Examples 
 
-The `example` folder contains a simple example using a dummy communication interface.
-It can be run like this on Linux
+The [`examples`](https://github.com/robamu-org/tmtccmd/tree/main/examples) folder contains a simple
+example using a  dummy communication interface. It sends a PUS ping telecommand and then reads the
+ping reply and  the verification replies back from the dummy interface. It can be run like this
+on Linux:
 
 ```sh
-cd example
-./tmtccli.py
+cd examples
+./tmtcc.py
 ```
 
-or on Windows
+or on Windows:
 
 ```sh
-cd example
-py tmtccli.py
+cd examples
+py tmtcc.py
 ```
 
+You can run the GUI mode by supplying `-g` to the commands above.
+
 The [EIVE](https://egit.irs.uni-stuttgart.de/eive/eive-tmtc) and
 [SOURCE](https://git.ksat-stuttgart.de/source/tmtc) project implementation of the TMTC commander
 provide more complex implementations.

docs/README_PyPI.md

@@ -1,6 +1,6 @@
-![](https://github.com/rmspacefish/tmtccmd/blob/source/docs/logo_tmtccmd_smaller.png)
+![](https://github.com/robamu-org/tmtccmd/blob/master/misc/logo.png)
 
-TMTC Commander Core [![Documentation Status](https://readthedocs.org/projects/tmtccmd/badge/?version=latest)](https://tmtccmd.readthedocs.io/en/latest/?badge=latest)
+TMTC Commander[![Documentation Status](https://readthedocs.org/projects/tmtccmd/badge/?version=latest)](https://tmtccmd.readthedocs.io/en/latest/?badge=latest)
 [![package](https://github.com/spacefisch/tmtccmd/actions/workflows/package.yml/badge.svg)](https://github.com/spacefisch/tmtccmd/actions/workflows/package.yml)
 [![codecov](https://codecov.io/gh/robamu-org/tmtccmd/branch/develop/graph/badge.svg?token=BVOE3A4WE4)](https://codecov.io/gh/robamu-org/tmtccmd)
 [![PyPI version](https://badge.fury.io/py/tmtccmd.svg)](https://badge.fury.io/py/tmtccmd)
@@ -10,4 +10,3 @@ TMTC Commander Core [![Documentation Status](https://readthedocs.org/projects/tm
 
 - Project Homepage: https://github.com/robamu-org/tmtccmd
 - Documentation: https://tmtccmd.readthedocs.io/en/latest/
-

docs/api.rst

@@ -5,10 +5,10 @@ API
 Core Submodules
 ===============
 
-tmtccmd.runner module
+tmtccmd module
 ---------------------
 
-.. automodule:: tmtccmd.runner
+.. automodule:: tmtccmd
    :members:
    :undoc-members:
    :show-inheritance:
@@ -17,7 +17,6 @@ tmtccmd.runner module
    :maxdepth: 4
 
    api/tmtccmd.core
-   api/tmtccmd.sendreceive
 
 Configuration Submodules
 =========================
@@ -34,11 +33,10 @@ CCSDS & ECSS Submodules
 .. toctree::
    :maxdepth: 4
 
-   api/tmtccmd.ccsds
+   api/tmtccmd.cfdp
    api/tmtccmd.pus
    api/tmtccmd.tc
    api/tmtccmd.tm
-   api/tmtccmd.cfdp
 
 Other Submodules
 =========================
@@ -47,4 +45,5 @@ Other Submodules
    :maxdepth: 4
 
    api/tmtccmd.utility
-   api/tmtccmd.logging
+   api/tmtccmd.logging
+   api/tmtccmd.fsfw

docs/api/tmtccmd.com_if.rst

@@ -1,85 +1,77 @@
 tmtccmd.com\_if package
 =======================
 
+Module contents
+---------------
+
+.. automodule:: tmtccmd.com_if
+   :members:
+   :undoc-members:
+   :show-inheritance:
+
 Submodules
 ----------
 
-tmtccmd.com\_if.com\_if\_utilities module
+tmtccmd.com\_if.utils module
 -----------------------------------------
 
-.. automodule:: tmtccmd.com_if.com_if_utilities
+.. automodule:: tmtccmd.com_if.utils
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.com\_interface\_base module
+tmtccmd.com\_if.tcp module
 -------------------------------------------
 
-.. automodule:: tmtccmd.com_if.com_interface_base
+.. automodule:: tmtccmd.com_if.tcp
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.dummy\_com\_if module
+tmtccmd.com\_if.udp module
 -------------------------------------
 
-.. automodule:: tmtccmd.com_if.dummy_com_if
+.. automodule:: tmtccmd.com_if.udp
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.qemu\_com\_if module
-------------------------------------
+tmtccmd.com\_if.tcpip\_utils module
+-------------------------------------
 
-.. automodule:: tmtccmd.com_if.qemu_com_if
+.. automodule:: tmtccmd.com_if.tcpip_utils
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.serial\_com\_if module
---------------------------------------
+tmtccmd.com\_if.qemu module
+------------------------------------
 
-.. automodule:: tmtccmd.com_if.serial_com_if
+.. automodule:: tmtccmd.com_if.qemu
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.serial\_utilities module
-----------------------------------------
+tmtccmd.com\_if.serial module
+--------------------------------------
 
-.. automodule:: tmtccmd.com_if.serial_utilities
+.. automodule:: tmtccmd.com_if.serial
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.tcpip\_tcp\_com\_if module
-------------------------------------------
+tmtccmd.com\_if.ser\_utils module
+----------------------------------------
 
-.. automodule:: tmtccmd.com_if.tcpip_tcp_com_if
+.. automodule:: tmtccmd.com_if.ser_utils
    :members:
    :undoc-members:
    :show-inheritance:
 
-tmtccmd.com\_if.tcpip\_udp\_com\_if module
+tmtccmd.com\_if.dummy module
 ------------------------------------------
 
-.. automodule:: tmtccmd.com_if.tcpip_udp_com_if
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
-tmtccmd.com\_if.tcpip\_utilities module
----------------------------------------
-
-.. automodule:: tmtccmd.com_if.tcpip_utilities
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
-Module contents
----------------
-
-.. automodule:: tmtccmd.com_if
+.. automodule:: tmtccmd.com_if.dummy
    :members:
    :undoc-members:
    :show-inheritance: