injones / mycroft_ros

A ROS wrapper for Mycroft AI
Apache License 2.0
5 stars 0 forks source link

Install instructions aren't clear #1

Open machinekoder opened 4 years ago

machinekoder commented 4 years ago

First of all, thanks for your work. I thought about creating a ROS/Mycroft package myself and you probably save me a lot of time.

I tried to follow your install instructions, but I can't quite get it to work. How is catkin_virtualenv coming into play? And does it require a Python3 build of ROS?

I really would prefer if the package just uses the system installed version of Mycroft, since the virtualenv install fails.

injones commented 4 years ago

Apologies for the late response, I have been unable to work on this for awhile and recently tried to get the package working on a new system and ran into problems regarding catkin_virtualenv myself because it doesn't seem to compile correctly using Python 3, however I have worked around this by changing an import within the catkin_virtualenv/scripts/glob_requirements file of the catkin_virtualenv ROS package shown below:

Python 2

import argparse
import sys
from Queue import Queue

Python 3

import argparse
import sys
from queue import Queue

The use case I had was to be able to just install this package and run its setup without seperately installing Mycroft. I used catkin_virtualenv to mimic the way that Mycroft is installed and run inside a Python virtual environment seperate from the system Python installation, which catkin_virtualenv allows. Although, not entirely necessary so catkin_virtualenv could be removed depending on the use case.

The package almost certainly could be changed to use the system installed version of Mycroft however the source code in this package has a slight alteration so you would be required to add the change before installing.

I will try to write more detailed installation steps soon. Hope this helps in the mean time.