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

test: PRT - Attempt to fix statistical optimizer test #1718

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

shleikes
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Sep 26, 2024

Test Results

2 082 tests   - 125   2 081 ✅  - 126   25m 1s ⏱️ -39s
  143 suites  -   2       0 💤 ±  0 
    6 files    -   1       1 ❌ +  1 

For more details on these failures, see this check.

Results for commit b7cb628. ± Comparison against base commit 130d0f1.

This pull request removes 125 tests.
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheExpirationMultiplier
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheExpirationMultiplier/Multiplier_of_1
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheExpirationMultiplier/Multiplier_of_1.2
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheExpirationMultiplier/Multiplier_of_2
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheExpirationMultiplier/Multiplier_of_2.5
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheExpirationMultiplier/Multiplier_of_200
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheFailSetWithInvalidRequestBlock
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheFailSetWithInvalidRequestBlock/Finalized_After_delay_No_Hash
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheFailSetWithInvalidRequestBlock/Finalized_After_delay_No_Hash#01
github.com/lavanet/lava/v3/ecosystem/cache ‑ TestCacheFailSetWithInvalidRequestBlock/Finalized_After_delay_With_Hash
…

♻️ This comment has been updated with latest results.

@shleikes
Copy link
Collaborator Author

shleikes commented Sep 29, 2024

Still happens after setting a delta of 10 to the require:

Max difference between 346 and 300 allowed is 10, but difference was 46

@omerlavanet Any ideas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants