Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dictionnaires job-utilisateur visibles dans l'interface web #185

Merged
merged 29 commits into from
Mar 28, 2024

Conversation

notoraptor
Copy link
Contributor

Bonsoir @soline-b ! Voici enfin une PR pour les dictionnaires job-utilisateurs dans l'inferface web. Il s'agit d'une version à jour et nettoyée de mon autre grosse PR en draft, que je vais fermer.

Dans cette PR-ci:

  • les dictionnaires job-utilisateur sont visibles dans une colonne dans la page job/search
  • la colonne est cachée par défaut, l'option est modifiable dans les settings
  • les paires clé-valeur sont cliquables dans la colonne (quand elle est affichée), et le clic permet de filtrer uniquement les jobs qui ont la même paire clé-valeur (de la même manière qu'on filtre par job array en cliquant sur l'icône de l'array dans sa colonne). Je ne sais pas si cette fonctionnalité précise était nécessaire. Si non, je peux l'enlever.
  • J'ai ajouté deux tests pour l'affichage de la colonne et le filtrage.
  • J'ai gardé mon script store_huge_fake_data, car il permet d'insérer de grosses quantités de données, et donc de tester le serveur dans des cas extrêmes. Si nécessaire, ça aussi, je peux le déplacer dans une PR à part, ou simplement l'enlever.
  • Ce code contient aussi l'optimisation dans la petite PR Accélérer la récupération du statut "jobs are old" d'un cluster #184 . Je pense donc qu'il faut merger Accélérer la récupération du statut "jobs are old" d'un cluster #184 d'abord, et ensuite je pourrai rebaser celle-ci.

Voilà !

Allow to show/hide labels column in settings.
Allow to group by job labels on interface.
…if cluster jobs are old. Much faster to use than get_jobs().
…bs, --nb-dicts, --nb-props-per-dict

Add scripts for benchmarking
…IL as username to query requests, instead of randomly taking in server usernames.
Import users directly from fake_data.json.
…rops to jobs. It's irrelevant, as it must return raw data, not transformed data.
@notoraptor notoraptor changed the title Web user prop dicts in Dictionnaires job-utilisateur visibles dans l'interface web Mar 21, 2024
@soline-b soline-b merged commit 1000d19 into mila-iqia:master Mar 28, 2024
2 checks passed
@notoraptor notoraptor deleted the web-user-prop-dicts-in branch April 2, 2024 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants