-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Init flare dex #7243
Init flare dex #7243
Conversation
…nto 1202-init-flare
thank you 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add too: (I didn't put commas!)
('phili-inu', 'PHIL', 0x932E691aA8c8306C4bB0b19F3f00a284371be8Ba,18)
('flare-inu', 'FINU', 0x2F8d7ff4f55D7B3E10D4910BeB38209291843B36, 18)
updates on decoded tables:
|
dbt_subprojects/tokens/models/prices/flare/prices_flare_tokens.sql
Outdated
Show resolved
Hide resolved
looks like only enosys v3 source is empty now. can you validate the data from my recent commits for enosys v2, to ensure correct? |
I think that's correct. |
@Hosuke let's prep this for merge. remove enosys v3, add back union of flare data into final dex trades, etc. we will revisit enosys v3 in a follow up PR when ready |
This reverts commit 90f0256.
- Remove enosys v3 model and its references - Keep enosys v2, sparkdex v2/v3, and blazeswap in flare trades
Updated, but the unioned final dex trades may be too large to run in CI. |
This reverts commit e72e698.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for getting it merge ready. yes, CI cluster is struggling on dex builds on both cluster sizes (bigger machines/less nodes, smaller machines, more nodes). we will need to continue experimenting with sizing. in the meantime, we should consider forcing CI to filter dex_base_trades
on a hardcoded date, see it run through, then revert commit.
Init flare dex