Project

General

Profile

Actions

bug #10410

closed

Newly created distributions are added to computed description

Added by Katja Luther 11 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Highest
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal
Found in Version:

Description

Descriptions created in distribution editor sometimes added to computed description. This should not be possible.


Files

distribution.png (9.36 KB) distribution.png Katja Luther, 10/05/2023 08:43 AM

Related issues

Related to EDIT - feature request #7922: Add other possibilities to save distributions created with the distribution editor NewKatja Luther

Actions
Related to EDIT - feature request #5973: Implement DescriptionBase.isDefault in TaxEditorClosedKatja Luther

Actions
Actions #1

Updated by Katja Luther 11 months ago

  • Tags set to distributionEditor
Actions #2

Updated by Katja Luther 11 months ago

  • Assignee changed from Katja Luther to Andreas Müller

This should be fixed. Please review.

Actions #3

Updated by Andreas Müller 11 months ago

  • Related to feature request #7922: Add other possibilities to save distributions created with the distribution editor added
Actions #4

Updated by Andreas Müller 11 months ago

  • Status changed from New to Resolved

AM:

es ist zwar nicht falsch und auch gut, die computed descriptions auszuschließen, aber das wird im gegebenen Fall nichts bringen, denn die Endemismus Descriptions haben zwar im Titel das Wort „computed“ aber sie haben nicht das Flag/den Type computed, und das aus gutem Grund, weil sie lediglich beim Import berechnet wurden aus E+M Daten. Das computed Flag sollte aber eher für Daten benutzt werden, die sich immer wieder berechnen lassen und somit eher ein Cache darstellen, so wie das z.B. bei den berechneten Verbreitungsdaten auf Gattungsebene der Fall ist oder im Matrixbreich für die aggregierten Taxonbeschreibungen.

Nachhaltigere (aber auch etwas aufwendigere Lösungen) sind in #7922 beschrieben. Hier wurden 3 Lösungen angedacht:

  1. Verwendung des default Flags: dieses wurde mit #5875 im CDM implementiert, die Implementierung im TaxEditor fehlt aber noch (#5973). Vielleicht könnten wir das in diesem Kontext mal angehen. Idee: es gibt für jedes Taxon genau (oder maximal) eine default description, in der grundsätzlich alle Daten gespeichert werden, sofern sie nicht explizit in eine andere description gespeichert werden. Hier könnten auch die Verbreitungsdaten aus dem Verbreitungseditor gespeichert werden, wenn nicht explizit anders ausgewählt.
  2. Auswahl der Description erfolgt anhand der Quelle an der Description. Somit landen alle Daten, die zu einer bestimmten Quelle gehören alle in derselben Description. Eine explizite Angabe der Quelle in der Einzelverbreitung ist dann nicht mehr unbedingt notwendig, sofern man nicht explizite Angaben wie Seitenzahl, Links, etc. miteingeben will. Dies ist soweit mir bekannt auch die ursprüngliche Idee der Descriptions gewesen.
  3. Explizite Beschreibungen für den Distribution Editor: Alle mit dem Distribution Editor erzeugten Verbreitungsdaten kommen in eine explizit geflaggte Description. Von dort können sie bei Bedarf in andere Descriptions manuell verschoben werden.
  4. Explizite Beschreibung für alle Verbreitungsdaten: per default, werden alle Verbreitungsdaten in dieser Description angelegt, außer sie werden explizit wo anders hingeschoben.

Vermutlich gibt es auch noch mehr Möglichkeiten, aber ich denke mit diesen 4 könnten wir schon mal sehr gut leben. Und sie sollten auch alle nicht sehr aufwendig sein zu implementieren.

Eckhard, was wäre denn aus deiner E+M/Caucasus-Usersicht die präferierte Methode, wie entschieden werden soll, in welche Description eine neuangelegte Verbreitung des Verbreitungseditors geschoben werden sollte bzw. ob eine ganze neue Description angelegt werden soll?

kleiner Nachtrag: die „default“ description ist dabei i.d.R. die Description mit dem Label „Factual Dataset for {taxonname}“

Actions #5

Updated by Katja Luther 11 months ago

mail ERS:

Hallo zusammen,

ich versuche mich in diese Argumentation hineinzudenken und bin mir nicht ganz sicher, was für die Nutzer tatsächlich relevant ist. Zu den angedachten Lösungen habe ich folgende Kommentare:

1.) Genau eine default description: dies scheint mir sehr sinnvoll für Verbreitungsdaten aus dem Verbreitungseditor. Ich gehe davon aus, dass andere factual data sets (z.B. mit dem Editor hierhin verschobene Daten von anderen Taxa, andere sets wie common names) davon unberührt bleiben.
2.) Das mag funktionieren. Aber ich bin daran gewöhnt, dass für jede Einzelverbreitung eine explizit genannte Quelle existiert. Aus meiner Sicht ist dies also eher problematisch.
3.) Vielleicht nützlich zu sehen, welche Daten mit dem Distribution Editor erzeugt wurden, ja. Für den Output im Portal spielt es ja eh keine Rolle.
4.) Mir ist nicht ganz klar, was der Unterschied zu 1) ist.

Nochmals zur Illustration verschiedener Faktendaten: typisch ist in Euro+Med folgende Ansicht. Nach dem Verschieben und/oder Aggregation von Faktendaten finden sich Distribution-Angaben unter 2 bis 3 verschiedenen descriptions. Ist das so ok, oder stört es irgendwie? Die Distribution-Angaben von Hand zu verschieben, ist kaum machbar, da viel zu aufwändig für den Nutzer und ohne für uns erkennbaren Effekt.

Viele Grüße,

Eckhard

Actions #6

Updated by Andreas Müller 11 months ago

Actions #7

Updated by Andreas Müller 11 months ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther

This generally works, but if a new description needs to be created this description does not yet get flagged as "default" but should I guess. This is closely related to #5973#note-8

Actions #8

Updated by Katja Luther 11 months ago

New descriptions are set to default if no default description exists and if distribution editor creates new description and the preference is "always use default description" the newly created description is set to default as well.

Actions #9

Updated by Katja Luther 11 months ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Katja Luther to Andreas Müller

Please review.

Actions #10

Updated by Katja Luther 11 months ago

  • % Done changed from 0 to 40
Actions #11

Updated by Andreas Müller 11 months ago

  • Status changed from Resolved to Closed
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 40 to 100

Works as expected. We can implement some of the other 3 options later in case there is a request for it.

Actions #12

Updated by Katja Luther 8 months ago

There were still problems when the taxon already has descriptions but none of them is the default description, then the distribution is added to one of the existing descriptions.

This is fixed by creating a new default description if the preference is "Always use default description" and no default exists.

Actions

Also available in: Atom PDF