Skip to content

Conversation

@vboiteau
Copy link

The command line script was returning error, so with some quick fixes, it's now meeting the expected behaviour.

@lemieux
Copy link
Owner

lemieux commented Aug 31, 2016

Les formats ont changé sur Cooptel?

if type == "json":
return api.getData(phase,room,datetime.now().month)
if type =="all":
return "Used :\t\t{:0.2f}GB ({:0.2f}%)\nLeft :\t\t{:0.2f}GB ({:0.2f}%)\nTotal :\t\t{:0.2f}GB".format(data["usage"],pct,data["left"],100-pct,data["maximum"])
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cooptel les donne en GB maintenant?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'imagine parce que j'ai pas changer l'interprétation des données dans le fichier api.py.

@lemieux
Copy link
Owner

lemieux commented Aug 31, 2016

Wow, le formattage de cette lib là est vraiment troublant...

@vboiteau
Copy link
Author

En fait, l'output json reste le meme sur getData dans l'api. Mais pour le netusage.py j'utilise le dict direct sans le formatter en json. J'ai tester les differentes commandes avec les types et les outputs me semble correct. Par exemple, '... netusage.py all ...' avec mes fixes me donne:
Used : 143.46GB (89.67%)
Left : 16.54GB (10.33%)
Total : 160.00GB
Tandis qu'avant mes fixes, j'avais une erreur genre "variable pct not define" ou de quoi dans le genre.

@lemieux
Copy link
Owner

lemieux commented Aug 31, 2016

donc le seul fix réel devrait être l'ajout de la variable?

@vboiteau
Copy link
Author

vboiteau commented Aug 31, 2016

Mais pour définir cette variable avec la version originale de l'api.py, j'ai besoin du dictionnaire et non du json. Je comprends que l'api retourne un json au source externe. Toutefois, vu que tu importes le api.py dans ton netusage.py, ça servirait a rien de devoir parser à nouveau le json.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants