Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

BuySell does not work with xlm/euro in the feed from kraken via the config file #13

Closed
brianopedal opened this issue Sep 27, 2018 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@brianopedal
Copy link

buysell config file

When changing the buysell configuration file. to read krakens xlm/usd orders to xlm/euro
The bot does not understand the xlm euro pair, and throws an error.

I think the correct behavior in this situation is for the bot to ...

Steps To Reproduce

Open config file and change what is read from kraken from xxlm/zusd to xxlm/zeur
https://support.kraken.com/hc/en-us/articles/360001185506-Asset-Codes

@nikhilsaraf
Copy link
Contributor

@brianopedal can you share the DATA_FEED_URL string that you are trying to use?

@brianopedal
Copy link
Author

buysell.cfg file
line 17:

DATA_FEED_A_URL="kraken/XXLM/ZEUR"

@nikhilsaraf
Copy link
Contributor

nikhilsaraf commented Sep 29, 2018

@brianopedal that's a great find. The EUR currency isn't recognized by the current version of the bot (v1.0.0-rc2).

I'm just adding a fix for this and uploading a new release candidate.

@nikhilsaraf nikhilsaraf added the bug Something isn't working label Sep 29, 2018
@nikhilsaraf nikhilsaraf added this to the v1.0.0-rc3 milestone Sep 29, 2018
Reidmcc added a commit to Reidmcc/kelp that referenced this issue Oct 2, 2018
* cap3: 1 - implement liabilities for XLM

* cap3: 2 - integrate liabilities into willOversell and willOverbuy methods

* cap3: 3 - remove fractionalReserveMultiplier cli arg

* cap3: 4 - offers in the same tx will contribute to liabilities, incorporate into cachedLiabilities

* cap3: 5 - handle case of setting incrementalBuy for native asset

* cap3: 6 - refactored willOversell and willOverbuy to extract common offer logic

* cap3: 7 - added support for checking XLM fee and min reserves

* cap3: 8 - update ordering of operations in strategies (sellSideStrategy, mirrorStrategy) when not all offers can be placed — always place inside orders first

* cap3: 9 - native fee inclusion checks source/trader account usage

* cap3: 10 - delete offer if nothing was created and we were planning to modify an existing offer

* cap3: 11 - prepend deleteOps so we "free" up our liabilities capacity to place the new/modified offers

* cap3: 12 - better error propagation from sdex.createModifySellOffer

* cap3: 13 - extracted liability updates after placing/modifying offers to callers in sellSideStrategy and mirrorStrategy

* cap3: 14 - prepend deleteOps for mirror strategy too

* cap3: 15 - log liabilities in trader.go after resetting and after updating ops

* cap3: 16 - add liabilities in mirror strat

* cap3: 17 - added support for partial offers in sellSideStrategy, refactored updateSellLevel

* cap3: 18 - updated comment when resetting cached liabilities

* cap3: 19 - add current offer amounts to liabilities when not modifying an offer

* cap3: 20 - add a caching layer for asset balances to reduce requests

* cap3: 21 - cleaner logging of asset balance and trust amounts

* cap3: 22 - let new capacity constraint system handle max limits for sellSideStrategy

* cap3: 23 - reset caches before pruning and updating operations

* cap3: 24 - update CHANGELOG

* new release: v1.0.0-rc2

* expand assets allowed to use with the Kraken exchange, fixes stellar-deprecated#13

* new release: v1.0.0-rc3

* remove default rate offset percent from sample strategy config files

* Print strat (stellar-deprecated#15)

* Update trade.go

* Update trade.go

* Update trade.go

* enable Travis CI, closes stellar-deprecated#17 (stellar-deprecated#18)

* enable Travis CI

* go_import_path

* travis: ./bin/kelp version

* doc clarify full path needed for configs (stellar-deprecated#16)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants