Skip to content
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

♻️ Change 🧠 Logic limits type in Protobuf #784

Closed
ccamel opened this issue Oct 14, 2024 · 0 comments · Fixed by #791
Closed

♻️ Change 🧠 Logic limits type in Protobuf #784

ccamel opened this issue Oct 14, 2024 · 0 comments · Fixed by #791
Assignees

Comments

@ccamel
Copy link
Member

ccamel commented Oct 14, 2024

Purpose

This implementation addresses the proposal outlined in Axone Protocol Discussion #2, which involves modifying some fields of the Interpreter protobuf structure, so that it changes from string to uint64.

  • Limits message:
    • max_size
    • max_result_count
    • max_user_output_size
    • max_variables
  • GasPolicy message:
    • weighting_factor
    • default_predicate_cost
  • PredicateCost messsage:
    • cost

Impact

  • A migration of the logic module should be considered to ensure that the storage of these values is still functional in the dentrite-1 network.
@github-project-automation github-project-automation bot moved this to 📋 Backlog in 💻 Development Oct 14, 2024
@ccamel ccamel self-assigned this Oct 21, 2024
@ccamel ccamel moved this from 📋 Backlog to 🏗 In progress in 💻 Development Oct 21, 2024
@ccamel ccamel mentioned this issue Oct 22, 2024
4 tasks
@ccamel ccamel linked a pull request Oct 22, 2024 that will close this issue
4 tasks
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in 💻 Development Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant