-
Notifications
You must be signed in to change notification settings - Fork 20
added Ranger repo #395
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
base: master
Are you sure you want to change the base?
added Ranger repo #395
Conversation
| ## 3. Run Ranger | ||
| Use the certoraRanger command to launch the job: | ||
|
|
||
| ```bash | ||
| certoraRanger ranger.conf | ||
| ``` | ||
|
|
||
| This will start the Ranger process. A link to the Ranger Job Report in the dashboard will be pasted into your command line when the job is submitted. |
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.
Do we want to mention somewhere that ranger runs lots of processes/is compute intensive and suggest that users use it with --rule in order to test only one rule/invariant, or use --split_rules "*" so each rule will run in a separate job?
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.
I've added a new warning with this info. IDK if it fits a warning or an info message
naftali-g
left a comment
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.
looks ok to me
* start * Added CLI docs * Fixing ref * spelling * CR * Update docs/ranger/ranger_client.md Co-authored-by: urikirsh <[email protected]> * Removed "usage" section and commented range_failure_limit attribute until decided otherwise. * fixed attribute typo * How Ranger Works (#398) * how ranger works * Uri's CR --------- Co-authored-by: urikirsh <[email protected]> Co-authored-by: Naftali Goldstein <[email protected]>
johspaeth
left a comment
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.
I have some tiny request for changes, overall looks good! Thanks guys.
- We should state that Ranger operates on CVL early on.
- We should document the
setupfunction.
docs/ranger/ranger_client.md
Outdated
|
|
||
| [//]: <> ((--ranger_failure_limit)=) | ||
|
|
||
| [### `ranger_failure_limit` |
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.
| [### `ranger_failure_limit` | |
| ### `ranger_failure_limit` |
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.
It is commented on purpose
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.
Ok, it currently just renders strangely on https://certora-certora-prover-documentation--395.com.readthedocs.build/en/395/docs/ranger/ranger_client.html
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.
I see, thanks!
I tried different ways of commenting out lines in markdown, none worked, I've removed the lines entirely
docs/ranger/how_ranger_works.md
Outdated
| # How Ranger Works | ||
|
|
||
| **Ranger** is a bounded model checker. This means that, in contrast with "full" | ||
| formal verification, its initial state isn't arbitrary, but is instead reached |
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.
| formal verification, its initial state isn't arbitrary, but is instead reached | |
| formal verification, its initial state is not arbitrary, but is instead reached |
index.rst
Outdated
| * :doc:`docs/sunbeam/index` -- instructions for installing and using *Certora Sunbeam* | ||
| for formal verification of `Soroban`_ contracts. | ||
| * :doc:`docs/solana/index` -- instructions for installing and using *Certora Solana Prover* | ||
| * :doc:`docs/ranger/index` -- TODO |
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.
| * :doc:`docs/ranger/index` -- TODO | |
| * :doc:`docs/ranger/index` -- explains how to use Ranger, Certora's Bounded Model Checker. |
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.
Good catch!
docs/ranger/ranger_intro.md
Outdated
| - ✅ **Realistic counterexamples**: All violations are reachable from the initial state. | ||
| - ✅ **Faster time to value**: Minimal setup required to get useful results. | ||
| - ✅ **Fewer false positives**: No need to precondition rules to filter out invalid states. | ||
| - ✅ **High coverage**: Symbolically tests all function call sequences from the initial state up to a certain range. |
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.
I'd rephrase this slightly.
| - ✅ **High coverage**: Symbolically tests all function call sequences from the initial state up to a certain range. | |
| - ✅ **High coverage**: Tests all function call sequences on symbolic inputs from the initial state up to a certain range. |
|
|
||
| This guide will help you run your first Ranger job using a Solidity contract and a [CVL](/docs/cvl/index) [invariant](/docs/cvl/invariants). | ||
|
|
||
| Ranger uses the same installation process, configuration files, and spec files as the [Certora Prover](/docs/user-guide/index). If you're already familiar with the Prover, getting started with Ranger will feel familiar. |
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.
That Ranger uses CVL could be stated also ranger_intro?
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.
Done
Adding documentation for ranger
Link to generated documentation: https://certora-certora-prover-documentation--395.com.readthedocs.build/en/395/docs/ranger/index.html