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

Floutage des données sensibles #2687

Merged
merged 12 commits into from
Jan 5, 2024

Conversation

bouttier
Copy link
Contributor

@bouttier bouttier commented Sep 5, 2023

Actuellement les données sensibles peuvent être exclus grâce au filtre de permission « Exclure les données sensibles ».
Cette PR ajoute un nouveau paramètre BLUR_SENSITIVE_OBSERVATIONS permettant de transformer ce filtre en « Flouter les données sensibles ».

Le floutage est implémenté dans la visualisation de la synthèse, et nouvellement des exports de la synthèse.

En raison de l’impact important sur les performances de la synthèse, le floutage est désactivé par défaut.

Secondairement, cette PR monte shapely en version 2, et apporte un schéma Marshmallow pour les observations de la synthèse SyntheseSchema.

Reprise de la PR #2571
Issue - #2558

@camillemonchicourt camillemonchicourt added this to the 2.14 milestone Sep 5, 2023
@bouttier bouttier force-pushed the feat/permissions-blurring branch from 00f1edc to 5eb7663 Compare September 6, 2023 14:27
@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (a23dde9) 77.97% compared to head (c26da9c) 78.44%.

Files Patch % Lines
...nature/core/gn_synthese/utils/query_select_sqla.py 90.90% 2 Missing ⚠️
backend/geonature/core/gn_synthese/models.py 87.50% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2687      +/-   ##
===========================================
+ Coverage    77.97%   78.44%   +0.47%     
===========================================
  Files           87       88       +1     
  Lines         7033     7182     +149     
===========================================
+ Hits          5484     5634     +150     
+ Misses        1549     1548       -1     
Flag Coverage Δ
pytest 78.44% <98.38%> (+0.47%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bouttier bouttier force-pushed the feat/permissions-blurring branch from 100f477 to f954be6 Compare September 8, 2023 09:30
@bouttier bouttier force-pushed the feat/permissions-blurring branch 3 times, most recently from baa19e7 to 6cd50de Compare December 18, 2023 11:04
@bouttier bouttier force-pushed the feat/permissions-blurring branch 5 times, most recently from ab94ab8 to a1d9f0b Compare January 4, 2024 23:37
@bouttier bouttier force-pushed the feat/permissions-blurring branch from a1d9f0b to c26da9c Compare January 4, 2024 23:54
@bouttier bouttier changed the title Feat/permissions blurring Floutage des données sensibles Jan 5, 2024
@jacquesfize jacquesfize merged commit f2e8e00 into PnX-SI:develop Jan 5, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants