-
Notifications
You must be signed in to change notification settings - Fork 0
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
Simplify FHEPayment #189
Comments
The idea is to:
|
This is almost surely impossible, since both TFHEExecutor and FHEPayment are already just below the smart contract size limit. So we need two separate contracts here, unfortunately. |
Ok ok so good for FHEGasMonitoring.sol 👍 |
And also we'll remove payment library from fhevm |
Specified in #198 |
The question is should we use this contract only for FHEGas price? This is still unclear since we are waiting ZWS to decide on how to handle Payment for FHE operations. If it is totaally offchain we might have issues with composability if some handles point to nothing.
The text was updated successfully, but these errors were encountered: