Skip to content
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

Patch 1 #1

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Patch 1 #1

wants to merge 8 commits into from

Conversation

mcmenon
Copy link

@mcmenon mcmenon commented Feb 19, 2018

Lots of updates to instructions.

Modifications to use TCP instead of UDP. This works now.

@mcmenon mcmenon closed this Feb 19, 2018
@mcmenon mcmenon reopened this Feb 19, 2018
README.md Outdated
- `cd /etc/systemd/network`
- edit lidar.network and set IP that talks to the beaglebone (where the LIAR is normally) to be 192.168.0.4
- `sudo apt install socat`
- Run `ifconfig` on each to verify that the microhard radio is `enp0s20f0u5u3`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer necessary now that it uses TCP instead of UDP multicast (only multicast needs to specify the device)

README.md Outdated
- edit lidar.network and set IP that talks to the beaglebone (where the LIAR is normally) to be 192.168.0.4
- `sudo apt install socat`
- Run `ifconfig` on each to verify that the microhard radio is `enp0s20f0u5u3`
- From home directory, `git clone https://github.com/shield-ai/rtlizer.git`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Robot does not need any code, just socat installed

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

robot still needs ./shockwave_proxy.sh, doesn't it?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, because the contents of it are actually just copied directly into skywalker

README.md Outdated
- `cd rtlizer/client`
- `make`
- `cd ..`
- Edit fake_ground_station to set the device to the network device through which your ground station is connected to wave relay (perhaps `eth0` or `enp3s0` or `wlan0` or `wlp2s0`)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer necessary with TCP, only UDP multicast

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GCS still needs socat, correct?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it still needs everything except setting the network device

README.md Outdated
On the beaglebone, see `~/rtlizer/start.sh`. If you set `fake_data` to true, it will
On the beaglebone:
- `cd ~/rtlizer`
- `./start.sh`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These new steps are not necessary because the rc.local change runs start.sh automatically

README.md Outdated
On skywalker, run `shockwave_proxy.sh`.
On skywalker,:
- `cd ~/rtlizer`
- `./shockwave_proxy.sh`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script is actually full redundant with the socat changes in skywalker. No need to run it manually; it should either be deleted or called from skywalker

@@ -2,4 +2,4 @@

microhard=enp0s20f0u5u3

socat UDP4-RECV:5555 UDP4-DATAGRAM:224.255.0.1:6666,ip-add-membership=224.255.0.1:$microhard
socat UDP4-RECV:5555 TCP4-LISTEN:8888
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this script is not run, so either these changes need to be made in skywalker or this script should be called from there

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added to the instructions to run this line on skywalker


On the ground station, run `fake_ground_station` or `real_ground_station`, espective of the value of `fake_data` on the beaglebone.
On the ground station:
- `socat TCP4:<robot>:8888 -`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The old line was correct, this change is not necessary, instead fake_ground_station needs to be updated


On skywalker, run `shockwave_proxy.sh`.
On skywalker,:
- `socat UDP4-RECV:5555 TCP4-LISTEN:8888`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is automatically run in the skywalker script (which needs to be updated) so nothing needs to be run manually

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants