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

[FW-296] Added the code needed to execute the simulator tests with GitHub actions #46

Merged
merged 11 commits into from
Mar 12, 2025

Conversation

jdmarmen
Copy link
Contributor

No description provided.

Copy link
Member

@g0nz4I0 g0nz4I0 left a comment

Choose a reason for hiding this comment

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

Why did you change the ALLOWED_SLACK?, everything else looks good.

@jdmarmen
Copy link
Contributor Author

jdmarmen commented Mar 3, 2025

Why did you change the ALLOWED_SLACK?, everything else looks good.

Test wasn't passing with lower slacks, it literally works from 100 onwards. I guess it's due to the issues with low time measurwments that Juan has mentioned a bunch of times.

@jmaralo
Copy link
Member

jmaralo commented Mar 3, 2025

The thing with the SLACK is probably due to github action runners being low on resources, maybe we should execute the simulation with a slower clock both in python and c++

jmaralo
jmaralo previously approved these changes Mar 3, 2025
jmaralo
jmaralo previously approved these changes Mar 6, 2025
@jmaralo jmaralo merged commit 29eba02 into main Mar 12, 2025
13 checks passed
@jmaralo jmaralo deleted the FW-296/Execute-tests-with-actions branch March 12, 2025 20:09
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.

3 participants