Skip to content

Twap tracker production #214

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

Merged
merged 3 commits into from
Jul 28, 2022
Merged

Twap tracker production #214

merged 3 commits into from
Jul 28, 2022

Conversation

guibescos
Copy link
Contributor

No description provided.

* make pythd robust

* made the oracle accept new size

* left space for extra publishers
* make pythd robust

* made the oracle accept new size

* left space for extra publishers
* changed c return value of update price to indicate aggregation (#190)

* changed c return value of update price to indicate aggregation

* forgot to add bindings

* removed the need for casting

* fixed update_no_fail

* removed unnecessary docker packages

* timeless coments

* a convinient way to derive traits

* changed importing style

* added register_traits method

* fixed formatting

* add a new insctruction and interecepted update price calls

* fixed formatting

* more helpful error message for incorrect c account size

* fixed merging errors

* addressed some warning

* updated comment

* merging formatting issues

* memory borrowing issues after rebasing

* updated import

* more import style changes

* removed panics

* bug introduced by merge

* moved dispatching to processor

* better formatting

* cleaner error handling

* error handling wrapper for c_entrypoint

* removed unwrap

* more comments

* removed unused imports + changed import style
@guibescos guibescos marked this pull request as ready for review July 28, 2022 18:21
@guibescos guibescos merged commit 1857314 into main Jul 28, 2022
@guibescos guibescos deleted the twap-tracker-production branch July 28, 2022 19:42
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.

3 participants