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

feat: tx simulator #1613

Merged
merged 18 commits into from
Apr 20, 2023
Merged

feat: tx simulator #1613

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
b64a530
tx simulator prototype
cmwaters Apr 11, 2023
e6d43ac
Merge branch 'main' into cal/fuzz
cmwaters Apr 11, 2023
9f9ef76
clean up logs
cmwaters Apr 11, 2023
fb36328
Merge branch 'cal/fuzz' of github.com:celestiaorg/celestia-app into c…
cmwaters Apr 11, 2023
7ef5695
fix up the data race (I think)
cmwaters Apr 12, 2023
5482401
result to ignoring race failure
cmwaters Apr 12, 2023
60e53ca
address PR comments
cmwaters Apr 13, 2023
5cf229f
add some concurrency protection to the map
cmwaters Apr 13, 2023
e193524
Merge branch 'main' into cal/fuzz
cmwaters Apr 14, 2023
3407ec2
Merge branch 'main' into cal/fuzz
cmwaters Apr 17, 2023
846d4b1
allow sequences to dictate gas
cmwaters Apr 18, 2023
b04537f
Merge branch 'main' into cal/fuzz
cmwaters Apr 18, 2023
9bc19b3
lint
cmwaters Apr 18, 2023
304cf78
fix estimation tests
cmwaters Apr 18, 2023
f44c569
Merge branch 'cal/fuzz' of github.com:celestiaorg/celestia-app into c…
cmwaters Apr 18, 2023
34f12c9
fix node test
cmwaters Apr 18, 2023
6373b93
Merge branch 'main' into cal/fuzz
cmwaters Apr 19, 2023
f3d229f
Merge branch 'main' into cal/fuzz
cmwaters Apr 20, 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
2 changes: 1 addition & 1 deletion app/estimate_square_size.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ func estimateSquareSize(normalTxs [][]byte, blobTxs []core.BlobTx) (squareSize u
// estimate as much
totalSharesUsed := uint64(txSharesUsed + pfbTxSharesUsed + blobSharesUsed)
totalSharesUsed *= 2
minSize := uint64(math.Sqrt(float64(totalSharesUsed)))
minSize := uint64(math.Ceil(math.Sqrt(float64(totalSharesUsed))))
squareSize = shares.RoundUpPowerOfTwo(minSize)
if squareSize >= appconsts.DefaultMaxSquareSize {
squareSize = appconsts.DefaultMaxSquareSize
Expand Down
6 changes: 3 additions & 3 deletions app/estimate_square_size_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,12 @@ func Test_estimateSquareSize(t *testing.T) {
}
tests := []test{
{"empty block", 0, 0, 0, appconsts.DefaultMinSquareSize},
{"one normal tx", 1, 0, 0, appconsts.DefaultMinSquareSize},
{"one normal tx", 1, 0, 0, 2},
{"one small pfb small block", 0, 1, 100, 2},
{"mixed small block", 10, 12, 500, 16},
{"small block 2", 0, 12, 1000, 16},
{"mixed medium block 2", 10, 20, 10000, 32},
{"one large pfb large block", 0, 1, 1000000, 64},
{"one large pfb large block", 0, 1, 1000000, appconsts.DefaultMaxSquareSize},
{"one hundred large pfb large block", 0, 100, 100000, appconsts.DefaultMaxSquareSize},
{"one hundred large pfb medium block", 100, 100, 100000, appconsts.DefaultMaxSquareSize},
{"mixed transactions large block", 100, 100, 100000, appconsts.DefaultMaxSquareSize},
Expand Down Expand Up @@ -75,7 +75,7 @@ func Test_estimateSquareSize_MultiBlob(t *testing.T) {
func() [][]int {
return blobfactory.Repeat([]int{1000}, 10)
},
8, 8,
16, 8,
},
{
"10 multiblob 4 share transactions",
Expand Down
Loading