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

Fix IKOS tutorial (#138). #139

Merged
merged 3 commits into from
Apr 26, 2024
Merged

Fix IKOS tutorial (#138). #139

merged 3 commits into from
Apr 26, 2024

Conversation

xfiderek
Copy link
Contributor

This PR addresses #138, by adjusting the Earthfile, entrypoint.sh, and README.md, so that IKOS tutorial is runnable.

@xfiderek
Copy link
Contributor Author

Note that space_nav2 fails also on main, so it is unrelated to this PR

@ivanperez-keera
Copy link
Contributor

I'm looking into this.

@xfiderek
Copy link
Contributor Author

xfiderek commented Apr 23, 2024

@ivanperez-keera failure of space_robots CI is unrelated to this PR

@ivanperez-keera
Copy link
Contributor

Thanks, @xfiderek .

@ivanperez-keera
Copy link
Contributor

@xfiderek I made a minor change to the commit messages prior to the merge. Just waiting on the build but it should all be fine.

@ivanperez-keera
Copy link
Contributor

ivanperez-keera commented Apr 25, 2024

@xfiderek I'm not able to merge this until we fix #143 , which is breaking the current build.

Can you please send a PR also for #143. Don't rebase it on top of this one, base the new PR on top of the current main.

EDIT: Related issue fixed, this PR has been rebased on top of main.

Add environment variable to entrypoint of spaceros image. This variable is
required for ikos-scan to execute and generate .ikosbin files without having to
run the ikos-scan server.
The current instructions specify how to re-run all tests, which takes
substantial time. This change explains how to run ikos-scan specifically, which
should be faster than running all tests.
@ivanperez-keera ivanperez-keera added this to the humble-2024.04.0 milestone Apr 26, 2024
@ivanperez-keera ivanperez-keera merged commit 3bbbaa2 into space-ros:main Apr 26, 2024
4 checks passed
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