search

urbit / azimuth

General-purpose PKI on Ethereum
MIT License
134 stars 36 forks source link

Remove pointsOwnedBy and related logic #55

Open calvinmccarter opened 2 years ago

calvinmccarter commented 2 years ago

This fixes #50 .

calvinmccarter commented 3 months ago

@cgyarvin This fixes #50. Fixing #50 fixes gas costs. Fixing gas costs fixes Azimuth. Fixing Azimuth fixes Urbit. And "Urbit fixes this".

cgyarvin commented 3 months ago

Wait what? Can you explain a little moreOn Aug 21, 2024, at 6:20 PM, Calvin McCarter @.***> wrote: @cgyarvin This fixes #50. Fixing #50 fixes gas costs. Fixing gas costs fixes Azimuth. Fixing Azimuth fixes Urbit. And "Urbit fixes this".

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

calvinmccarter commented 3 months ago

Instead of the naive rollup approach, the original Azimuth contract should've just been made more gas efficient. Implementation changes like this one -- which makes ownership changes less costly, while shifting gas costs to token trackers -- should be tried first.

cgyarvin commented 3 months ago

Yep

On Wed, Aug 21, 2024 at 6:33 PM Calvin McCarter @.***> wrote:

Instead of the naive rollup approach, the original Azimuth contract should've just been made more gas efficient. Implementation changes like this one -- which makes ownership changes less costly, while shifting gas costs to token trackers -- should be tried first.

— Reply to this email directly, view it on GitHub https://github.com/urbit/azimuth/pull/55#issuecomment-2303430364, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABPEALF4Q44FPQYIYIXJU3ZSU5W5AVCNFSM6AAAAABM5DNAECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBTGQZTAMZWGQ . You are receiving this because you were mentioned.Message ID: @.***>