8000 Extend debugger interface by jdtzmn · Pull Request #4 · jdtzmn/go-algorand · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Extend debugger interface #4

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
Draft
wants to merge 426 commits into
base: psuedo-eval-endpoint
Choose a base branch
from
Draft

Extend debugger interface #4

wants to merge 426 commits into from

Conversation

jdtzmn
Copy link
Owner
@jdtzmn jdtzmn commented Aug 16, 2022

Summary

This draft PR includes an extended debugger interface which extends the existing logic.DebuggerHooks interface with additional events which will become useful within the simulator.

The PR currently points to the wrong base to make it easier to see what files have been changed since then while gathering initial reviews.

Test Plan

  • Additional tests in debugger_test.go for the new hooks
  • Test eval.TransactionGroupWithDebugger() within eval_test.go

jasonpaulos
jasonpaulos reviewed Aug 17, 2022
View reviewed changes
@jdtzmn jdtzmn requested a review from jasonpaulos August 18, 2022 19:21
jasonpaulos
jasonpaulos approved these changes Aug 19, 2022
View reviewed changes
Copy link
@jasonpaulos jasonpaulos left a comment

Choose a reason for hiding this comment

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

This looks good!

My one comment is that it would be nice to see a unit test or two for the changes in data/transactions/verify/txn.go, but that can be done in a different place before release

@jdtzmn jdtzmn mentioned this pull request Aug 19, 2022
Merged
3 tasks
michaeldiamant and others added 26 commits October 26, 2022 15:15
@michaeldiamant
Tests: Fix handling of empty local state in TestResourcesDataApp (alg…
4e36847 
…orand#4699)
@algoidurovic
AVM: match, pushints, and pushbytess opcodes (algorand#4645)
ab87a8a 
add match opcode along with assembler and eval unit tests
implementation of match, pushints, and pushbytess
add docs for pushints and pushbytess
@cce
network: align duplicateFilterCount to prevent panic on 32-bit platfo…
40974ee 
…rms (algorand#4702)
@AlgoStephenAkiki
Bug-fix: return v2 spec from swagger endpoint (algorand#4697)
e2c2bc4
@michaeldiamant
AVM: Add support for Boxes (algorand#4149)
0cb9a2e
@jasonpaulos
Merge branch 'master' into feature/simulate-endpoint
b99b80b
@jasonpaulos
Fix generate config files
ad5f515
@cce
performance: don't create child cow in TestTransactionGroup (algorand…
7602d4f 
…#4706)
@michaeldiamant
db: print dbutil.go error in stderr (algorand#4712)
fb1f498
@jasonpaulos
Merge branch 'feature/simulate-endpoint' into extend-debugger-interface
2a2b8a9
@iansuvak
performance: agreement state serialization using msgp (algorand#4644)
ba9c593
@algorandskiy
prefetcher: enable prefetcher for assets and apps (algorand#4352)
e6f396b
@jasonpaulos
Reorder debugger hook functions
115421f
@algorandskiy
test: fix after boxes and prefetcher remerge (algorand#4716)
97c232d
@tzaffi
blank import needs commenting out for indexer's lint (algorand#4720)
959fb5a
@jannotti @michaeldiamant
AVM: fix early eval exits for Debugger (algorand#4719)
bac4117 
Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com>
@onetechnical
DevOps: Update docker container in build (algorand#4723)
daca5d3
@algorandskiy
catchpoint: actualize limit constants and fix txtail table init (algo…
15efc02 
…rand#4703)
@barnjamin
tools: allow specification of RewardPoolBalance in genesis (algorand#…
a8f1b4a 
…4643)
@jannotti
Consensus: v36 upgrade (algorand#4722)
06f9109 
v36 consensus upgrade
@jasonpaulos
Merge branch 'master' into feature/simulate-endpoint
533c239
@jasonpaulos
Merge branch 'feature/simulate-endpoint' into extend-debugger-interface
1219bef
@algolucky
dispenser: add input validation (algorand#4713)
f37bb6e
@Algo-devops-service
Update the Version, BuildNumber, genesistimestamp.data
2c0dd9e
@algojohnlee
Merge pull request algorand#4730 from Algo-devops-service/relbeta3.12.0
31a1099
@algolucky
mergeback: feature/alphanet (algorand#4726)
e387553
jasonpaulos and others added 30 commits January 12, 2023 10:44
@jasonpaulos
Fix test issue
8b4384b
@jasonpaulos
Update license headers
0c43566
@jannotti
AVM: Clearer cost benchmarks, and a nice optimization of b== and b< (a…
5d38a15 
…lgorand#5010)
@jasonpaulos
minor updates from CR
54b7534
@jasonpaulos
Use assembler instead of hardcoding program bytes
5927a63
@tzaffi
goal: TEAL assembler error UX improvement (algorand#4951)
4631571 
Changed assembler functionality so that logic.assemble errors when receiving empty input
@jasonpaulos
Make tracer a field in BlockEvaluator and generalize txn group hooks
0b02d9e
@jasonpaulos
Don't add tracer to GroupContext
10e74cb
@jasonpaulos
Fix incorrect txn depth logic in debuggerEvalTracerAdaptor
c0489c7
@algolucky
container: update documentation and remove unused tools (algorand#4982)
5c17ad6
@algorandskiy
tools: box key utility (algorand#5026)
9efd9f3
@algojack
Bump version in buildnumber.dat
5c74933
@AlgoStephenAkiki
avm-abi: Update functions migrated to avm-abi library (algorand#4979)
a9fe2c7
@algorandskiy
Empty box hotfix for relstable (algorand#16)
aa42ea1
@onetechnical
Merge pull request algorand#17 from algorand/relstable3.13.3
9ddd136 
go-algorand v3.13.3-stable
@algorandskiy
fix build after algorand#4979 (algorand#5032)
e01e374
@winder
Remove unused NetworkFetcher code. (algorand#5028)
5dd909c
@brianolson
ledger: rearrange blockqueue start/stop (algorand#4964)
992b318
@algolucky
util/s3: remove extra println (algorand#5024)
1b05d17
@cce
debug: unmarshal and print transaction distribution (algorand#5008)
2aa2c91
@winder
Remove accidental devtesting output. (algorand#5033)
75e42aa
@winder
Remove topAccountListener (algorand#5027)
724633a
@brianolson
tools: pingpong total latency (algorand#4757)
67b06a4
@cce
config: add CadaverDirectory and prevent algod from starting if cadav…
b8866c9 
…er.init() fails (algorand#4740)
@onetechnical
Merge branch 'rel/stable' into relstable-3-13-3-remerge
b3dc601
@algojohnlee
Merge pull request algorand#5036 from algorand/onetechnical/relstable…
4a26591 
…-3-13-3-remerge
@algorandskiy
network: increase max incoming connections limit to 2400 (algorand#5018)
8445d67 
* Decrease connections per ip addr limit to 15
@algojack
CI: Fix PartitionTest Github Action (algorand#5006)
a6d0231 
* Updating action versions
* Removing inlining and optimizations from building partitiontest-linter
* Adding flags to golangci-lint
* Syncing typeparams dependency version
* Changing tools package version
* running go mod tidy and adding compat 1.17
* Chris's fix for linter
* Filtering on added only, upgrading reviewdog, diplaying errors in log
@michaeldiamant
Tests: Fix t.Parallel() errors in shared package (algorand#4989)
a1eaafb 
* Fix t.Parallel() errors in shared package
* Correct .golangci.yml paths
@jasonpaulos
Merge branch 'master' into extend-debugger-interface
b4e0999
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jasonpaulos jasonpaulos jasonpaulos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
0