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.

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

[Bug] Crypto/DeFi/Vaults - Command appears to be broken #1822

Closed
deeleeramone opened this issue May 16, 2022 · 2 comments
Closed

[Bug] Crypto/DeFi/Vaults - Command appears to be broken #1822

deeleeramone opened this issue May 16, 2022 · 2 comments
Assignees
Labels
bug Fix bug

Comments

@deeleeramone
Copy link
Contributor

Big red message:

/miniconda3/envs/gst/lib/python3.8/site-packages/urllib3/connectionpool.py:1043: InsecureRequestWarning: Unverified HTTPS request is being made to host 'apiv2.coindix.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(
Error: Invalid Response: {"hasPrevPage":false,"hasNextPage":true,"total":1648,"totalPages":50,"data":[{"id":75836,"name":"BSHARE-FTM","icon":"https://api.coindix.com/icons/BSHARE-FTM.png","chain":"Fantom","protocol":"Beefy","base":50.65453484,"reward":0,"rewards":{},"apy":50.65453484,"tvl":1763814,"link":"https://api.coindix.com/vault/75836/redirect","is_new":false,"watched":false},{"id":96613,"name":"ESHARE-BNB","icon":"https://api.coindix.com/default_icon.png","chain":"BNB Chain","protocol":"Beefy","base":17.84666437,"reward":0,"rewards":{},"apy":17.84666437,"tvl":1143455,"link":"https://api.coindix.com/vault/96613/redirect","is_new":false,"watched":false},{"id":135903,"name":"WETH-OSMO","icon":"https://api.coindix.com/icons/WETH-OSMO.png","chain":"Osmosis","protocol":"Osmosis","base":9.0462,"reward":0,"rewards":{},"apy":9.0462,"tvl":1333486,"link":"https://api.coindix.com/vault/135903/redirect","is_new":true,"watched":false},{"id":66309,"name":"sCASH-COW","icon":"https://api.coindix.com/icons/sCASH-COW.png","chain":"Solana","protocol":"Quarry","base":0,"reward":6.3367,"rewards":{"COW":0},"apy":6.3367,"tvl":10346484,"link":"https://api.coindix.com/vault/66309/redirect","is_new":false,"watched":false},{"id":35035,"name":"CAKE-BUSD","icon":"https://api.coindix.com/icons/CAKE-BUSD.png","chain":"BNB Chain","protocol":"Alpaca","base":2.26854681,"reward":1.22931794,"rewards":{"ALPACA":0.03532050031534184},"apy":4.9363341,"tvl":3750348,"link":"https://api.coindix.com/vault/35035/redirect","is_new":false,"watched":false},{"id":75835,"name":"BASED-TOMB","icon":"https://api.coindix.com/icons/BASED-TOMB.png","chain":"Fantom","protocol":"Beefy","base":4.83451901,"reward":0,"rewards":{},"apy":4.83451901,"tvl":1297014,"link":"https://api.coindix.com/vault/75835/redirect","is_new":false,"watched":false},{"id":120475,"name":"BSW-BNB","icon":"https://api.coindix.com/icons/BSW-BNB.png","chain":"BNB Chain","protocol":"Alpaca","base":0.69255329,"reward":3.16590782,"rewards":{"ALPACA":0.014293352529960177},"apy":4.59551874,"tvl":1442104,"link":"https://api.coindix.com/vault/120475/redirect","is_new":false,"watched":false},{"id":93985,"name":"pwrdPWRD-3Crv","icon":"https://api.coindix.com/icons/pwrdPWRD+3Crv.png","chain":"Ethereum","protocol":"Convex","base":3.158,"reward":0.4028,"rewards":{"CRV":0.2243,"CVX":0.17850000000000002},"apy":3.5608,"tvl":2600000,"link":"https://api.coindix.com/vault/93985/redirect","is_new":false,"watched":false},{"id":31336,"name":"D2D-USDC","icon":"https://api.coindix.com/icons/D2D-USDC-NEW.png","chain":"Ethereum","protocol":"Balancer","base":0.0071,"reward":2.6606,"rewards":{"Max BAL":1.9003999999999999,"Min BAL":0.7602},"apy":2.6677,"tvl":2602808,"link":"https://api.coindix.com/vault/31336/redirect","is_new":false,"watched":false},{"id":73680,"name":"pAVAX-AVAX","icon":"https://api.coindix.com/icons/pAVAX-AVAX.png","chain":"Avalanche","protocol":"Beefy","base":2.65168649,"reward":0,"rewards":{},"apy":2.65168649,"tvl":1026510,"link":"https://api.coindix.com/vault/73680/redirect","is_new":false,"watched":false},{"id":97504,"name":"TOMB-FTM","icon":"https://api.coindix.com/icons/TOMB-FTM.png","chain":"Fantom","protocol":"Beefy","base":2.51074943,"reward":0,"rewards":{},"apy":2.51074943,"tvl":4962010,"link":"https://api.coindix.com/vault/97504/redirect","is_new":false,"watched":false},{"id":14150,"name":"LDO-WETH","icon":"https://api.coindix.com/icons/LDO-WETH.png","chain":"Ethereum","protocol":"Balancer","base":0.041,"reward":2.3027,"rewards":{"Max BAL":1.6447999999999998,"Min BAL":0.6579},"apy":2.3437,"tvl":9030291,"link":"https://api.coindix.com/vault/14150/redirect","is_new":false,"watched":false},{"id":100981,"name":"UST-WETH","icon":"https://api.coindix.com/icons/UST-WETH.png","chain":"Ethereum","protocol":"Sushi","base":1.96622567,"reward":0,"rewards":{},"apy":1.96622567,"tvl":1982547,"link":"https://api.coindix.com/vault/100981/redirect","is_new":false,"watched":false},{"id":105317,"name":"PTP","icon":"https://api.coindix.com/icons/PTP.png","chain":"Avalanche","protocol":"Vector","base":0,"reward":1.95768568,"rewards":{"PTP":0.3487334936759784,"VTX":1.608952190596531},"apy":1.95768568,"tvl":2392868,"link":"https://api.coindix.com/vault/105317/redirect","is_new":false,"watched":false},{"id":17266,"name":"ibEUR","icon":"https://api.coindix.com/icons/ibEUR.png","chain":"Ethereum","protocol":"Yearn","base":1.9212,"reward":0,"rewards":{},"apy":1.9212,"tvl":4908560,"link":"https://api.coindix.com/vault/17266/redirect","is_new":false,"watched":false},{"id":96612,"name":"EMP-ETH","icon":"https://api.coindix.com/icons/EMP-ETH.png","chain":"BNB Chain","protocol":"Beefy","base":1.89713324,"reward":0,"rewards":{},"apy":1.89713324,"tvl":1530155,"link":"https://api.coindix.com/vault/96612/redirect","is_new":false,"watched":false},{"id":15985,"name":"JOE","icon":"https://api.coindix.com/icons/JOE.png","chain":"Avalanche","protocol":"Beefy","base":1.79325168,"reward":0,"rewards":{},"apy":1.79325168,"tvl":1013389,"link":"https://api.coindix.com/vault/15985/redirect","is_new":false,"watched":false},{"id":35034,"name":"BNB-BUSD","icon":"https://api.coindix.com/icons/BNB-BUSD.png","chain":"BNB Chain","protocol":"Alpaca","base":1.77095977,"reward":0.18767963,"rewards":{"ALPACA":0.03532050031534184},"apy":1.68564403,"tvl":42818347,"link":"https://api.coindix.com/vault/35034/redirect","is_new":false,"watched":false},{"id":11829,"name":"CVX-WETH","icon":"https://api.coindix.com/icons/CVX-WETH.png","chain":"Ethereum","protocol":"Sushi","base":1.67595149,"reward":0,"rewards":{},"apy":1.67595149,"tvl":2002548,"link":"https://api.coindix.com/vault/11829/redirect","is_new":false,"watched":false},{"id":59324,"name":"ROWAN-OSMO","icon":"https://api.coindix.com/icons/ROWAN-OSMO.png","chain":"Osmosis","protocol":"Osmosis","base":1.669,"reward":0,"rewards":{},"apy":1.669,"tvl":2754736,"link":"https://api.coindix.com/vault/59324/redirect","is_new":false,"watched":false}]}

