yearn-watch-legacy icon indicating copy to clipboard operation
yearn-watch-legacy copied to clipboard

Published 20 hours ago • verified publisher icon yearn

Bug: Strat solidexJoint not matching in group

Open storming0x opened this issue 2 years ago • 7 comments

This strat has the name Strategy_ProviderOfUSDCToNoHedgeSolidexJoint,

https://yearn.watch/network/fantom/vault/0xEF0210eB96c7EB36AF8ed1c20306462764935607/strategy/0x80332ecdc7ccdb187b95c97d4b33a8e918f8dd70

the group created here is not matching it https://github.com/yearn/yearn-watch/pull/246/files

storming0x avatar Mar 11 '22 17:03 storming0x

If I understand correctly, we simply need to change Solidex Joint in line 742 of risk-framework.json to Strategy_ProviderOfUSDCToNoHedgeSolidexJoint? And I don't think this strategy is active for the Ethereum network so I don't need to change the label in line 550 for now?

p.s. I am not sure how to derive the nameLike value.

0xYao avatar Mar 22 '22 19:03 0xYao

If I understand correctly, we simply need to change Solidex Joint in line 742 of risk-framework.json to Strategy_ProviderOfUSDCToNoHedgeSolidexJoint? And I don't think this strategy is active for the Ethereum network so I don't need to change the label in line 550 for now?

For me, it seems like the issue was mainly concerning the one on Fantom, and it should be handled by now (after the merge in f89995242af34da5eb372b4d7528592923ff70d8) I think the implementation in risk-framework.json file was fine, the mismatch of data in SDK and the helper contract seemed to be the problem atm

kyhoon avatar Mar 22 '22 20:03 kyhoon

If I understand correctly, we simply need to change Solidex Joint in line 742 of risk-framework.json to Strategy_ProviderOfUSDCToNoHedgeSolidexJoint? And I don't think this strategy is active for the Ethereum network so I don't need to change the label in line 550 for now?

For me, it seems like the issue was mainly concerning the one on Fantom, and it should be handled by now (after the merge in f899952) I think the implementation in risk-framework.json file was fine, the mismatch of data in SDK and the helper contract seemed to be the problem atm

Can you elaborate on what and where the SDK and the helper contract are? I am new to Yearn, sorry if the questions are silly :p

0xYao avatar Mar 22 '22 20:03 0xYao

Can you elaborate on what and where the SDK and the helper contract are? I am new to Yearn, sorry if the questions are silly :p

hehe no worries, I am also trying to figure out how things work so by the SDK I mean the yearn-sdk called from the VaultService instance and by the helper contract I mean the strategiesHelper instance, which is a smart contract

kyhoon avatar Mar 22 '22 20:03 kyhoon

what is the data mismatch that you have found? e.g. for solidexJoint, what are the expected results from the sdk and helper contract, and what are the actual output?

0xYao avatar Mar 23 '22 00:03 0xYao

what is the data mismatch that you have found? e.g. for solidexJoint, what are the expected results from the sdk and helper contract, and what are the actual output?

tbh I am also new to the codebase, not sure what exactly the expected results should be 🤔 What I have noticed is just that there are more strategies registered in the SDK compared to the helper contract atm (in Fantom) If you are interested you could call the ABI from the code yourself which is pretty cool hehe

kyhoon avatar Mar 23 '22 11:03 kyhoon

https://github.com/yearn/yearn-watch/pull/246/files

we need to debug why the entry in fantom is not being picked up.

Strategy_ProviderOfUSDCToNoHedgeSolidexJoint string should match for search string solidexjoint, solidex is a strategy that only exists on fantom

My guess is that having an ethereum group with the fantom and same name gives a problem on matching, can we debug by removing the ethereum group to confirm this is the bug?

storming0x avatar Mar 23 '22 19:03 storming0x