Ulm-IQO / qudi-core

A framework for modular measurement applications.
GNU General Public License v3.0
38 stars 21 forks source link

DictTableModel bugfix #78

Closed Neverhorst closed 1 year ago

Neverhorst commented 1 year ago

Description

Bugfix in the qudi.util.models.DictTableModel.get_key_by_index method. Index was off-by-one. Also improved IndexError message.

Motivation and Context

Resolves #70

How Has This Been Tested?

Load config with one or more modules configured as allow_remote: True and click in manager GUI View -> Show remote

Types of changes

Checklist:

LukePiewalker commented 1 year ago

I just tested on my office PC and with remote modules, while starting qudi from the console with two different configs via the "-c" option. The table lists all the modules, with "allow_remote: True", as expected. Just to double check: The column "remote modules" aka remote_module_listview, besides "shared modules" so far is unpopulated, since it is probably not implemented yet, correct?