-
Notifications
You must be signed in to change notification settings - Fork 9
describe provable assertions #123
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
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
1a21e2d
State before partial rewrite
nikeshnazareth 57bccc7
Simplify discussion about transaction ordering
nikeshnazareth 5811b92
Describe the same-slot mechanism
nikeshnazareth 7467e25
Describe Realtime L1 reads
nikeshnazareth de9a1d0
Describe interdependent transactions
nikeshnazareth 40f06b6
Describe multi-block unsecured loan
nikeshnazareth 5526d7b
Describe cross-rollup setting
nikeshnazareth 1c7fad9
Describe realtime proving
nikeshnazareth 8a6fda1
Describe staked claim
nikeshnazareth 0137672
Fix typo
nikeshnazareth cc4e70c
Fix typo
nikeshnazareth 15c872b
Update diagrams to better package the rollup publication
nikeshnazareth 93742dd
Several minor clarifications
nikeshnazareth 51789ce
Describe sub publication proving
nikeshnazareth cb02c44
Describe implementation framework
nikeshnazareth ace8530
Update Realtime section based on feedback
nikeshnazareth ae10e47
Fix typos
nikeshnazareth 3059dfa
Emphasise the transcript-level conditions
nikeshnazareth 5513a90
Add one-sided caveat to the first sequence it shows up
nikeshnazareth 62a5406
Use "pauser" instead of "asserter" mechanism.
nikeshnazareth 72ed375
Better explain the Prisoner's Dilemma example.
nikeshnazareth aef20d5
Describe the `value` for the assertions.
nikeshnazareth 08cb83d
Create example walkthrough
nikeshnazareth 97ab800
Fix images
nikeshnazareth efe8b36
Make heading sizes consistent
nikeshnazareth 03db32f
Describe the open abstraction caveat
nikeshnazareth f85e873
Describe the one-line idea after explaining the pattern
nikeshnazareth a7dd3dd
Use "explained" instead of "noted" to avoid repetition
nikeshnazareth 267d3a9
Rephrase cross-rollup setting to mention compatibility with the agg-l…
nikeshnazareth a02fd5b
Explain how the posted state is useful for AggLayer
nikeshnazareth 3b39a47
Explain that the staked claim should not be used for the native bridge
nikeshnazareth 773c1c9
Add introductory clause to implementation section
nikeshnazareth 147798c
Include caveat that walkthrough may be overkill in practice
nikeshnazareth 74107ae
Include acknowledgement
nikeshnazareth 25c6118
Fix typo
nikeshnazareth ffbc928
Describe simplified version of prisoners dilemma solution
nikeshnazareth b43c1e6
Grammar fix
nikeshnazareth a5c1aea
Fix typo
nikeshnazareth File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| name="provable_assertion_images"; | ||
|
|
||
| for i in {0..19} | ||
| do | ||
| pdflatex "\def\sourceNo{$i}\input{$name.tex}"; | ||
| convert -density 600x600 $name.pdf -quality 90 $name.$i.png; | ||
| done | ||
|
|
||
| rm $name.aux $name.log $name.pdf | ||
|
Comment on lines
+3
to
+11
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Add dependency checking. The script should verify required tools are available before attempting to use them. name="provable_assertion_images";
+# Check for required dependencies
+for cmd in pdflatex convert; do
+ if ! command -v "$cmd" &> /dev/null; then
+ echo "Error: $cmd is not installed or not in PATH"
+ exit 1
+ fi
+done
+
for i in {0..19}🤖 Prompt for AI Agents |
||
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Add error handling for compilation failures.
The script doesn't handle compilation errors, which could leave the system in an inconsistent state with partial files.
for i in {0..19} do - pdflatex "\def\sourceNo{$i}\input{$name.tex}"; - convert -density 600x600 $name.pdf -quality 90 $name.$i.png; + if ! pdflatex "\def\sourceNo{$i}\input{$name.tex}"; then + echo "Error: pdflatex failed for sourceNo=$i" + exit 1 + fi + if ! convert -density 600x600 $name.pdf -quality 90 $name.$i.png; then + echo "Error: convert failed for sourceNo=$i" + exit 1 + fi done🤖 Prompt for AI Agents