Coquille dans le code python de l'académie

Olivier
Olivier Dataiku DSS Core Designer, Registered Posts: 2 ✭✭✭
Bonjour,
Je suis vos cours de l'academie en ligne et je crois avoir vu une coquille dans le cours advanced-designer => automation-course-1 => Hands-On: Custom Metrics, Checks & Scenarios => Create a Custom Trigger
Le cas pris en exemple, "déclenchement le premier vendredi du mois", me semble très pertinent. Le code python se termine ainsi :
day = today.day
if dayofweek == 5 and day < 7:
 t.fire()
La variable day représente le jour dans le mois (25 pour Noël, 1 pour le jour de l'an...), la valeur zero n'est donc pas possible.
Si le mois comment par un samedi, le premier vendredi sera le 7.
Je pense donc qu'il faut inclure dans le cas où day est égal à 7 et donc écrire :
day = today.day
if dayofweek == 5 and day <= 7:
 t.fire()
Voici un petit code qui m'a permis de me conforter dans l'idée ci-dessus :
import datetime as dt
my_date = dt.datetime(2022, 10, 7)
dayofweek = my_date.isoweekday()
day = my_date.day
print("Day of week was : ",dayofweek)
print("Day in month was : ", day)
if dayofweek == 5 and day < 7 :
print ("It's was time to trigger!")
else:
print ("Was it the right day to trigger?")
Merci pour cette academie et la pédagogie mise en oeuvre.
Olivier

Answers

  • Boris_Lallemand
    Boris_Lallemand Dataiker, Dataiku DSS Core Designer, Registered Posts: 3 Dataiker

    Bonjour Olivier,

    Merci pour votre retour concernant cet excercice d'Avanced Designer.
    Votre remarque est pertinente et c'est tout naturellement que nous l'avons prise en compte.

    Nous sommes heureux que le contenu de l'Academie vous plaise.

Setup Info
    Tags
      Help me…