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

Router: lazy compute prices for cold mints #10

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

Conversation

farnyser
Copy link
Collaborator

@farnyser farnyser commented Oct 2, 2024

No description provided.

@farnyser farnyser force-pushed the serge/lazy-cached-prices-for-cold-mints branch from 47bc7a7 to 6ee0ae7 Compare October 7, 2024 12:22
@farnyser
Copy link
Collaborator Author

farnyser commented Oct 8, 2024

@godmodegalactus added a new commit after your approval, make senses to me to only update edges that match quote request direction. Wdyt ?

@farnyser farnyser changed the title DRAFT - Router: lazy compute prices for cold mints Router: lazy compute prices for cold mints Oct 8, 2024
@godmodegalactus
Copy link
Contributor

LGTM

@farnyser farnyser marked this pull request as draft October 9, 2024 09:47
@farnyser
Copy link
Collaborator Author

farnyser commented Oct 9, 2024

do not merge, we also need to recompute the pruned edges when getting out of the is_dirty state

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