fix(_misc.py): multithread, mutation silencieuse, logger.info

[miki4iaml]

propositions pour _misc.py

• threading.Lock() dans set_grib_defs() pour gérer le multithread
• da.copy dans geo_encode_cf() pour éviter la mutation silencieuse
• logger.info dans set_test_mode() au lieu de print()

[CyrilJl]

J’ai repris la PR pour ne garder que les deux changements qui restent utiles après les PR récentes.

• geo_encode_cf() copie maintenant la DataArray avant d’ajouter les métadonnées CF. J’ai utilisé copy(deep=False) pour éviter l’effet de bord sur l’objet passé en argument, sans recopier les grosses données météo en mémoire.
• set_test_mode() utilise maintenant logger.info() au lieu de print().
• Je n’ai pas gardé la partie threading.Lock() / set_grib_defs(). La suppression de eccodes.codes_context_delete() est déjà dans main, et la remettre, même sous lock, réintroduirait le crash eccodes traité dans test: fiabilise et raccourcit la suite pytest (fix crash eccodes) #22. Le lock ne protège pas non plus les lectures GRIB faites dans les processus enfants.

J’ai ajouté deux tests unitaires ciblés pour couvrir ces deux comportements.

[miki4iaml]

Excellent, merci. Désolé pour le #22 que je n'avais pas intégré. La bonne journée, Miki Le jeu. 9 juil. 2026 à 20:03, Cyril ***@***.***> a écrit :

…

*CyrilJl* left a comment (CyrilJl/MeteoFetch#20) <#20 (comment)> J’ai repris la PR pour ne garder que les deux changements qui restent utiles après les PR récentes. - geo_encode_cf() copie maintenant la DataArray avant d’ajouter les métadonnées CF. J’ai utilisé copy(deep=False) pour éviter l’effet de bord sur l’objet passé en argument, sans recopier les grosses données météo en mémoire. - set_test_mode() utilise maintenant logger.info() au lieu de print(). - Je n’ai pas gardé la partie threading.Lock() / set_grib_defs(). La suppression de eccodes.codes_context_delete() est déjà dans main, et la remettre, même sous lock, réintroduirait le crash eccodes traité dans #22 <#22>. Le lock ne protège pas non plus les lectures GRIB faites dans les processus enfants. J’ai ajouté deux tests unitaires ciblés pour couvrir ces deux comportements. — Reply to this email directly, view it on GitHub <#20?email_source=notifications&email_token=CG5LQECQB6M6AHO3J2JNMID5D7M6XA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIOJSHAYDSOJVHA4KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#issuecomment-4928099588>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/CG5LQEGS4WXTMR525FWCHP35D7M6XAVCNFSNUABFKJSXA33TNF2G64TZHM4TEMRRGM2DMMZWHNEXG43VMU5TINZVHEYTCMBRGE3KC5QC> . Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS <https://github.com/notifications/mobile/ios/CG5LQECKHOA6BPCTIAMSY5D5D7M6XA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIOJSHAYDSOJVHA4KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG> and Android <https://github.com/notifications/mobile/android/CG5LQECMBYPQ7L67GNJZFQL5D7M6XA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIOJSHAYDSOJVHA4KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>. Download it today! You are receiving this because you authored the thread.Message ID: ***@***.***>

[CyrilJl]

Nan nan pas de soucis, j'ai fait le #22 après votre PR, en plus du coup ça m'a incité à regarder ce qui allait pas avec les tests, c'est cool.