notadamking/RLTrader

Reasonable balances for trading

Open

#38 geöffnet am 12. Juni 2019

Auf GitHub ansehen
 (4 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)Python (535 Forks)batch import
bugenhancementhelp wanted

Repository-Metriken

Stars
 (1.592 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Currently, agents buy and sell using unrealistic USD and BTC balances. For instance, during a debugging session I saw one agent executing this kind of trades:

Buying BTC balance USD: 0.004812330813060336 btc held: 2.5835011089138864 Net worth: 17707.373082848768 ============= Buying BTC balance USD: 0.0032082205420402242 btc held: 2.5835013414545522 Net worth: 17777.0501037559

It doesn't look like there is a logical lower bound on the USD and BTC that can be used in a trade. In real life one wouldn't try to (and couldn't) buy BTC with $0.0016.

Also, using correct number of decimals (i.e., 8 for BTC) is important for precision. That might be another bug actually.

Contributor Guide

Research-Richtung
Das Problem beschreibt unrealistische USD und BTC Salden während Trades, z.B. Handel mit $0,0016. Die Korrektur sollte einen Mindestsaldenschwellenwert erzwingen und die BTC Dezimalgenauigkeit korrigieren. Beginnen Sie mit der Untersuchung der Trading Umgebungsdateien, wahrscheinlich im Verzeichnis `env` oder `agent`, um zu finden, wo Saldenprüfungen stattfinden. Kommentare zum Issue können zusätzlichen Kontext liefern. Implementieren Sie eine Logik, um Trades unterhalb eines festgelegten Minimums abzulehnen, und verwenden Sie eine korrekte Rundung für BTC Dezimalstellen.
Tech Stack
python
Domain
backenddata
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Python basicsUnderstanding of trading simulation
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40