Mycroft-core: La sortie mycroft.format.nice_duration() est gênante sur de longues périodes

Merci d'avoir signalé cela :)

Si quelqu'un souhaite s'y attaquer et souhaite gérer les années bissextiles, j'ai récemment utilisé ce qui suit dans la compétence Date-Heure :

def is_leap_year(self, year):
    return (year % 400 == 0) or ((year % 4 == 0) and (year % 100 != 0))

Il y a aussi la fonction calendar.isleap dans la bibliothèque standard...

Le module humanize pourrait être quelque chose à regarder aussi...

>>> humanize.naturaldelta(timedelta(minutes=40, hours=360000))
'41 years'

Si quelqu'un va refactoriser cette fonction, cela peut valoir la peine de donner une résolution à l'objet pendant qu'il est là-dedans. C'est amusant et à 100% dans le caractère pour Mycroft de vous dire combien de secondes se sont écoulées depuis ce jour il y a longtemps, mais les codeurs pourraient préférer spécifier, nice_duration_years() ou nice_duration(dt1, dt2, <str>resolution)

Oui, j'ai pensé à la "résolution" pour un certain nombre de ces fonctions, mais je ne veux pas faire de changements de rupture. On dirait que cela pourrait être moins un problème ici si nous ajoutons uniquement un paramètre facultatif et qu'il ne renvoie toujours qu'une chaîne, par exemple :
nice_duration(<int>seconds, <str>resolution)

Parce que je ne code pas ça moi-même, je vais faire mille suggestions !

Si ces fonctions obtiennent une résolution, peut-être que les années pourraient fonctionner sur des trimestres, ou considérer combien de mois ou etc. font la différence entre "environ 2 ans", "presque 2 ans", "un peu plus de 2 ans" ou simplement "2 ans"

J'ai juste commencé à m'embêter avec ça dans l'interprète, et je ne pense pas que ce sera aussi difficile que je le pensais. Si personne d'autre n'a encore commencé, je le ferai après le dîner.

Correction fusionnée.