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

Load A and gamma from fixture storage #260

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Load A and gamma from fixture storage #260

wants to merge 3 commits into from

Conversation

allt0ld
Copy link
Collaborator

@allt0ld allt0ld commented Sep 15, 2023

Description

Closes #254.

Hygiene checklist

  • Changelog entry
  • Everything public has a Numpy-style docstring
    (modules, public functions, classes, and public methods)
  • Commit history is cleaned-up with minor changes squashed together
    and descriptive commit messages following Tim Pope's style

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@allt0ld allt0ld self-assigned this Sep 15, 2023
@allt0ld allt0ld closed this Sep 16, 2023
@allt0ld allt0ld reopened this Sep 16, 2023
@allt0ld allt0ld marked this pull request as ready for review September 16, 2023 02:58
@github-actions
Copy link
Contributor

github-actions bot commented Sep 16, 2023

Coverage report

The coverage rate went from 75.9% to 75.61% ⬇️
The branch rate is 62%.

None of the new lines are part of the tested code. Therefore, there is no coverage data about them.

@chanhosuh
Copy link
Member

As discussed, let's put this on hold until we have further clarity on how this stuff will work.

@allt0ld allt0ld marked this pull request as draft September 21, 2023 13:04
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.

Initialize pool values from vyper contract storage during tests
2 participants