DLR-RM / RAFCON

RAFCON (RMC advanced flow control) uses hierarchical state machines, featuring concurrent state execution, to represent robot programs. It ships with a graphical user interface supporting the creation of state machines and contains IDE like debugging mechanisms. Alternatively, state machines can programmatically be generated using RAFCON's API.
https://dlr-rm.github.io/RAFCON/
Eclipse Public License 1.0
180 stars 34 forks source link

pylint under osl not working #264

Closed franzlst closed 4 years ago

franzlst commented 4 years ago

pylint under osl not working

Originally created by @sebastian-brunner (Sebastian.Brunner@dlr.de) at 2016-12-09 08:43:25+00:00 (moved from RMC internal repository)

franzlst commented 4 years ago

I think this is an issue with the OSL system configuration. python is linked to the Python 2.7 interpreter, but pylint is linked to the PyLint version with the Python 3 interpreter. I just talked to the admin (Stefan Engelhardt) and he will try to fix this.

Originally created by @franzlst (Franz.Steinmetz@dlr.de) at 2016-12-12 12:35:12+00:00 (moved from RMC internal repository)

franzlst commented 4 years ago

Alternatively, change epylint to epylint-2.7 in https://rmc-github-server/common/rafcon/blob/719c8aedb98d3f794255673259c11fd277449940/source/rafcon/gui/controllers/state_editor/source_editor.py#L227

Originally created by @franzlst (Franz.Steinmetz@dlr.de) at 2016-12-12 14:23:44+00:00 (moved from RMC internal repository)

franzlst commented 4 years ago

Replacing epylint by epylint-2.7 does not do the job, as epylint-2.7 is not available for SLED. Therefore, a routine must previously check, which binaries are available and choose the most appropriate one.

In addition, epytlint is currently not working under OSL, as there is an outdated version of astroids installed. I have already filed a request for an update.

Originally created by @franzlst (Franz.Steinmetz@dlr.de) at 2016-12-12 16:39:07+00:00 (moved from RMC internal repository)

franzlst commented 4 years ago

66236328d9f76a3ba91cd6e1ae7f90f4163f0704 fixes a lot of issues with the linter. As soon as astroids is updated, the linter will run under OSL.

Originally created by @franzlst (Franz.Steinmetz@dlr.de) at 2016-12-14 14:09:11+00:00 (moved from RMC internal repository)

franzlst commented 4 years ago

The admins updated astroids. Pylint is this finally working under OSL!

Originally created by @franzlst (Franz.Steinmetz@dlr.de) at 2017-01-16 10:37:10+00:00 (moved from RMC internal repository)