Discussion :

[rlelamer]

Bonsoir,
Je rencontre un soucis aléatoire de téléchargement de fichier vide (grib) auprès de la NOAA que je n'arrive pas à expliquer.
Ce soucis se passe systématiquement en sortie de la boucle while True:Lorsque les fichiers sont déjà présent (r.status_code == 200), le script ne plante jamais donc je suppose que c'est cette boucle while qui n'est pas correcte

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
[...]
s = Session()
noaaUrl = 'https://nomads.ncep.noaa.gov/cgi-bin/filter_gfs_1p00.pl'
params = {
    'file': 'file',
    'lev_10_m_above_ground': 'on',
    'var_UGRD': 'on',
    'var_VGRD': 'on',
    'leftlon': '0',
    'rightlon': '360',
    'toplat': '90',
    'bottomlat': '-90',
    'dir': 'dir'
}
 
[...]
 
for c_tm in range(6, 387, 3):
    params['file'] = 'gfs.t' + cpo_tz(c_h)[1] + 'z.pgrb2.1p00.f' + f'{c_tm:03}'
    params['dir'] = '/gfs.' + cpo_tz(c_h)[0] + '/' + cpo_tz(c_h)[1]
 
    grb2file = 'GFS1_' + cpo_tz(c_h)[0] + '_' + cpo_tz(c_h)[1] + '_' + f'{c_tm:03}' + '.grb2'
    grbfile = 'GFS1_' + cpo_tz(c_h)[0] + '_' + cpo_tz(c_h)[1] + '_' + f'{c_tm:03}' + '.grb'
    ncfile = 'GFS1_' + cpo_tz(c_h)[0] + '_' + cpo_tz(c_h)[1] + '_' + f'{c_tm:03}' + '.nc1'
 
    r = s.get(noaaUrl, params=params)
    while True:
        if r.status_code != 200:
            print('Waiting ...')
            sleep(60)
        break
 
    r = s.get(noaaUrl, params=params)
    with open(join(path_c_tz_grb2, grb2file), 'wb') as f:
        f.write(r.content)
[...]

Y'a t'il quelque chose qui n'est pas correct ?
Merci

[MPython Alaplancha]

Ce soucis se passe systématiquement en sortie de la boucle while True:Y'a t'il quelque chose qui n'est pas correct ?
Merci

Bonjour,
Quelque soit les conditions, vu l'emplacement de ton break, ton while ne rebouclera pas...

[wiztricks]

Salut,

Y'a t'il quelque chose qui n'est pas correct ?

Si l'idée est de dire je fais un .get et si le "status" n'est pas 200 on recommence 60 secondes plus tard, pas facile de la retrouver dans le code.

De plus vous écrivez:

Lorsque les fichiers sont déjà présent (r.status_code == 200)

Cela traduit le succès de la requête et çà n'a pas trop de rapport avec fichier présent ou pas.

- W

[rlelamer]

Bonsoir,
Merci pour vos réponses.

Je pense avoir compris ce que tu soulignes @Hominidé et j'ai donc fais une petite modification de la boucle while True, que je testerais à partir de 23h30 (UTC+2) pour j'espère la valider ...

@wiztricks
je comprends pas bien votre remarque

pas facile de la retrouver dans le code

Désolé si je répond à coté de la plaque ... mais ce script n'est pas un "gros" code ... il fait 500 lignes environ car il est très commenté pour m'y retrouver, gavé de print() qui me servent à valider que j'obtiens bien ce que je souhaite et également pas du tout optimisé (= surement très mal écrit avec mes nombreuses répétitions), c'est mon premier script donc il est loin d'être parfait 😉

Cela traduit le succès de la requête et çà n'a pas trop de rapport avec fichier présent ou pas.

Ça, ça relève d'une mauvaise comprehension/interpretation de ma part ... basé sur ceci

200 OK:
La requête a réussi. Le signification du succès peut varier selon la méthode HTTP :
=> GET : La ressource a été récupérée et est retransmise dans le corps du message.

😖

[rlelamer]

Ça fonctionne comme voulu, merci pour le commentaire @Hominidé c'est ce qui m'a mis sur la bonne piste 😉

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
    while True:
        r = s.get(noaaUrl, params=params)
        if r.status_code != 200:
            print('Waiting ...')
            sleep(60)
        else:
            break