Skip to content

Refactor DryRunExecutor #45

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

Merged
merged 82 commits into from
Jan 17, 2023
Merged

Refactor DryRunExecutor #45

Show file tree
Hide file tree
Changes from 75 commits
Commits
Show all changes
82 commits
Select commit Hold shift + click to select a range
4b78ce2
with cost no longer coming back, and the coming need to support simul…
Dec 22, 2022
d28308b
remove error prone method Invariant.inputs_and_invariants
Dec 22, 2022
f6f301a
need to tranform via `Invariant.as_invariants()`
Dec 22, 2022
026926f
more control over validation and whether a method selector is provide…
Dec 24, 2022
6ed2789
deprecate the majority of the original remaning dryrun.py
Dec 24, 2022
df452d5
small typing fix + EncodingType
Dec 24, 2022
fa09d15
type ignores
Dec 26, 2022
26d47e3
Small type fix (#43)
tzaffi Dec 26, 2022
b92911c
changelog
Dec 27, 2022
9b18a63
good place to commit as passing tests (except one error in mypy)
Dec 30, 2022
a573529
another good place to stop
Dec 30, 2022
b3e2ce3
DryRunInspector typing
Dec 30, 2022
9499253
passing
Dec 30, 2022
34801b3
green stop
Dec 30, 2022
9dcc532
orphan `execute_one_dryryrun` while still passing
Dec 30, 2022
9ea7247
refactor orphans dryrun_logicsig()
Dec 31, 2022
e312964
DryRunTranaactionParams.for_app()
Dec 31, 2022
dccdc74
Refactor orphans DryRunExecution.dryrun_app()
Dec 31, 2022
37c1e5b
Refactor orphans DryRunExecutor.dryrun_logicsig_on_sequence()
Dec 31, 2022
1cee24c
refactor orphans DryRunExecutor.dryrun_app_on_sequence()
Dec 31, 2022
854089b
allow maps as well as list for multiple inputs
Dec 31, 2022
ee89949
Refactor orphans dryrun_app_pair_on_sequence
Dec 31, 2022
e67f692
deprecated section
Dec 31, 2022
faf824c
deprecated_ prefix
Dec 31, 2022
b015ec3
deprecated_ prefix
Dec 31, 2022
e69ef17
remove deprecated
Dec 31, 2022
c032a16
convenience methods run_one and run_sequence
Dec 31, 2022
96bb5fd
v 0.7.0
Dec 31, 2022
1ec5213
remove unused DryRunnable
Dec 31, 2022
a6277fc
trim stale comments
Dec 31, 2022
9d2d4ec
better typing in csv_report()
Dec 31, 2022
a9f9984
prefer run_one/sequnce over bare run()
Dec 31, 2022
0818c16
prefer run_one/sequnce over bare run()
Dec 31, 2022
9e92339
Merge remote-tracking branch 'algorand/api-tightening' into exec-mult…
Dec 31, 2022
ca1a454
prefer run_one/sequence over plain run()
Dec 31, 2022
8b9c6c9
changelog
Dec 31, 2022
403278b
notebook as well
Jan 1, 2023
95f767c
addressing #38 mostly
Jan 1, 2023
87152ac
addressing #38 mostly
Jan 1, 2023
a68208e
test CI on py 3.11 as well
Jan 1, 2023
bb99b2d
Update CHANGELOG.md
tzaffi Jan 3, 2023
4015202
Update graviton/blackbox.py
tzaffi Jan 3, 2023
7aed927
remove todo's
Jan 3, 2023
6a82ebb
Merge branch 'main' into exec-multi-dispatch
Jan 3, 2023
cceea26
fix bad merge
Jan 3, 2023
4d4f869
Update CHANGELOG.md
tzaffi Jan 3, 2023
6f8ba8b
version bump
Jan 3, 2023
73a0c78
Test against Py 11 + Reordering of Classes in blackbox.py
Jan 12, 2023
ba1e0ea
break up into inspectory.py
Jan 12, 2023
d4c8c11
finish refactor?
Jan 12, 2023
a6ac0b2
remove commentary
Jan 12, 2023
a264ec7
refer to 48 in changelog
Jan 12, 2023
8d2fa35
Merge branch 'prelude-to-45' into exec-multi-dispatch
Jan 12, 2023
eca2e51
merge in #48
Jan 13, 2023
5e30838
fix bad merge
Jan 13, 2023
8c87d53
Generic `OneOrMany[T]` and privates run
Jan 13, 2023
c08adcc
Update graviton/blackbox.py
tzaffi Jan 13, 2023
14c7d04
Update graviton/blackbox.py
tzaffi Jan 13, 2023
6e993bf
lint
Jan 13, 2023
53f44f1
run the wwinder dockerized algod
Jan 13, 2023
55b0572
faster better integration test environment
Jan 13, 2023
23615c1
better build
Jan 13, 2023
d4fa910
Merge branch 'prelude-to-45' into exec-multi-dispatch
Jan 13, 2023
5752e84
better build
Jan 13, 2023
4ef8ec8
Merge branch 'prelude-to-45' into exec-multi-dispatch
Jan 13, 2023
0fed894
Update .github/workflows/build.yml
tzaffi Jan 13, 2023
f4cb00f
Merge branch 'prelude-to-45' into exec-multi-dispatch
Jan 13, 2023
1da27b1
better build
Jan 13, 2023
41c4b09
Merge branch 'prelude-to-45' into exec-multi-dispatch
Jan 13, 2023
6d8a7e6
Merge branch 'algorand/main' into exec-multi-dispatch
Jan 13, 2023
ec3e059
Apply suggestions from code review
tzaffi Jan 13, 2023
c83018d
Update CHANGELOG.md
tzaffi Jan 13, 2023
ec2446c
changelog
Jan 13, 2023
4ca02c2
changelog
Jan 13, 2023
96a5e28
addressed comment about untested pathway
Jan 17, 2023
4dfdcde
Update tests/unit/encode_test.py
tzaffi Jan 17, 2023
4c93051
Update tests/unit/encode_test.py
tzaffi Jan 17, 2023
17fd8ac
improve after CR feedback
Jan 17, 2023
a316388
per CR: No need for `Optional[EncodingType]` as it already unions in …
Jan 17, 2023
0f7a363
rename obsolete objects in commentary
Jan 17, 2023
203e196
improve per CR observation
Jan 17, 2023
97e2394
incorporate suggestion in CR
Jan 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,31 @@
<!-- markdownlint-disable MD024 -->
# Changelog


## `v0.8.0` (_aka_ 🦛)

### Breaking changes

In [#48](https://github.com/algorand/graviton/pull/48), the following classes were pulled out of `graviton/blackbox.py` and into a new file `graviton/inspector.py`
* `TealVal`
* `DryRunResults` (renamed from `BlackboxResults`)
* `DryRunInspector`

In [#44](https://github.com/algorand/graviton/pull/44):
* `DryRunExecutor` has been refactored. All the class methods `dryrun_*` have been removed in favor of the instance methods `run_one()` and `run_sequence()`. In particular, it is now required to instantiate a `DryRunExecutor` object before calling a dry run executing method.
* `DryRunInspector` no longer accepts `args` as a second parameter and uses `self.args` instead.
* Migration path to the above: it is recommended that calls be re-written to use the new API. If you find that this causes too much friction, please open an issue.
### Added
* Adding `budget_added` and `budget_consumed` to `DryRunInspector.csv_row()`

### Addressed
* [#45](https://github.com/algorand/graviton/pull/45): addresses [#38](https://github.com/algorand/graviton/issues/38) and [#40](https://github.com/algorand/graviton/issues/40)
* [#48](https://github.com/algorand/graviton/pull/48): addresses [#14](https://github.com/algorand/graviton/issues/14) and [#29](https://github.com/algorand/graviton/issues/29)

## `v0.7.1`

### Breaking changes

* Upgrade to `py-algorand-sdk` v2 ([#46](https://github.com/algorand/graviton/pull/46))

## `v0.7.0` (_aka_ 🦒)
Expand Down
Loading