pinorobotics / msgmonster

Application for generating message Java classes for jrosclient from ROS msg files.
5 stars 2 forks source link

Hanging on Message generation #3

Closed marlow-fawn closed 3 months ago

marlow-fawn commented 3 months ago

Having an issue when running on bosdyn_api_msgs. It appears to hang on the following message:

NetworkComputeResponse.txt

This also happens when trying to build just that message. I don't actually need this message, so if there's some way to ignore certain messages in a package that would be a sufficient workaround for now.

aeon9 commented 3 months ago

Hi,

Thanks for using jrosclient

We added debug mode support.

Could you please try prerelease version with option "-d" and send us "msgmonster-debug.log" file under system temporary directory.

Regards,

aeon9 commented 3 months ago

Hi @marlow-fawn ,

We added improvements for ROS constants and could generate Java class for NetworkComputeResponse.txt Could you please try new prerelease version.

If it still hangs please use option "-d" and send us "msgmonster-debug.log" file under system temporary directory.

Additionally we added "-exclude" option support (see README for details how to use it) so you can skip any problematic messages.

Regards,

marlow-fawn commented 3 months ago

Thanks so much! should have time to test it before the weekend.

marlow-fawn commented 3 months ago

This is all that's written to the debug file before hanging. msgmonster-debug.log

`

aeon9 commented 3 months ago

Hi @marlow-fawn ,

Thank you for sending the log.

Based on it could you please run:

ros2 interface show bosdyn_api_msgs/msg/NetworkComputeResponse && echo $?

And send the output here.

Additionally when msgmonster hangs could you please run:

jstack <msgmonster_PID> >> stack

And repeat same command 3 times with 5sec intervals. Then please send "stack" file as well.

Regards,

marlow-fawn commented 3 months ago

The exit status is 0.

Here's the stack file: stack.txt

aeon9 commented 3 months ago

Hi @marlow-fawn,

The fix is ready and will be included in the next version we plan to release. Until then you can use prerelease version

Regards,

aeon9 commented 3 months ago

Hi,

We just published a new v6 release where this issue is fixed.

Regards,