Open mskrzypkows opened 6 months ago
Wasn't there a unit test for this before? Could you add one for future proofing?
I don't see any tests for FieldVar
. I'll add some.
@mmagician What do you think, can we merge it?
It's currently failing due to recently introduced breaking changes in nightly
. Unfortunately there's no automatic fix for these (yet, see WIP), so removing the redundant imports needs to be done manually for now.
@mmagician, @Pratyush removed unused imports, should pass now.
🤔 Something changed in algebra with ark_std::sync
?
Can we rerun failing no_std
check?
Description
Fixed bug in Field_var::double_in_place
closes: #XXXX
Before we can merge this PR, please make sure that all the following items have been checked off. If any of the checklist items are not applicable, please leave them but write a little note why.
Pending
section inCHANGELOG.md
Files changed
in the Github PR explorer