Skip to content
This repository has been archived by the owner on Jan 8, 2025. It is now read-only.

feat: use scarb macro to generate const powers #841

Open
2 tasks
enitrat opened this issue Aug 21, 2024 · 4 comments
Open
2 tasks

feat: use scarb macro to generate const powers #841

enitrat opened this issue Aug 21, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers timebox-1hr 1 Hour exploration - tinkering

Comments

@enitrat
Copy link
Contributor

enitrat commented Aug 21, 2024

Feature Request

Describe the Feature Request

Instead of pre-computing powers of two and copy-paste the values in Cairo, use a macro to generate the values at compile time.

Describe Preferred Solution

Alexandria has a macros package that has exactly this. keep-starknet-strange/alexandria#324 (review)

Related Code

Additional Context

If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

  • Yes
  • No
@enitrat enitrat added the enhancement New feature or request label Aug 21, 2024
@github-project-automation github-project-automation bot moved this to 🆕 Backlog in Kakarot on Starknet Aug 21, 2024
@enitrat enitrat added good first issue Good for newcomers timebox-1hr 1 Hour exploration - tinkering labels Aug 22, 2024
@juankohler
Copy link

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

I have over 7 years of experience in backend development, primarily using Go. While my expertise is in building robust backend systems, I’m currently learning Cairo in Dojo Coding and am eager to apply my skills to contribute effectively to projects using this language.

@ClementWalter
Copy link
Member

@juankohler what's the status of this?

@enitrat
Copy link
Contributor Author

enitrat commented Sep 10, 2024

@ClementWalter the Language Server has some issues with procedural macros (they're undetected so considered as errors) - as such I don't want to merge it yet, I reported the issue and proposed them to simply mute diagnostics for macros.

@maxslimb
Copy link

@enitrat is this issue completed ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers timebox-1hr 1 Hour exploration - tinkering
Projects
No open projects
Status: 🆕 Backlog
Development

Successfully merging a pull request may close this issue.

4 participants