azimuth icon indicating copy to clipboard operation
azimuth copied to clipboard

Remove pointsOwnedBy and related logic

Open calvinmccarter opened this issue 3 years ago • 4 comments

This fixes #50 .

calvinmccarter avatar Apr 21 '22 01:04 calvinmccarter

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

calvinmccarter avatar Aug 22 '24 01:08 calvinmccarter

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: @.***>

cgyarvin avatar Aug 22 '24 01:08 cgyarvin

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.

calvinmccarter avatar Aug 22 '24 01:08 calvinmccarter

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: @.***>

cgyarvin avatar Aug 22 '24 04:08 cgyarvin