Project

General

Profile

Actions

feature request #4639

closed

Show quantitative data correctly in Data portal (for Cyprus portal)

Added by Andreas Müller about 9 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Priority14
Assignee:
Category:
cdm-dataportal
Start date:
Due date:
% Done:

0%

Estimated time:
Severity:
major

Description

In general we need to show quantitative data in the data portal somehow like (a)-b-c-e-(f) for extrema, min, max and average. Also count n, standard deviation etc. need to be shown.

However, for now urgently need to implement correct min and max handling for altitudes in the Cyprus portal. Currently these data are not in the feature tree to avoid showing them incorrectly, but most taxa do have altitude data.

"es gibt keine Specimens mit Höhenangaben. Es geht um kumulative Höhenangaben für fast alle zyprischen Arten. ... die Höhenangaben [wurden] in den Editor eingepflegt (Federführung A. Müller). Dazu gab es mit ihm und Dir schon ausführliche E-Mail-Austausche. Die Anzeige im Portal gebricht noch an marginalen Problemen, vor allem dem, wenn Minimum und Maximum gleich sind, also vor allem bei Küstenarten. Bei diesen sollte also nicht 0-0 m angezeigt werden, sondern nur 0 m o. ä.

" Ralf

Gewünsche Anzeige:

Altitudinal range

25–1050 m (_also bitte nicht mit Binde-, sondern mit Bis-Strich.

Maximum und Minimum also mit kumulierter Literaturangabe._)

siehe auch attachment

Zu klären:

  • Wenn Maximum und Minimum gleich sind, ist es unsinnig 25-25 m zu schreiben. Kann man das so programmieren, dass dann einfach nur „25 m“ angezeigt wird? Ist zwar kein Range mehr, aber damit muss man leben. Ist auch vergleichsweise selten.

  • Wenn Maximum und Minimum beide 0 m sind (bei vielen Küstenpflanzen), wäre zu klären, ob man dann nicht „at sea level“ schreiben könnte. 0 m sieht blöd aus und impliziert beim unbedarften Betrachter einen Fehler. Aber wenn das zuviel Arbeit macht, muss eben 0 m da stehen.

Arten mit Höhenangaben:

  • Salvia verbenaca

  • Salvia veneris

  • Salvia fruticosa

  • Trifolium argutum (sealevel example)

  • Trifolium arvense

  • Trifolium lappaceum

  • Bosea cypria

see also #4031

Actions #1

Updated by Andreas Kohlbecker about 9 years ago

  • Status changed from New to In Progress

implemented (r22755) generically not nicely compacted as first step

Actions #2

Updated by Andreas Kohlbecker about 9 years ago

another commit I forgot to mention:

adding missing statisticalValues to init strategy r22758

Actions #3

Updated by Andreas Kohlbecker about 9 years ago

was machen wir denn mit den möglichen 'modifiers' die an DescriptionElement selbst aber auch an den einzelnen statistical values hängen können? Sollen die einfach hinten angehängt werden?

Actions #4

Updated by Andreas Kohlbecker about 9 years ago

implemented by r22760

all modifieres belonging to the min max average values are just appended to the string.

TODO: check for inconsistent cases, eg. only Max and average given

Actions #5

Updated by Andreas Müller about 9 years ago

Replying to a.kohlbecker:

was machen wir denn mit den möglichen 'modifiers' die an DescriptionElement selbst aber auch an den einzelnen statistical values hängen können? Sollen die einfach hinten angehängt werden?

die an DescriptionElement selbst

Werden die denn bei anderen DescElements bereits ausgewertet? Wenn dann sollten wir das allgemein angehen.

aber auch an den einzelnen statistical values

Ich würde sie eher davor stellen. Typischerweise sind das ja so Wörter wie "ca.", "~", "meistens", ... und die schreibt man ja eher vorne dran. Um das genau zu bestimmen bräuchte wir sonst noch eine weitere Differenzierung im Model.

Actions #6

Updated by Andreas Müller about 9 years ago

Replying to a.kohlbecker:

