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.

Sign up for GitHub

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

Jump to bottom

Rename variables #58

Open
Jiangkm3 opened this issue Dec 12, 2024 · 2 comments
Open

Rename variables #58

Jiangkm3 opened this issue Dec 12, 2024 · 2 comments
Assignees
@Jiangkm3
Labels
refactor
Milestone
Select the optima...

Comments

@Jiangkm3
Copy link
Collaborator

Jiangkm3 commented Dec 12, 2024 •
edited
Loading

Rename the variables to avoid confusion:
num_instances -> num_circuits
num_proofs -> num_instances
@matthiasgoergens
Copy link
Collaborator

matthiasgoergens commented Dec 13, 2024 •
edited
Loading

You can just do the renames, it's fairly easy to do with something like vscode. (I can show you how.) I'm happy to approve the PRs quickly.

@kunxian-xia kunxian-xia moved this to Todo in Ceno - Recursion Dec 13, 2024
@kunxian-xia
Copy link
Collaborator

@Jiangkm3 Let's include this as one of the sub PRs in the giant PR #17.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Jiangkm3 Jiangkm3

Labels
refactor
Projects
Ceno - Recursion
Status: Todo
Milestone
Select the optimal proof system for r...
Development

No branches or pull requests
3 participants
@matthiasgoergens @kunxian-xia @Jiangkm3