G.A. vd. Hoorn

Results 1778 comments of G.A. vd. Hoorn
trafficstars

I've addressed most of the discussed changes in functionality. Tested it here as well and it still works. Just need to test behaviour when there are more jobs than allowed,...

I have tested, but I'd like some independent confirmation I'm not making (implicit) assumptions about how things work based on my controller configuration. I'll make a build of `micro_ros_motoplus` available...

@ted-miller: I believe this is the final version of this PR. Switched to named result codes and strings, renamed files to match the fact it's now specifically about *INFORM* jobs...

I'll likely squash some of the commits before merging.

I'd like to take another look at how this reports errors to the client. I just haven't found time to do that yet.

Current design of the services: [here](https://github.com/gavanderhoorn/motoros2_interfaces/tree/inform_crud) ([diff](https://github.com/Yaskawa-Global/motoros2_interfaces/compare/main...gavanderhoorn:motoros2_interfaces:inform_crud)).

Current WIP of everything except `inform_job/list`: https://github.com/Yaskawa-Global/motoros2/compare/inform_job_crud_list...gavanderhoorn:motoros2:inform_job_crud_the_rest. Service definitions: https://github.com/Yaskawa-Global/motoros2_interfaces/compare/main...gavanderhoorn:motoros2_interfaces:inform_crud.

I've been trying to test my latest changes (adding 'the rest' of the CRUD operations), but for some reason none of the new services 'work'. The client just hangs. No...

Ah, right :coffee: :sleepy: --- Edit: switched to using `micro_ros_utilities` instead of doing this all manually: 2bc333f7d2060eb5b3e9da2968e9e0b0f2ed61d8.

The latest additions now work, but there's one thing to consider/discuss: I initially configured a maximum job file size of 4 KB. That turns out to be way too small,...