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

[sacra-subgraph] Update sacra subgraph strategy #1620

Conversation

alexandersazonof
Copy link
Contributor

  • Support all chains in one proposal
  • Change score logic

@ChaituVR
Copy link
Member

@alexandersazonof Looks like some test cases are failing, can try it in your local?

@alexandersazonof
Copy link
Contributor Author

@alexandersazonof Looks like some test cases are failing, can try it in your local?

Yes all works fine, our subgraph down 5 min ago, can you run tests again ?

@ChaituVR
Copy link
Member

hmm still failing

Test strategy "sacra-subgraph" with example index 0 (with 500 addresses) › Should take less than 20 sec. to resolve with 500 addresses

    expect(received).toBeLessThanOrEqual(expected)

    Expected: <= 20000
    Received:    49073.03940799998

      215 |       moreArg || 500
      216 |     } addresses`, () => {
    > 217 |       expect(getScoresTimeMore).toBeLessThanOrEqual(20000);
          |                                 ^
      218 |     });
      219 |   }
      220 | );

      at Object.<anonymous> (test/strategy.test.ts:217:33)

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

Successfully merging this pull request may close these issues.

2 participants