-
Notifications
You must be signed in to change notification settings - Fork 103
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
feat: auto_validation function #2768
Conversation
Hello, J'ai créée cette PR suite aux discussions de l'issue #2600 . J'ai testé sur des observation en local , la fonction fonnctionne bien . Si vous avez des retours remarques , que ce soit sur la forme ou sur le fond n'hésitez pas . En tout cas je me suis appuyé sur les idées de l'issue mentionnée plus haut. |
fec216e
to
15751a9
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2768 +/- ##
========================================
Coverage 78.43% 78.44%
========================================
Files 88 89 +1
Lines 7183 7213 +30
========================================
+ Hits 5634 5658 +24
- Misses 1549 1555 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
af06fa9
to
57d4dd6
Compare
57d4dd6
to
3297b60
Compare
Il manquerait quelques lignes sur la validation automatique et comment elle fonctionne dans la documentation administrateur. |
@CodiumAI-Agent /review |
PR Analysis(review updated until commit 3297b60)
PR Feedback💡 General suggestions: The PR is well-structured and the changes are logically grouped. The use of Celery for scheduling the auto-validation task is a good choice. However, it would be beneficial to provide more comments in the code to explain the logic, especially in the SQL function and the Celery task. This would make the code easier to maintain and understand for other developers. ✨ Usage guide:Overview:
With a configuration file, use the following template:
See the review usage page for a comprehensive guide on using this tool. |
Persistent review updated to latest commit 3297b60 |
...e_validation/backend/gn_module_validation/migrations/9a4b4b6f8fe6_add_fct_auto_validation.py
Outdated
Show resolved
Hide resolved
@CodiumAI-Agent /improve |
...e_validation/backend/gn_module_validation/migrations/9a4b4b6f8fe6_add_fct_auto_validation.py
Show resolved
Hide resolved
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.
Il faut encore quelques changements sur les requêtes sur la BD (style SQLA 2.0). De plus, comme l'a dit @camillemonchicourt, il faut absolument faire de la documentation dans la doc GeoNature mais aussi dans le code des nouvelles méthodes/fonctions proposées (docstring).
...e_validation/backend/gn_module_validation/migrations/9a4b4b6f8fe6_add_fct_auto_validation.py
Show resolved
Hide resolved
- create migration alembic to create function auto_validation (gn_profiles.auto_validation) - create task celery in validation module - add parameters in validation toml config [Refs_ticket] PnX-SI#2600 Reviewed-by: andriacap
Add depends_on gn_commons migration alembic Reviewed-by: andriacap
Reviewed-by: andriacap
Move migration alembic to validation module Reviewed-by: andriacap
Test to check changes for "STATUT_VALID" on synthese observation with score = 3 and auto_validation = True and validation.id_validator is None Reviewed-by: andriacap
Rerviewed-by: andriacap
71d563d
to
f6878a4
Compare
A vérifier :
|
…dation configuration file
f6878a4
to
5555ff0
Compare
Fait
Fait
Fait
Fait
|
aa68ed4
to
be31bec
Compare
be31bec
to
47043f6
Compare
df62801
to
ef46352
Compare
...e_validation/backend/gn_module_validation/migrations/9a4b4b6f8fe6_add_fct_auto_validation.py
Outdated
Show resolved
Hide resolved
...e_validation/backend/gn_module_validation/migrations/9a4b4b6f8fe6_add_fct_auto_validation.py
Outdated
Show resolved
Hide resolved
...e_validation/backend/gn_module_validation/migrations/9a4b4b6f8fe6_add_fct_auto_validation.py
Outdated
Show resolved
Hide resolved
d539a35
to
40b0e1c
Compare
40b0e1c
to
517757c
Compare
[Refs_ticket] #2600
Reviewed-by: andriacap