metadata-and-data issueshttps://forge.grandlyon.com/web-et-numerique/web-et-numerique-internet/data.grandlyon.com/web-portal/components/indexers/metadata-and-data/-/issues2021-05-17T09:31:50Zhttps://forge.grandlyon.com/web-et-numerique/web-et-numerique-internet/data.grandlyon.com/web-portal/components/indexers/metadata-and-data/-/issues/3Remplacer boucle `while` par `for`2021-05-17T09:31:50ZGhost UserRemplacer boucle `while` par `for``while True` est un peu dangereux ici (et je pense que tu peux t'en passer) :
https://gitlab.alpha.grandlyon.com/acerioni/data-grandlyon-com-indexer/blob/master/1-metadata-getter.py#L35
Si `records_per_page=1` on ne sort jamais de la b...`while True` est un peu dangereux ici (et je pense que tu peux t'en passer) :
https://gitlab.alpha.grandlyon.com/acerioni/data-grandlyon-com-indexer/blob/master/1-metadata-getter.py#L35
Si `records_per_page=1` on ne sort jamais de la boucle.
Aussi je n'ai pas l'impression que le code gère le cas d'une liste de UUID finie.
Dans ce cas dès que `count == len(uuid)` (ou quelque chose comme ça), il faut sortir de la boucle.
Et la requête ci-dessous nous permet de connaitre le nombre de fiches de métadonnées :
https://download.data.grandlyon.com/catalogue/srv/fre/q?&from=1&to=0&buildSummary=true
https://download.data.grandlyon.com/catalogue/srv/fre/q?&from=1&to=0&buildSummary=true&_content_type=json
Maëlhttps://forge.grandlyon.com/web-et-numerique/web-et-numerique-internet/data.grandlyon.com/web-portal/components/indexers/metadata-and-data/-/issues/21-metadata-getter.py#L64 : `yield` ou `yield from`2021-05-17T09:31:41ZGhost User1-metadata-getter.py#L64 : `yield` ou `yield from`Ne serait-ce pas plutôt `yield from` qu'il faudrait utiliser ici :
https://gitlab.alpha.grandlyon.com/acerioni/data-grandlyon-com-indexer/blob/master/1-metadata-getter.py#L64
MaëlNe serait-ce pas plutôt `yield from` qu'il faudrait utiliser ici :
https://gitlab.alpha.grandlyon.com/acerioni/data-grandlyon-com-indexer/blob/master/1-metadata-getter.py#L64
Maëlhttps://forge.grandlyon.com/web-et-numerique/web-et-numerique-internet/data.grandlyon.com/web-portal/components/indexers/metadata-and-data/-/issues/1Error `local variable 'filtered_records' referenced before assignment`2019-02-04T11:04:04ZGhost UserError `local variable 'filtered_records' referenced before assignment`Le problème est par ici :
https://gitlab.alpha.grandlyon.com/acerioni/data-grandlyon-com-indexer/blob/master/1-metadata-getter.py#L58
```
# apply filter
if the_filter != None:
logging.debug("Filtering out unwanted records, if prese...Le problème est par ici :
https://gitlab.alpha.grandlyon.com/acerioni/data-grandlyon-com-indexer/blob/master/1-metadata-getter.py#L58
```
# apply filter
if the_filter != None:
logging.debug("Filtering out unwanted records, if present.")
filtered_records = [x for x in records if x['geonet:info']['uuid'] not in the_filter]
yield filtered_records
```
Avec la conf suivante :
```
metadata_getter:
uuids_to_get:
- a3ae7043-a640-4286-b604-48e4827a2b04
uuids_to_filter_out:
```
Maël