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.

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

Unite impls of DomainEvaluationAccumulator #936

Merged

Conversation

Gali-StarkWare
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

Gali-StarkWare commented Dec 17, 2024

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

Copy link
Collaborator

@shaharsamocha7 shaharsamocha7 left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ohad-starkware)

@Gali-StarkWare Gali-StarkWare force-pushed the 12-16-generic_backend_secure_power_generation branch from 7e1be59 to 3bfc747 Compare December 17, 2024 08:35
@Gali-StarkWare Gali-StarkWare force-pushed the 12-17-unite_impls_of_domainevaluationaccumulator branch 2 times, most recently from 5df95d0 to 7d9b756 Compare December 17, 2024 08:39
@codecov-commenter
Copy link

codecov-commenter commented Dec 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.46%. Comparing base (cf1eca6) to head (9eb2221).
Report is 1 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #936   +/-   ##
=======================================
  Coverage   91.46%   91.46%           
=======================================
  Files          99       99           
  Lines       13818    13818           
  Branches    13818    13818           
=======================================
  Hits        12638    12638           
  Misses       1065     1065           
  Partials      115      115           

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

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: d0c8038 Previous: 7d9b756 Ratio
iffts/simd ifft/21 6175818 ns/iter (± 117800) 3057935 ns/iter (± 49649) 2.02

This comment was automatically generated by workflow using github-action-benchmark.

CC: @shaharsamocha7

Copy link
Collaborator

@shaharsamocha7 shaharsamocha7 left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ohad-starkware)

@Gali-StarkWare Gali-StarkWare force-pushed the 12-17-unite_impls_of_domainevaluationaccumulator branch 8 times, most recently from d0c8038 to 7ba577a Compare December 17, 2024 19:41
@Gali-StarkWare Gali-StarkWare changed the base branch from 12-16-generic_backend_secure_power_generation to 12-10-add_secure_powers_generation_for_simd December 17, 2024 19:42
@Gali-StarkWare Gali-StarkWare force-pushed the 12-17-unite_impls_of_domainevaluationaccumulator branch from 7ba577a to 145bba4 Compare December 18, 2024 07:11
@Gali-StarkWare Gali-StarkWare changed the base branch from 12-10-add_secure_powers_generation_for_simd to 12-16-generic_backend_secure_power_generation December 18, 2024 07:12
@Gali-StarkWare Gali-StarkWare force-pushed the 12-16-generic_backend_secure_power_generation branch from 3bfc747 to d133010 Compare December 18, 2024 07:24
@Gali-StarkWare Gali-StarkWare force-pushed the 12-17-unite_impls_of_domainevaluationaccumulator branch from 145bba4 to 119ff86 Compare December 18, 2024 07:25
Copy link
Contributor Author

Gali-StarkWare commented Dec 18, 2024

Merge activity

  • Dec 18, 5:58 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Dec 18, 6:06 AM EST: Graphite rebased this pull request as part of a merge.
  • Dec 18, 6:09 AM EST: A user merged this pull request with Graphite.

@Gali-StarkWare Gali-StarkWare changed the base branch from 12-16-generic_backend_secure_power_generation to graphite-base/936 December 18, 2024 11:00
@Gali-StarkWare Gali-StarkWare changed the base branch from graphite-base/936 to dev December 18, 2024 11:04
@Gali-StarkWare Gali-StarkWare force-pushed the 12-17-unite_impls_of_domainevaluationaccumulator branch from 119ff86 to 9eb2221 Compare December 18, 2024 11:05
@Gali-StarkWare Gali-StarkWare merged commit af0b35f into dev Dec 18, 2024
18 of 19 checks passed
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