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.

Sign up for GitHub

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

Jump to bottom

Exchangerate controller #2226

Merged
merged 5 commits into from
Oct 14, 2024
Merged

Exchangerate controller #2226

merged 5 commits into from
Oct 14, 2024

Conversation

gameofpointers
Copy link
Contributor

No description provided.

@gameofpointers gameofpointers force-pushed the exchangerate branch 8 times, most recently from 124487d to 6e09890 Compare October 14, 2024 21:01
jdowning100
jdowning100 reviewed Oct 14, 2024
View reviewed changes
core/headerchain.go Outdated Show resolved Hide resolved
jdowning100
jdowning100 reviewed Oct 14, 2024
View reviewed changes
core/worker.go Outdated Show resolved Hide resolved
@gameofpointers gameofpointers force-pushed the exchangerate branch 2 times, most recently from f7ace9b to e51f5ca Compare October 14, 2024 22:25
jdowning100
jdowning100 reviewed Oct 14, 2024
View reviewed changes
core/worker.go Outdated Show resolved Hide resolved
kiltsonfire and others added 3 commits October 14, 2024 17:35
@kiltsonfire @gameofpointers
Exchange Rate controller
d43a183 
Every zone block is a clearing market, the conversions are played
once in the EVM and then price effect from that conversions determine
the exchange rate for that block and the converted value gets adjusted
based on that price slip
@gameofpointers
bugfix: Added rate mutex in ProcRequestRate
e097795
@gameofpointers
Use a number check before TimeToStartEtx and use gas check after
6efa70f
@gameofpointers gameofpointers merged commit 2a7b069 into dominant-strategies:main Oct 14, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kiltsonfire kiltsonfire kiltsonfire approved these changes

@jdowning100 jdowning100 jdowning100 approved these changes

@wizeguyy wizeguyy Awaiting requested review from wizeguyy wizeguyy is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gameofpointers @jdowning100 @kiltsonfire