-
Notifications
You must be signed in to change notification settings - Fork 18
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
Sources: Some defillama assets are failing to materialize #2762
Comments
The underlying issue is that we are asking DefiLlama for invalid slugs. For instance:
Here are the steps on how to check if a protocol exists: #2681 (comment). And here are all the invalid slugs we currently have: Protocols that do not exist: ['ether.fi', 'compound-finance', 'lisk', 'derive', 'contango', 'kelp-dao', 'resolv', 'kroma', 'uniswap', 'velodrome', 'origin', 'origin-protocol', 'polynomial-protocol', 'extra-finance', 'frax-finance', 'mode'] I do not know which protocols exactly we need. Maybe @evanameyer1 can help us choose the ones he needs :)
|
FYI, I'm adding a DefiLlama validator here What this means is that we'll automatically validate defillama URLs in oss-directory like this Once we start using that as the source of truth, then I think this issue is moot |
@Jabolol Sorry, just saw your message now. Thanks for the clarification, I realize now I completely misunderstood how we accessed defillama data, hence so many invalid slugs! I can fix this now. The only thing is I'm not always 100% sure of which exact protocol corresponds to each project. For now I'll update the defillama links in the oss-directory to the best of my ability here: opensource-observer/insights#115 (comment), so that way I can use @ryscheng's new validator and we can at least ensure all links are real. Maybe @ccerv1 @Jabolol could we set up a call after I add my initial guesses of which links correspond to help validate some of the protocols that I selected? Just want to make sure I select the right one for each project. |
I updated the relevant s6 growth grant projects .yml file with defillama links here: opensource-observer/oss-directory#633 |
Let's say the remaining scope on this issue is to connect the oss-directory data to the DefiLlama staging models as the source of truth on which defillama slugs to pull |
Which area(s) are affected? (leave empty if unsure)
No response
To Reproduce
See: https://dagster.opensource.observer/runs/00c38c05-802f-409e-bc35-2afd3eba7728
Describe the Bug
Some things seem to be faililng still. Not high urgency but we should debug to see what's up.
Expected Behavior
These things should succeed.
The text was updated successfully, but these errors were encountered: