search

jeremyomer / KidneyExchange.jl

MIT License
3 stars 2 forks source link

Proposition pour une nouvelle interface #2

Closed pnavaro closed 2 years ago

pnavaro commented 2 years ago

Bonjour Jeremy,

Je te propose une nouvelle interface pour ton package. Dans le programme actuel, le lien entre l'instance et la résolution n'est pas évident.

using KidneyExchange

instance = SparseUnos()

generate!(instance, 500, 25, 1)

solve(instance, 3, 4, BP(true, true))

solve(instance, 3, 4, MIP(KidneyExchange.EXTENDED_EDGE, true))

On pourrait créer des types SparseUnos, Saidman,... qui ne contiendraient que les noms des fichiers. Je peux le faire et proposer les modifs via une Pull Request si tu es d'accord. Il n'y a rien à modifier dans le programme actuel, juste ajouter quelques types et fonctions pour proposer cette interface.

jeremyomer commented 2 years ago

Salut Pierre,

Merci pour la suggestion. Le truc est que l'appli est plutôt pensée pour résoudre des instances stockées dans des fichiers que des instances générées aléatoirement. On génère des instances aléatoires pour nos tests faute de mieux. Je préfère l'interface actuelle car elle se réfère justement au nom des fichiers d'instance. J'y réfléchirai à nouveau plus tard, mais pour l'instant je préfère ne pas trop toucher à ça car Ayse a commencé à lancer les tests.

Par ailleurs, je reçois des messages d'échec de github action à chaque push à cause de la documentation semble-t-il et github m'a dit que j'avais utilisé tout mon quota de github actions pour des répos privés… Je ne sais pas si ces messages sont problématiques. Je vais ignorer pour le moment et m'en occuper quand on aura fini les tests.

Bonne fin d'après-midi,

Jérémy

Le 29 janv. 2022 à 12:17, Pierre Navaro @.***> a écrit :

Bonjour Jeremy,

Je te propose une nouvelle interface pour ton package. Dans le programme actuel, le lien entre l'instance et la résolution n'est pas évident.

using KidneyExchange

instance = SparseUnos()

generate!(instance, 500, 25, 1)

solve(instance, 3, 4, BP(true, true)

solve(instance, 3, 4, MIP(KidneyExchange.EXTENDED_EDGE, true))) On pourrait créer des types SparseUnos, Saidman,... qui ne contiendraient que les noms des fichiers. Je peux le faire et proposer les modifs via une Pull Request si tu es d'accord. Il n'y a rien à modifier dans le programme actuel, juste ajouter quelques types et fonctions pour proposer cette interface.

— Reply to this email directly, view it on GitHub https://github.com/jeremyomer/KidneyExchange.jl/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDR6X3MEAAFD3UIM7EF44TUYPEGBANCNFSM5NCYVCWA. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.

pnavaro commented 2 years ago

Je vais désactiver certaines vérifications. Merci pour ces précisions.