pylint & pychecker

pylint & pychecker

asked 2012-07-03 00:22:57 -0500

narcispr
307 ●12 ●15 ●22

updated 2014-01-28 17:12:53 -0500

ngrennan
1 ●1 ●1

Have you tried to use pylint or pychecker to improve/check your python code? I've tried with pylint but it can't import rospy and srv or msg packages:

No config file found, using default configuration
F: 6: Unable to import 'rospy'
F: 8: Unable to import 'my_package.srv'
F: 9: Unable to import 'my_package.msg'
F: 10: Unable to import 'std_srvs.srv'
F: 11: Unable to import 'std_msgs.msg'
...

With PyChecker I've a similar problem:

Processing module my_node (my_node.py)...
ImportError: No module named srv

Warnings...

my_node:1: NOT PROCESSED UNABLE TO IMPORT

Any idea? I found this ticket related with this topic, but it was not very useful for me. Thank you!

answered 2012-07-03 01:08:52 -0500

KruseT

7848 ●40 ●87 ●146

updated 2012-07-04 23:54:57 -0500

It should be sufficient to have your ros environment set up. Meaning if you create your own ros package, make sure it is on the ROS_PACKAGE_PATH.

In your case the python file that has

import rospy

need to to have before that

import roslib
roslib.load_manifest('...')

with .. being either rospy or your package, but for the latter your package also needs to declare rospy in it's manifest.xml.

pychecker seems to be unable to check scripts that do not end with .py. The solution is to remove most of the code from such files like "my_node" and put the code inside the src folder. Then my_node can import the module from src, and pychecker can check the code in src.

EDIT: This should work in fuerte, and without roslib.load_manifest. In electric and earlier, load_manifest is required for pychecker. pylint ignores that, so pylint cannot find libraries in electric and before.

edit flag offensive delete link

Comments

My node starts with: import roslib roslib.load_manifest('my_package') import rospy and the manisfest contains <depend package="rospy"/> but I've the problem previously mentioned.

narcispr ( 2012-07-03 01:14:26 -0500 )edit

Just a note: in Fuerte, roslib.load_manifest should not be required anymore and there is a pretty good chance that pychecker will work.

Lorenz ( 2012-07-03 01:55:15 -0500 )edit

pylint --version No config file found, using default configuration pylint 0.23.0, astng 0.21.1, common 0.55.2 Python 2.7.2+ (default, Oct 4 2011, 20:03:08) [GCC 4.6.1] I'm using Ubuntu 11.10 & ROS Electric

narcispr ( 2012-07-03 05:35:40 -0500 )edit

$ pylint safety_g500.py | grep 'F:' No config file found, using default configuration F: 6: Unable to import 'rospy' F: 8: Unable to import 'safety_g500.srv' F: 9: Unable to import 'safety_g500.msg' F: 10: Unable to import 'std_srvs.srv' ...

narcispr ( 2012-07-03 05:36:33 -0500 )edit

ok, in electric, it is still required to run load_manifest. pylint however ignores load_manifest, so it will never be able to find rospy, unless you manually put it onto your PYTHONPATH before manually.

KruseT ( 2012-07-03 06:11:44 -0500 )edit

However pychecker should work, as long as you put your code into files named xyz.py. You can also name your nodes like that, to start with.

KruseT ( 2012-07-03 06:14:27 -0500 )edit

add a comment

1 Answer

Comments

Question Tools

Stats

Related questions

pylint & pychecker edit

1 Answer

Comments

Question Tools

Stats

Related questions

pylint & pychecker