Skip to content

Conversation

@tutunarsl
Copy link
Contributor

No description provided.


fs::path ROSCameraCameraProgram::fetchOutputPath() {
if (output_path == "") {
// if (output_path == "") {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was breaking package build on the robot, I dont think it is intended

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@fulkast whenever you have time, can you take a look

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 caused by missing changes from the dev/camera_camera_offline_feature branch. For now, this patch here works.

@tutunarsl tutunarsl requested a review from JonasFrey96 August 19, 2025 11:10

# set process priority
sched_priority 90
sched_priority 95
Copy link
Contributor

Choose a reason for hiding this comment

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

is this required ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not required

Type=simple
User=rsl
ExecStart=/bin/bash -c 'source /opt/ros/noetic/setup.bash && source /home/rsl/catkin_ws/devel/setup.bash && python3 /home/rsl/git/grand_tour_box/box_configuration/anymal/lpc/powerline.py'
ExecStart=/bin/bash -c 'source /opt/ros/noetic/setup.bash && source /home/integration/catkin_ws/devel/setup.bash && python3 /home/integration/catkin_ws/src/grand_tour_box/box_configuration/anymal/lpc/powerline.py'
Copy link
Contributor

Choose a reason for hiding this comment

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

we hardcode from one user to the other

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In anymal robots, the user must exist, in current RSL setup only integration user is used on the robots

rm -rf /dev/shm/COSMO_SHM

# On the robot
export GRANDTOUR_WORKSPACE="i_hate_docker_ws"
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried to combine all the workspace entries as an environment variable export under cleanup-session-robot script

currently we have a "i_hate_docker_ws" on democracy. Which has the grpc node source code

@tutunarsl
Copy link
Contributor Author

I will test this on the robot and merge

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.

4 participants