Skip to content

Conversation

SozinM
Copy link
Collaborator

@SozinM SozinM commented Jun 2, 2025

📝 Summary

Turned out data_gas_fjord was not a fitting function because it has extra logic of multiplying by 16.
I replaces it with original function, while manually dividing it by 1e6

💡 Motivation and Context


✅ I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

Copy link
Contributor

@karim-agha karim-agha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: A link to something that explains why it is divided by million or more context would be helpful.

@SozinM SozinM merged commit 10de3b5 into main Jun 2, 2025
2 checks passed
@SozinM SozinM deleted the msozin/fix-da-scaling-again branch June 2, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants