Skip to content

perf: bitpack encoding for LID and MID blocks (draft)#327

Closed
cheb0 wants to merge 1 commit intomainfrom
318-bitpack-encoding
Closed

perf: bitpack encoding for LID and MID blocks (draft)#327
cheb0 wants to merge 1 commit intomainfrom
318-bitpack-encoding

Conversation

@cheb0
Copy link
Member

@cheb0 cheb0 commented Jan 26, 2026

Description

Fixes #317


  • I have read and followed all requirements in CONTRIBUTING.md;
  • I used LLM/AI assistance to make this pull request;

If you have used LLM/AI assistance please provide model name and full prompt:

Model: {{model-name}}
Prompt: {{prompt}}

@cheb0
Copy link
Member Author

cheb0 commented Jan 26, 2026

@seqbenchbot up search-keyword-exact-match

@seqbenchbot
Copy link

seqbenchbot commented Jan 26, 2026

Oh-oh, @cheb0 ('>o.o)'>!

Something went wrong and I couldn't process your request.
Please take a closer look at error message:

expected "<scenario>"

@cheb0
Copy link
Member Author

cheb0 commented Jan 26, 2026

@seqbenchbot up main search-keyword-exact-match

@seqbenchbot
Copy link

seqbenchbot commented Jan 26, 2026

Nice, @cheb0 <(-^,^-)=b!

Your request was successfully served.
Identificator for your ongoing benchmark - af3bd0e0.

Here is a list of helpful links:

  • Take a look at Grafana dashboard;
  • Live-tailing logs are also available;

Have a great time!

@github-actions
Copy link
Contributor

🔴 Performance Degradation

Some benchmarks have degraded compared to the previous run.
Click on Show table button to see full list of degraded benchmarks.

Show table
Name Previous Current Ratio Verdict
Block_Pack-4 ------ 1460f0
NaN B/op 1144.00 B/op NaN 🔴
NaN allocs/op 8.00 allocs/op NaN 🔴
NaN ns/op 1459.00 ns/op NaN 🔴
Block_Unpack-4 ------ 1460f0
NaN B/op 19006.00 B/op NaN 🔴
NaN allocs/op 3.00 allocs/op NaN 🔴
NaN ns/op 6432.00 ns/op NaN 🔴
ProcessDocuments-4 e2e0db 1460f0
381.00 B/op 33575.00 B/op 88.12 🔴
4.00 allocs/op 5.00 allocs/op 1.25 🔴
Sealing_NoSort-4 e2e0db 1460f0
24848384.00 B/op 71103536.00 B/op 2.86 🔴
5903.00 allocs/op 19957.00 allocs/op 3.38 🔴
Sealing_WithSort-4 e2e0db 1460f0
42308296.00 B/op 87612088.00 B/op 2.07 🔴
5994.00 allocs/op 20015.00 allocs/op 3.34 🔴

@codecov-commenter
Copy link

codecov-commenter commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 68.08511% with 90 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.53%. Comparing base (a81b353) to head (d73e08a).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
frac/sealed/lids/block2.go 0.00% 44 Missing ⚠️
packer/bitpack_unpacker.go 54.44% 32 Missing and 9 partials ⚠️
packer/bitpack_packer.go 88.57% 2 Missing and 2 partials ⚠️
frac/sealed/seqids/loader.go 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #327      +/-   ##
==========================================
- Coverage   71.56%   71.53%   -0.04%     
==========================================
  Files         204      207       +3     
  Lines       14763    15013     +250     
==========================================
+ Hits        10565    10739     +174     
- Misses       3442     3512      +70     
- Partials      756      762       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cheb0
Copy link
Member Author

cheb0 commented Jan 26, 2026

@seqbenchbot down af3bd0e0

@seqbenchbot
Copy link

seqbenchbot commented Jan 26, 2026

Nice, @cheb0 <(-^,^-)=b!

Your request was successfully served.
The benchmark with identificator af3bd0e0 was stopped.

Have a great time!

@cheb0 cheb0 closed this Jan 26, 2026
@cheb0 cheb0 deleted the 318-bitpack-encoding branch January 26, 2026 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants