Taurus development guidelines

Overview

This document describes taurus from the perspective of developers. Most importantly, it gives information for people who want to contribute to the development of taurus. So if you want to help out, read on!

How to contribute to taurus

Taurus is Free Software developed in open way. Contributions to code, documentation, etc. are always welcome.

The “official” Taurus source code is hosted in a git repository.

The details in how to contribute are described in the CONTRIBUTING.md file at the root of the git repository.

Documentation

All standalone documentation should be written in plain text (.rst) files using reStructuredText for markup and formatting. All such documentation should be placed in directory docs/source of the taurus source tree. The documentation in this location will serve as the main source for taurus documentation and all existing documentation should be converted to this format.

Coding conventions

  • Code in Taurus should follow the standard Python style conventions as described in PEP8, and more specifically it should be formatted according to black and flake8 conventions (which are enforced by our continuous integration tests). Specially:

    • Use 4 spaces for indentation

    • Respect the maximum of 79 characters per line

    • Surround top-level function and class definitions with two blank lines.

    • use lower_case for module names. If possible prefix module names with the word taurus (like taurusutil.py) to avoid import mistakes.

    • use CamelCase for class names

    • use lower_case for method names, except in the context of taurus.qt where the prevailing convention is mixedCase due to influence from PyQt

  • Code must be compatible with python >=3.5.

  • Every python module file should contain license information (see template below). The preferred license is the LGPL. If you need/want to use a different one, it should be compatible with the LGPL v3+.

  • avoid polluting namespace by making private definitions private (__ prefix) or/and implementing __all__ (see template below)

  • whenever a python module can be executed from the command line, it should contain a main function and a call to it in a if __name__ == "__main__" like statement (see template below)

  • All public API code should be documented (modules, classes and public API) using Sphinx extension to reStructuredText

Tip

Run black <taurus_root_dir> to automatically reformat the code to follow black conventions before commit.

Tip

Run flake8 <taurus_root_dir> to check your code against flake8 violations.

Tip

taurus ships a pre-commit hook configuration which you can install by running pre-commit install in the taurus root (you need pre-commit) in order to get all your commits automatically checked with black and flake8

The following code can serve as a template for writing new python modules to taurus:

#!/usr/bin/env python

#############################################################################
##
# This file is part of Taurus
##
# http://taurus-scada.org
##
# Copyright 2011 CELLS / ALBA Synchrotron, Bellaterra, Spain
##
# Taurus is free software: you can redistribute it and/or modify
# it under the terms of the GNU Lesser General Public License as published by
# the Free Software Foundation, either version 3 of the License, or
# (at your option) any later version.
##
# Taurus is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
# GNU Lesser General Public License for more details.
##
# You should have received a copy of the GNU Lesser General Public License
# along with Taurus.  If not, see <http://www.gnu.org/licenses/>.
##
#############################################################################

"""A :mod:`taurus` module written for template purposes only"""

__all__ = ["TaurusDemo"]

__docformat__ = "restructuredtext"

class TaurusDemo(object):
    """This class is written for template purposes only"""

def main():
    print "TaurusDemo"

if __name__ == "__main__":
    main()

Special notes about Qt programming

The following Qt guidelines are intended to ensure compatibility between the supported Qt bindings.

  1. Avoid importing PyQt5 / PySide2 directly. Imports like:

    from PyQt5 import Qt
    from PyQt5 import QtCore
    from PyQt5 import QtGui
    from PyQt5 import QtNetwork
    from PyQt5 import QtWebKit
    

    Should be replaced by:

    from taurus.external.qt import Qt
    from taurus.external.qt import QtCore
    from taurus.external.qt import QtGui
    from taurus.external.qt import QtNetwork
    from taurus.external.qt import QtWebKit
    

Note

this guideline applies to code which is part of the taurus module or its plugins. For end-user applications that use taurus, this rule may not apply, as mentioned in TEP18:

For an end-user application based on taurus it is probably better to import directly from a specific binding (PyQt5 is the best supported) and let taurus to adapt to that choice. In this way, one can write idiomatic code that better matches the chosen binding. Using the taurus.external.qt shim is also possible if one wants to make the code binding-agnostic, but in that case one must keep in mind that the resulting code will be less idiomatic and that the shim’s API may be eventually altered to better fit with taurus own requirements (and that those changes may not be aligned with the application needs).

  1. Since Taurus v>=5.0, Qt-based code in Taurus assumes Qt=5 (i.e. Qt4 is no longer supported). In particular, when porting old code, pay attention to ensure that PyQt API v2 is used.

    • Use standard python strings (e.g., use str for Qt strings instead of QString). Code like:

      my_string = Qt.QString(" hello ")
      my_string2 = my_string.trimmed()
      

      Should be replaced by:

      my_string = " hello "
      my_string2 = my_string.strip()
      
    • Do not use QVariant. Code like:

      def setData(self, index, qvalue, role=Qt.Qt.EditRole):
          value = qvalue.toString()  # this assumes qvalue to be a :class:`QVariant`
          self.buffer[index.column()] = value
      
      def data(self, index, role=Qt.Qt.DisplayRole):
          value = self.buffer[index.column()]
      
          if role == Qt.Qt.DisplayRole:
              return Qt.QVariant(value)
          else:
              return Qt.QVariant()
      

      Should be replaced by:

      def setData(self, index, value, role=Qt.Qt.EditRole):
          self.buffer[index.column()] = value  # value is already a python object
      
      def data(self, index, role=Qt.Qt.DisplayRole):
          value = self.buffer[index.column()]
      
          if role == Qt.Qt.DisplayRole:
              return value
          else:
              return None
      

      For backwards-compatibility reasons, taurus.external.qt.QtCore defines QVariant, from_qvariant() and to_qvariant(), but they are deprecated and should not be used anymore.

  2. Use “new-style” signals. Code that still uses “old style” signals like the following:

    class MyWidget(Qt.QWidget):
    
    def foo(self):
        self.connect(self, Qt.SIGNAL('mySignal(int)', self.bar))
        self.emit(Qt.SIGNAL('mySignal(int)', 123))
    

    Must be replaced by:

    class MyWidget(Qt.QWidget):
    
        mySignal = Qt.pyqtSignal(int)
    
        def foo(self):
            self.mySignal.connect(self.bar)
            self.mySignal.emit(123)
    
  3. The taurus.external.qt.compat module defines some convenience utilities that help in writing Qt-binding agnostic code

  4. Use of taurus.qt.qtgui.application.TaurusApplication instead of QApplication is recommended (it takes care of various initialization and exit tasks that are convenient).