Skip to content

support instance scalar#16

Closed
hero78119 wants to merge 1 commit intoscroll-tech:chore/sw_curve_defaultfrom
hero78119:chore/sw_curve_default
Closed

support instance scalar#16
hero78119 wants to merge 1 commit intoscroll-tech:chore/sw_curve_defaultfrom
hero78119:chore/sw_curve_default

Conversation

@hero78119
Copy link
Collaborator

@hero78119 hero78119 closed this Nov 2, 2025
@hero78119
Copy link
Collaborator Author

This change is improper as instance scalar is a scalar, e.g. like challenge. So it should not be put in ScaledSum x * a + b x part. The right way is to extend

Expression::InstanceScalar(E::BaseField, Instance) // with a extra scalar to defined its sign

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.

1 participant