Skip to content

[ci] Add benchmarker to CI and fix unit test - #1779

Open
barroco wants to merge 2 commits into
interuss:mainfrom
Orbitalize:benchmarker-ci-tests
Open

barroco wants to merge 2 commits into
interuss:mainfrom
Orbitalize:benchmarker-ci-tests

Conversation

@barroco

@barroco barroco commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@barroco
barroco marked this pull request as ready for review October 7, 2026 12:02
@barroco barroco changed the title [ci] Add to benchmarker to CI and fix unit test [ci] Add benchmarker to CI and fix unit test Oct 7, 2026
@barroco
barroco marked this pull request as draft October 7, 2026 12:09
@barroco
barroco force-pushed the benchmarker-ci-tests branch from e52e93d to 11e09b5 Compare October 7, 2026 12:10
@barroco
barroco marked this pull request as ready for review October 7, 2026 13:37

@mickmis mickmis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM but consider defining once those scripts (to be ran from the root) instead of a third duplication.

@barroco

barroco commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor Author

consider defining once those scripts (to be ran from the root) instead of a third duplication.

I wanted to keep them isolated since they are in different independent volumes. This can be revisited in a follow up PR.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants