Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Pierre-Narcisi
Copy link
Contributor

@Pierre-Narcisi Pierre-Narcisi commented Jan 15, 2025

Revert feat added in #3293

Celle-ci posait en effet des soucis de performances car on comptait le nombre d'occurrences de taxons et d'habitat de chaque JDD à chaque fois qu'on appelait la route /datasets, donc partout où on affiche ou liste des JDD.

Il faut que la propriété qui compte les occurrences soit optionnelles et calculée que si explicitement demandée, comme c'est déjà le cas avec la propriété synthese_records_count utilisée uniquement quand on liste les JDD d'un CA dans le module métadonnées (https://github.com/PnX-SI/GeoNature/blob/master/frontend/src/app/metadataModule/metadata-dataset.component.html#L75)

@Pierre-Narcisi Pierre-Narcisi marked this pull request as draft January 15, 2025 08:57
@Pierre-Narcisi Pierre-Narcisi marked this pull request as ready for review January 15, 2025 08:57
@codecov
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.93%. Comparing base (6f6b91a) to head (954a5cf).
Report is 6 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3320   +/-   ##
========================================
  Coverage    83.93%   83.93%           
========================================
  Files          122      122           
  Lines         9715     9711    -4     
========================================
- Hits          8154     8151    -3     
+ Misses        1561     1560    -1     
Flag Coverage Δ
pytest 83.93% <ø> (+<0.01%) ⬆️

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.

@jacquesfize jacquesfize merged commit 5e166f3 into develop Jan 15, 2025
9 checks passed
@jacquesfize jacquesfize deleted the fix/dataset_count branch January 15, 2025 09:34
@camillemonchicourt camillemonchicourt changed the title Fix(model): temporary fix for dataset counts of habs and taxs Revert(model): revert dataset counts of habs and taxs Jan 15, 2025
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