-
Notifications
You must be signed in to change notification settings - Fork 0
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
Evaluating new clustering + sensor model #123
base: main
Are you sure you want to change the base?
Conversation
RUN cd src \ | ||
&& git clone https://github.com/Ekumen-OS/beluga \ | ||
&& cd beluga \ | ||
&& git checkout 3a800c6 && echo "Checking out fbattocchia_likelihoob_prob..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove before final version of the PR
RUN cd src \ | ||
&& git clone https://github.com/Ekumen-OS/beluga \ | ||
&& cd beluga \ | ||
&& git checkout 3a800c6 && echo "Checking out fbattocchia_likelihoob_prob..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove before final version of the PR
<param name="use_sim_time" value="$(var use_sim_time)" /> | ||
</node> --> | ||
|
||
</launch> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
end of line.
Proposed changes
This PR merges into the main Beluga evaluation report changes created to evaluate the impact of Ekumen-OS/beluga#476
Type of change
💥 Breaking change! Explain why a non-backwards compatible change is necessary or remove this line entirely if not applicable.
Checklist
Put an
x
in the boxes that apply. This is simply a reminder of what we will require before merging your code.Additional comments
Anything worth mentioning to the reviewers.