review

diff --git a/comments.txt b/comments.txt
new file mode 100644 (file)
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 507f79642c62aa64343251747706173d9bf31bb8..9d8fd9754235f89f114477fe22f1f1d71b1def49 100644 (file)
--- 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,
                     }