implemented by r22760

all modifieres belonging to the min max average values are just appended to the string.

Please do prepend, not append.

Actions #7

Updated by Andreas Kohlbecker about 9 years ago

  • Status changed from In Progress to Resolved
  • Assignee changed from Andreas Kohlbecker to r.hand -

Replying to a.mueller:

Replying to a.kohlbecker:

implemented by r22760

all modifieres belonging to the min max average values are just appended to the string.

Please do prepend, not append.

Ok, i implemented it that way: r22778

so the implementation of this feature is ready and can be revied!

Actions #8

Updated by Andreas Müller about 9 years ago

I just checked this in production by adding the altitudinal range to the feature tree.

Everything to me as a non-taxonomist looked fine. However, the 2 extra cases seem to be not yet implemented. These are min = max and min = max = 0; they still show up as 25-25 m or 0-0 m respectively.

An example is Trifolium argutum.

After testing I removed the feature from the feature again as this is something Ralf needs to decide.

Actions #9

Updated by Andreas Müller about 9 years ago

  • Target version set to cdm_dataportal RELEASE 3.5.0
Actions #10

Updated by Andreas Kohlbecker about 9 years ago

  • Assignee changed from r.hand - to Andreas Müller

Replying to a.mueller:

I just checked this in production by adding the altitudinal range to the feature tree.

Everything to me as a non-taxonomist looked fine. However, the 2 extra cases seem to be not yet implemented. These are min = max and min = max = 0; they still show up as 25-25 m or 0-0 m respectively.

this pug is now fixed! git: 441f7184

but the bugfix will only be available in release 3.5.1

Actions #11

Updated by Andreas Müller about 9 years ago

  • Assignee changed from Andreas Müller to r.hand -

The min/max problem seems to be fixed.

„at sea level“ is not yet fixed, but probably difficult to handle because the modul is for quantitative data in general and not only for altitudinal ranges.

I give review back to Ralf.

Actions #12

Updated by Andreas Müller about 9 years ago

  • Target version changed from cdm_dataportal RELEASE 3.5.0 to cdm_dataportal RELEASE 3.5.1
Actions #13

Updated by Andreas Kohlbecker about 9 years ago

Replying to a.mueller:

The min/max problem seems to be fixed.

„at sea level“ is not yet fixed, but probably difficult to handle because the modul is for quantitative data in general and not only for altitudinal ranges.

it would be possible to implement this as a special case for the FeatureType Altitudinal Range

Actions #14

Updated by Andreas Müller about 9 years ago

be aware of i18n issues when implementing "at sea level"

Actions #15

Updated by Andreas Kohlbecker about 7 years ago

  • Description updated (diff)
  • Status changed from Resolved to Closed

Hallo Andreas,

aus Zypern-Sicht dürfte alles umgesetzt sein. Was ich auf die Schnelle nicht überprüfen kann, sind Fälle, wo Min. und Max. gleich sind. Der "alte" Fall Trifolium argutum passt leider nicht mehr, da es bei der Art inzwischen ungleiche Werte gibt. Bei den Strandarten, wo Min. und Max. 0 m sind, wird aber offenbar nur einmal 0 m angezeigt. Beispiel: Maresia nana.

In den letzten Monaten habe ich bei den Höhenangaben nur einzelne geändert. Dabei fielen aber sonst keine Probleme auf. Thema kann also zunächst mal ruhen.

Viele Grüße
Ralf

Actions #16

Updated by Andreas Kohlbecker about 7 years ago

Hallo Ralf,

danke für die Kontrolle.

Bei den Strandarten, wo Min. und Max. 0 m sind, wird aber offenbar nur einmal 0 m angezeigt.
Das ist bewusst so gemacht, um um die Anzeige zu entschlacken. Min: 0m, Max 0m bringt meines Erachtens keinen Zugewinn an Information gegenüber 0m

Viele Grüße
Andreas

Actions #17

Updated by Andreas Kohlbecker about 7 years ago

  • Private changed from Yes to No
Actions

Also available in: Atom PDF