From e39768daf9e799ac1dd86277f6a5d00a0cfa0681 Mon Sep 17 00:00:00 2001 From: jloup Date: Wed, 17 Jan 2018 12:02:09 +0100 Subject: [PATCH] review --- comments.txt | 11 +++++++++++ portfolio.py | 4 ++++ 2 files changed, 15 insertions(+) create mode 100644 comments.txt diff --git a/comments.txt b/comments.txt new file mode 100644 index 0000000..a84b9ed --- /dev/null +++ b/comments.txt @@ -0,0 +1,11 @@ +J'ai mis un peu de temps à relire. Y avait des notions que je connaissais pas :) + +Le truc que je pige pas encore je pense c'est le bid/ask pour une pair ou son invert. J'ai mis des # COMMENT dans portfolio.py + +Dommage qu'il y ait pas de package pour gérer les amount de crypto comme pour les currencies traditionnelles. Ca eviterait les roundings errors. + +Le reste ca va, il faut que je me créer des comptes sur les exchanges pour essayer tout ca: + - Je me rends pas compte combien de temps un order mets de temps à passer ? ca dépend de l'état de la blockchain ? Parce que le cours peut pas mal changer entre temps ? + - Quelles erreurs as tu rencontrées lundi matin dernier ? + +Je pense que si ce script tourne pas mal, il faudra peut-être leur donner accès a une mini interface web. diff --git a/portfolio.py b/portfolio.py index 507f796..9d8fd97 100644 --- a/portfolio.py +++ b/portfolio.py @@ -112,7 +112,11 @@ class Amount: def invert(ticker): return { + # COMMENT: c'est pas dangereux de faire l'invert comme ca ? il peut pas y avoir d'asymétrie ? + # et je pense que ca simplifierai le code de pas avoir le inverted. Il me semble qu'il est là + # juste pour une histoire de cache. "inverted": True, + # COMMENT: j'ai pas compris pourquoi tu prenais l'average ? "average": (float(1/ticker["bid"]) + float(1/ticker["ask"]) ) / 2, "notInverted": ticker, } -- 2.41.0