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

Merge folders #73

Merged
merged 19 commits into from
Feb 6, 2025
Merged

Merge folders #73

merged 19 commits into from
Feb 6, 2025

Conversation

jeanrjc
Copy link
Collaborator

@jeanrjc jeanrjc commented Feb 5, 2025

update to V2, to take into account the new models, where they are stored in a single folder, instead of 5.

@jeanrjc
Copy link
Collaborator Author

jeanrjc commented Feb 5, 2025

Salut @bneron,

on est passé en macysfinder 2.1.4, et on a maintenant ce warning, tu sais si on peut le désactiver ?

/home/jean/miniforge3/envs/defensefinder_dev/lib/python3.11/site-packages/macsypy/scripts/macsydata.py:56: UserWarning: GitPython is not installed, `macsydata init` is disabled ()
To turn this feature ON:
  - install git
  - then run `python -m pip install macsyfinder[model]` in your activated environment.
  warnings.warn("GitPython is not installed, `macsydata init` is disabled ()\n"

ou bien on rajoute gitpyhon ds les requirements ?
Merci

@bneron
Copy link
Contributor

bneron commented Feb 5, 2025

macsyfinder[model] c'est la cible d'installation pour ceux qui veulent créer des models et ca nécessite maintenant git et gitpython
si tu ne veux pas créer de nouveaux model (juste en utiliser) alors l'installation de macsyfinder par défaut pip install macsyfinder est suffisant et pas besoin de git/gitpython

@jeanrjc
Copy link
Collaborator Author

jeanrjc commented Feb 5, 2025

macsyfinder[model] c'est la cible d'installation pour ceux qui veulent créer des models et ca nécessite maintenant git et gitpython si tu ne veux pas créer de nouveaux model (juste en utiliser) alors l'installation de macsyfinder par défaut pip install macsyfinder est suffisant et pas besoin de git/gitpython

@bneron Et on peut supprimer le warning ? Je pense que ça va rendre confus les utilisateurs

@bneron
Copy link
Contributor

bneron commented Feb 5, 2025

non. si tu es modeleur tu es censé comprendre un peu ce que tu fais, le warning est important. je ne veux pas de quelque chose qui plante a runtime et qui soit incompréhensible.
si tu es utilisateur installe la version utilisateur il n'y aura pas de warning et personne ne sera perdu

@jeanrjc
Copy link
Collaborator Author

jeanrjc commented Feb 5, 2025

non. si tu es modeleur tu es censé comprendre un peu ce que tu fais, le warning est important. je ne veux pas de quelque chose qui plante a runtime et qui soit incompréhensible. si tu es utilisateur installe la version utilisateur il n'y aura pas de warning et personne ne sera perdu

@bneron Mais là on l'install en mode utilisateur et on a quand mm le warning. On l'a silencé ds le commit 2a381bc

@jeanrjc jeanrjc merged commit feace4b into master Feb 6, 2025
4 checks passed
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.

3 participants