-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
README setup instructions are confusing #44
Comments
Then there's something missing on the Do you remember what was missing from the wstool? |
Based on the errors I got, that I fixed installing the following with
I was going carzy because I thought I had already posted this in a issue, now I see you already saw it :-D |
Bring here the discussion about the Set Up instructions: |
I've removed the Manual setup section in 53c5ae2, I should work now with the wstools. |
@marioney This is still an issue. I needed to install these packages manually. |
I see now https://github.com/rosin-project/metacontrol_experiments is not in the metacontrol_sim.rosinstall file. I think I added the dependencies in the wrong repo, they should be here. |
OK, I've added the dependencies to this repo in 08f8c5c rosdep should install everything now |
Because there are two sections: "Setup using wstool" and "Manually setup", I though I could simply follow one of them, so I did with wstool.
But you actually need to do BOTH.
I propose to have a single "Installation and setup section", and introduce both sets of instructions with a sentence in normal text, not as a header.
The text was updated successfully, but these errors were encountered: