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

Proof queue #2147

Closed
wants to merge 8 commits into from
Closed

Proof queue #2147

wants to merge 8 commits into from

Conversation

h0nzZik
Copy link
Contributor

@h0nzZik h0nzZik commented Oct 31, 2023

We add a queue containing proof tasks; a worker process, whenever is ready, will read a proof task from the queue and calls the prover on it. This mechanism is disabled when there is only one proof task to perform.

Subsumes #2094

@h0nzZik h0nzZik changed the title [TEST][DRAFT] Proof queue Nov 8, 2023
@h0nzZik h0nzZik mentioned this pull request Nov 8, 2023
@h0nzZik h0nzZik marked this pull request as ready for review November 8, 2023 17:50
KEVM_PYK_DIR := ./kevm-pyk
KEVM_PYK_DIR := $(abspath ./kevm-pyk)
Copy link
Member

Choose a reason for hiding this comment

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

Why is this change needed? Also, why is poetry.lock updated? These changes should probably be backed out, should only be updated by the auto-updater.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My version of Poetry (I think it is 1.6.1) complained about a relative path and this fixed it. I will try reverting poetry.lock and see whether it still works.


def get_a_ready_id() -> str | None:
for job_label in remaining_job_labels:
if set({d.claim.label for d in id_to_job[job_label].dependencies}).issubset(finished_job_labels.keys()):
Copy link
Member

Choose a reason for hiding this comment

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

What if the dependency was finished in a prior job, and we just need to read from disk that it's the case? Will this catch that? It seems here that it will only work if it was proved in this session, not if it was proved in a prior session. Is that the case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't think about that; you are probably right.

@ehildenb
Copy link
Member

CLosing due to age.

@ehildenb ehildenb closed this Jun 25, 2024
@ehildenb ehildenb deleted the proof-queue-3 branch June 25, 2024 15:43
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.

2 participants