-
Notifications
You must be signed in to change notification settings - Fork 9
Feat/clustering mn #40
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
Conversation
|
Merci @MNIKIEMA ! Ce serait effectivement mieux. Dans library j'ai bien séparé la partie librairie python contenant des fonctions utiles (dans |
|
Le code respecte déjà la structure. Il a deux dossiers qui ne sont pas importables et j'ai gardé dans |
|
Pour moi ce n'est pas tout à fait pareil : ma suggestion était justement de sortir de src ces deux dossiers pour que src ne contienne que des fonctions importables, comme une vraie librairie (et encore il faudrait aussi sortir le main de agentic_data_policies__extraction). |
|
Ça peut se faire aussi. La structure actuelle du |
fraboniface
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci ! Ça mixe encore un peu trop scripts et code fonctionnel à mon goût mais pas très grave à ce stade. Il y a juste un lien cassé dans le README dont je peux me charger.
policy_analysis/README.md
Outdated
| ``` | ||
|
|
||
| [Voir ici pour plus d'infos](dspy_policies_extraction/README.md) | ||
| [Voir ici pour plus d'infos](src/dspy_policies_extraction/README.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mauvais lien, retirer le src
Refactorisation de la structure + ajout du clustering
src/pour standardiser les imports Python.data/archive_data/afin de séparer clairement données anciennes et nouvelles.Nouvelle organisation
Impact
library