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

Prover: beta v1.2 #605

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

Prover: beta v1.2 #605

wants to merge 27 commits into from

Conversation

gusiri
Copy link
Contributor

@gusiri gusiri commented Jan 27, 2025

This PR implements issue(s) #

Checklist

  • I wrote new tests for my new core changes.
  • I have successfully ran tests, style checker and build against my new changes locally.
  • I have informed the team of any breaking changes if there are any.

@gusiri gusiri had a problem deploying to docker-build-and-e2e January 27, 2025 07:06 — with GitHub Actions Error
@codecov-commenter
Copy link

codecov-commenter commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.88%. Comparing base (253a51a) to head (0ce3326).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #605      +/-   ##
============================================
+ Coverage     68.39%   68.88%   +0.49%     
- Complexity     1181     1186       +5     
============================================
  Files           323      324       +1     
  Lines         13125    13117       -8     
  Branches       1316     1316              
============================================
+ Hits           8977     9036      +59     
+ Misses         3595     3532      -63     
+ Partials        553      549       -4     
Flag Coverage Δ *Carryforward flag
hardhat 98.94% <ø> (ø)
kotlin 66.54% <ø> (+0.52%) ⬆️ Carriedforward from 28996d4

*This pull request uses carry forward flags. Click here to find out more.

see 17 files with indirect coverage changes

@gusiri gusiri had a problem deploying to docker-build-and-e2e January 27, 2025 08:26 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 27, 2025 09:05 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 27, 2025 17:35 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 27, 2025 17:39 — with GitHub Actions Error
* Revert "adjusted size for corset columns"

This reverts commit b1a7319.

* fixed bug and added panic message for a non power of two size column

* removing panic

* reinsteaded the panic

---------

Co-authored-by: gusiri <[email protected]>
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 29, 2025 11:54 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 29, 2025 11:57 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 29, 2025 14:17 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 29, 2025 15:30 — with GitHub Actions Error
@gusiri gusiri temporarily deployed to docker-build-and-e2e January 29, 2025 16:36 — with GitHub Actions Inactive
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 30, 2025 08:36 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 30, 2025 09:42 — with GitHub Actions Error
@gusiri gusiri had a problem deploying to docker-build-and-e2e January 30, 2025 11:06 — with GitHub Actions Error
@gusiri gusiri temporarily deployed to docker-build-and-e2e January 31, 2025 08:39 — with GitHub Actions Inactive
@gusiri gusiri temporarily deployed to docker-build-and-e2e January 31, 2025 08:51 — with GitHub Actions Inactive
@gusiri gusiri marked this pull request as ready for review January 31, 2025 09:31
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