Error: 'list' object has no attribute 'empty'
@JakubPluta
Copy link
Contributor

It seems, that coindix changed their API, now the results structure does not cointain risk field, and it's also not available in request body.
I will fix this one.

@Chavithra Chavithra assigned JakubPluta and unassigned jose-donato May 23, 2022
DidierRLopes added a commit that referenced this issue Jun 1, 2022
* Add separate view for finbrain for crypto curreny sentiment analysis

* Move json with symbols to separate directory, little refactoring, update readme

* Add screenshot of finbrain for PolkaDot

* Fix defi coindix vaults. Risk param is disabled. Disable urllib3 warning bout not verified request url

* update tests for coindix - cassetes, csvs, txts

* update vaults docs

* Update _index.md

* run crypto tests with record_mode=rewrite

Co-authored-by: didierlopes.eth <dro.lopes@campus.fct.unl.pt>
@piiq piiq added the bug Fix bug label Jun 1, 2022
@piiq
Copy link
Contributor

piiq commented Jun 1, 2022

resolved by #1874

@piiq piiq closed this as completed Jun 1, 2022
deeleeramone pushed a commit to deeleeramone/OpenBBTerminal that referenced this issue Jun 1, 2022
…ance#1874)

* Add separate view for finbrain for crypto curreny sentiment analysis

* Move json with symbols to separate directory, little refactoring, update readme

* Add screenshot of finbrain for PolkaDot

* Fix defi coindix vaults. Risk param is disabled. Disable urllib3 warning bout not verified request url

* update tests for coindix - cassetes, csvs, txts

* update vaults docs

* Update _index.md

* run crypto tests with record_mode=rewrite

Co-authored-by: didierlopes.eth <dro.lopes@campus.fct.unl.pt>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug
Projects
None yet
Development

No branches or pull requests

4 participants