Skip to content

Conversation

@JayWhite2357
Copy link
Contributor

@JayWhite2357 JayWhite2357 commented Mar 27, 2025

Rationale for this change

A variety of getters and setters are needed for upcoming PRs. This PR adds them all at once.

What changes are included in this PR?

The following function are added:

  • builder_get_first_round_mles
  • builder_get_final_round_mles
  • builder_get_aggregate_evaluation
  • builder_get_column_evaluations
  • builder_set_first_round_commitments
  • builder_get_first_round_commitments
  • builder_set_final_round_commitments
  • builder_get_final_round_commitments
  • builder_set_bit_distributions
  • builder_get_chi_evaluations
  • builder_get_table_chi_evaluations

Are these changes tested?

Yes

@JayWhite2357 JayWhite2357 marked this pull request as ready for review March 27, 2025 05:29
@codecov
Copy link

codecov bot commented Mar 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.25%. Comparing base (264dfa3) to head (357f7cf).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #675   +/-   ##
=======================================
  Coverage   96.25%   96.25%           
=======================================
  Files         325      325           
  Lines       55964    55964           
=======================================
  Hits        53866    53866           
  Misses       2098     2098           

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

@iajoiner iajoiner force-pushed the feat/sol/expand-verification-builder branch from ed27ee3 to 49e621f Compare March 27, 2025 14:21
@JayWhite2357 JayWhite2357 force-pushed the feat/sol/expand-verification-builder branch from 49e621f to cbef38d Compare March 27, 2025 14:39
@JayWhite2357 JayWhite2357 enabled auto-merge March 29, 2025 02:26
@JayWhite2357 JayWhite2357 force-pushed the feat/sol/expand-verification-builder branch from cbef38d to 357f7cf Compare March 29, 2025 03:08
@JayWhite2357 JayWhite2357 merged commit 06171df into main Mar 29, 2025
11 checks passed
@github-actions
Copy link

🎉 This PR is included in version 0.86.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@iajoiner iajoiner deleted the feat/sol/expand-verification-builder branch April 9, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants