Skip to content

log compoisition polynomial degree bound#1323

Closed
ilyalesokhin-starkware wants to merge 1 commit intodevfrom
ilya/all_preprocessed
Closed

log compoisition polynomial degree bound#1323
ilyalesokhin-starkware wants to merge 1 commit intodevfrom
ilya/all_preprocessed

Conversation

@ilyalesokhin-starkware
Copy link
Collaborator

log compoisition polynomial degree bound

renamed stwo-prover to stwo

removed artifacts from bad rebase

removed mask module

hashbrown instead of std hashmap

stwo-air-utils to take stwo with prover feature

downgrade hashbrown version

no std verifier

Add remainder length to poseidon hash_node (#1166)

stop using assert_matches

no longet using is_empty on ExactSizedIterators

stop using array windows unstable feature

no longer using iter_array_chunks in non-prover builds

stop using unstable remove_try_from_fn_array

feature flagged porable_imd in constraint-framework

stable rust ci tests for no-prover tests

relaxed cargo lint denies

rfft throughput benchmark

fixed pow performance bug

fixed pow bug

fixed doc for MekleProver 'commit'

added typos check in CI

Bump_rust_toolchain (#1187)

Rename preproccessed into preprocessed (#1189)

Update pow_bits in default PcsConfig (#1192)

Use poseidon_hash2. (#1194)

Add length pad to poseidon mix_u32s.

Length Padding Function

Domain Seperation in Blake

domain separation in poseidon252 channel (#1199)

Split PoW verification from nonce mixing. (#1197)

Remove ChannelTime. (#1200)

Change n_draws from usize to u32. (#1202)

Simplify accumulate_row_quotients. (#1201)

Add domain separation to Blake2sMerkleHasher. (#1203)

Add draw_u32s and remove draw_random_bytes. (#1204)

Add zeros in the middle for blake pow verification. (#1205)

Add split_at_mid to Column (#1208)

grind(): Make fields inside hash 16-byte aligned. (#1215)

Add CirclePoly split_at_mid (#1223)

SecurePoly split_at_mid (#1224)

Split composition poly into 2 (#1225)

Add unsorted_query_locations to prover output. (#1226)

Add Blake2sM31Channel. (#1219)

Add ExtendedFriProof with a map of all FRI values involved in the proof. (#1228)

commit-id:aaae180e

Add traits for lifted vcs. (#1220)

Implement MerkleHasherLifted for Blake. (#1221)

Add Merkle prover in bit reverse. (#1227)

Add Merkle tree node infromation to the proof. (#1229)

commit-id:9e162d13

Evaluation By Folding

Evaluation by Folding Bench

Poly Struct for Storing Trace Polynomials

change variable name (#1239)

Store Trace Polys in Poly Struct

Rename CirclePoly -> CircleCoefficients

Skip layers with no samples. (#1236)

Boxed Air Utils

Boxed Air Utils Derive

Barycentric Evaluation

Remove unneccesary feature cfg. (#1265)

Calculate Barycentric Weights Only When Necessary (#1279)

Change alerting target. (#1268)

Fix non-parallel bug and modify CI. (#1290)

Currently the CI misses the stwo tests with feature prover enabled but
without parallel.

Made clippy script more extensive + fixed issue (#1295)

Turns out a per-crate clippy catches issues that are not caught with a workspace level clippy (due to features sets differences)

Cleaned up some clippy allows (#1298)

Add column log size print (#1303)

Parallelize fft over columns (#1304)

Parallel denominator_inverses in simd quotients (#1305)

Removed unused file (#1299)

Parallelize OOD evaluation. (#1306)

Merge lifted into dev. (#1307)

All the code related to the previous architecture of quotient ops has
been removed.

Co-authored-by: Alon F alonf@starkware.co

These changes make it easier to integrate with stwo_cairo_prover.


Co-authored-by: Alon F alonf@starkware.co

Fix names in test. (#1309)

Add FrameworkComponent.is_enabled. (#1308)

Add serde for auxialiary proof types. (#1310)

Updated documentation. (#1313)

Bump version to 2.0.0. (#1314)

Fix workspace deps. (#1315)

Add crates description. (#1316)

Update LookupElements::dummy. (#1319)

make BaseColumn::from_cpu run without taking ownership over input (#1318)

Change pcs config mixing (#1320)

Add an option to include all the preprocessed columns in the proof.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.02%. Comparing base (b0ba0c4) to head (d2c2182).

Files with missing lines Patch % Lines
crates/stwo/src/core/air/components.rs 62.50% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1323      +/-   ##
==========================================
- Coverage   92.04%   92.02%   -0.02%     
==========================================
  Files         119      119              
  Lines       15136    15140       +4     
  Branches    15136    15140       +4     
==========================================
+ Hits        13932    13933       +1     
- Misses       1115     1117       +2     
- Partials       89       90       +1     

☔ 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.

Copy link
Contributor

@leo-starkware leo-starkware left a comment

Choose a reason for hiding this comment

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

@leo-starkware made 1 comment.
Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @ilyalesokhin-starkware).


crates/stwo/src/core/air/components.rs line 16 at r1 (raw file):

    pub components: Vec<&'a dyn Component>,
    pub n_preprocessed_columns: usize,
    /// Whether `maks_points` should include all preprocessed columns, regardles of usage.

s

Code quote:

regardles

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.

4 participants