search

RobotLabLTH / skiros2

A skill-based platform for ROS v.2
Other
163 stars 20 forks source link

Getting error that the owl "must be an rdflib term" #83

Open shola-lawal opened 11 months ago

shola-lawal commented 11 months ago

Hi,

I am getting a must be an rdflib term errors. An example of the error message looks like this, "Subject http://www.inf.ufrgs.br/phi-group/ontologies/cora.owl#Robot-1 must be an rdflib term". It doesn't seem that the errors are due to the skiros.owl and skirosExtra.owl files but rather from the owl files in the IEEE-1872-2015 folder. However, this doesn't make sense because I am able to view the owl files from the IEEE-1872-2015 folder in Protégé 5.6.3.

I have attached to outputs below.

Please also note that there are _SyntaxWarning: "is" with a literal. Did you mean "=="?_ errors but this is related to python 3.8 raising a SyntaxWarning due to the usage of "is" instead of "==".

GetImage GetImage1 GetImage2

Environment: Ubuntu 20.04.6 Python 3.8.10 ROS Noetic rdflib 7.0.0

Please advise. Thank you.

matthias-mayr commented 11 months ago

Thank you very much for reporting this.

The Syntax warning seems easy to fix. Interestingly enough, I haven't seen it with my noetic setups. I should take a look at the python versions those use.

For the rdflib-error, can you attach the turtle file of your scene? Here in the user interface is a Save Scene button. That would help to debug it.

shola-lawal commented 11 months ago

Hi @matthias-mayr,

Here is the turtle sim scene:

(Terminal 1) $ roscore

(Terminal 2) $ roslaunch skiros2_examples turtlesim_example.launch

I just added an empty scene after running the skiros2.

Below is the image:

skillros

matthias-mayr commented 10 months ago

Thanks for the instructions. It looks like RDFlib in version 7.0.0 introduced some breaking changes. Until this is resolved, you can switch to the previous version with

pip3 install rdflib==6.3.2
TimKnissel commented 10 months ago

Thank you for the update! With the version rdflib=6.3.2 it works for me.

shola-lawal commented 10 months ago

Amazing, thank you it works now!!

matthias-mayr commented 10 months ago

Thanks for the feedback.

I am reopening this issue, because it's only a fix and currently not even enforced by the installation procedure.

matthias-mayr commented 9 months ago

Chances are that #90 fixes this.

Please also note that there are SyntaxWarning: "is" with a literal. Did you mean "=="? errors but this is related to python 3.8 raising a SyntaxWarning due to the usage of "is" instead of "==".

Thanks for bringing up the warning. It is fixed now in 8b11b9e0df03c1583c4af5b4dff18020309eddf8.