Project

General

Profile

bug #9143

Distribution data sources are missing if multiple distributions exist

Added by Andreas Kohlbecker 26 days ago. Updated 22 days ago.

Status:
Feedback
Priority:
Highest
Category:
cdm-dataportal
Target version:
Start date:
07/10/2020
Due date:
% Done:

0%

Severity:
normal
Found in Version:
Tags:

Description

for the same area and the same status. This is the case in the below example where aggregated distributions withou sources are merged with explicit distributions with sources.

ERS:

etwas anderes ist jetzt komisch: einige Quellen für Verbreitungsdaten werden nicht (mehr) angezeigt, obwohl eindeutig vorhanden.

Vergleiche: (Aster alpinus) Czech Republic, France, Greece.... keine Quellen!

http://www.europlusmed.org/cdm_dataportal/taxon/229a548c-fec8-4f15-9058-8d340bb97c25

picture103-1.png View (128 KB) Andreas Müller, 07/10/2020 06:45 PM


Related issues

Related to Edit - feature request #4366: Transmissionengine Distribution: implement rules for source references Duplicate 08/21/2014

History

#1 Updated by Andreas Kohlbecker 26 days ago

  • Status changed from New to In Progress
  • Assignee changed from Andreas Kohlbecker to Andreas Müller

AM:

also da sind in der DB auch keine Quellen dran, da kann im Portal auch nichts rauskommen.

Übrigens sehen die Verbreitungsdaten mir aus, als ob die durch einen Import reinkamen oder so. Jedenfalls ist kein CreatedBy genannt. War aber erst am 27.11. letztes Jahr. Müssten wir nochmal checken, was da war, falls dir dazu nichts einfällt, Eckhard.

#2 Updated by Andreas Müller 26 days ago

so, jetzt seh ich das ein bisschen genauer. Das Problem ist, dass es sich um aggregierte Daten handelt:

Aber nicht nur. Z.B Czech Republic hat sowohl aggregierte Daten als auch Originaldaten an dieser Art. Für aggregierte Daten hatten wir mal gesagt, dass diese keine Quellen anzeigen sollen, also weder einen Link zu den darunter liegenden Taxa noch zu den Originalquellen. Das könnte aber geändert werden.

Das Problem ist allerdings, dass es wohl einen Fehler im Anzeigealgorithmus gibt, wenn das gleiche Areal 2x vorkommt, 1x mit Quellen und 1x ohne. Da werden die Quellen nicht zusammengezogen, wenn es sich um den gleichen Status handelt. Bei abweichendem Status ist das glaube ich gewollt, dass nur der „höchste“ Werte genommen wird und auch nur die Quellen von diesem. Bei gleichrangigen Quellen ist da aber natürlich Quatsch, da muss man die Quellen addieren.

#3 Updated by Andreas Müller 26 days ago

  • Subject changed from Aparranly distribution data source are missing to Distribution data sources are missing if multiple distributions exist
  • Description updated (diff)

#4 Updated by Andreas Müller 22 days ago

  • Assignee changed from Andreas Müller to Andreas Kohlbecker
  • Priority changed from New to Highest

The problem is already described in #4366. However, there it is handled as pure data aggregation (transmission engine) issue. But I don't think it is an aggregation issue. If data exists from aggregation and from explicit data sources should be combined and not taken either from one or the other distribution record (except for use-cases where this is required).

So the problem is in DescriptionUtility.filterDistributions(). There is a rule applied that computed distributions are prefered over explicit distributions:

        // 2) remove not computed distributions for areas for which computed
        //    distributions exists
        //
        if(preferComputed) {

So this means that even if the status of the computed distribution is less it is still taken (e.g. if computed -from taxonomic children- status is "introduced" and the explicit distribution is "native" the computed status "introduced" is taken) which I do not understand.

@Andreas K., as you have written this code do you remember why it was written like this?
@Eckhard: is there a rule why it should be like this?

The consequence of this rule is that the sources of explicit distribution are never shown if a computed distribution exists.

See also DescriptionUtilityTest.testFilterDistributions_computed() for the currently expected behavior.

#5 Updated by Andreas Müller 22 days ago

  • Status changed from In Progress to Feedback

#6 Updated by Andreas Müller 22 days ago

  • Related to feature request #4366: Transmissionengine Distribution: implement rules for source references added

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)