search

FNSKtZH / apflora

Aktionsplan Flora der Fachstelle Naturschutz des Kantons Zürich
https://apflora.ch
Other
0 stars 1 forks source link

Qualitätskontrolle: noch Fehler bei Ansaatversuch #165

Closed csalzmann closed 7 years ago

csalzmann commented 7 years ago

Wenn Pop Ansaatversuch, darf keine der TPops aktuell sein

barbalex commented 7 years ago

Wo funktioniert das nicht?

Zur Präzisierung: TPops dürfen aktuell sein, solange sie angesiedelt sind. Sie dürfen bloss dann nicht aktuell sein, wenn sie ursprünglich sind.

Hier der code der existierenden Abfrage:

SELECT DISTINCT
  apflora.ap."ProjId",
  apflora.pop."ApArtId",
  'Population: Status ist "angesiedelt, Ansaatversuch", es gibt aber eine Teilpopulation mit Status "urspruenglich, aktuell":'::text AS "hw",
  ARRAY['Projekte', 1 , 'Arten', apflora.ap."ApArtId", 'Populationen', apflora.pop."PopId"]::text[] AS "url"
FROM
  apflora.ap
  INNER JOIN
    apflora.pop
    ON apflora.pop."ApArtId" = apflora.ap."ApArtId"
WHERE
  apflora.pop."PopHerkunft" = 201
  AND apflora.pop."PopId" IN (
    SELECT DISTINCT
      apflora.tpop."PopId"
    FROM
      apflora.tpop
    WHERE
      apflora.tpop."TPopHerkunft" = 100
  );

wobei:

csalzmann commented 7 years ago

Lieber Alex

Also umgekehrt formuliert:

Es gibt keine Fehlermeldung (obwohl es eine geben sollte), wenn Status der Pop Ansaatversuch ist und eine der TPop angesiedelt, aktuell ist. Das ist aber falsch, weil Status der Pop müsste dann auch angesiedelt, aktuell sein.

Dieser Fehler war z.B. bei Himantoglossum, Pop 138. Habe ich nun aber korrigiert.

Charlotte Salzmann Dr.sc.ETH topos Marti & Müller AG Idastr. 24 8003 Zürich Tel. 044 451 52 55 salzmann@toposmm.ch

Di - Fr

Am 09.01.2017 um 11:08 schrieb Alexander Gabriel notifications@github.com:

Wo funktioniert das nicht?

Zur Präzisierung: TPops dürfen aktuell sein, solange sie angesiedelt sind. Sie dürfen bloss dann nicht aktuell sein, wenn sie ursprünglich sind.

Hier der code der existierenden Abfrage:

SELECT DISTINCT apflora.ap."ProjId", apflora.pop."ApArtId", 'Population: Status ist "angesiedelt, Ansaatversuch", es gibt aber eine Teilpopulation mit Status "urspruenglich, aktuell":'::text AS "hw", ARRAY['Projekte', 1 , 'Arten', apflora.ap."ApArtId", 'Populationen', apflora.pop."PopId"]::text[] AS "url" FROM apflora.ap INNER JOIN apflora.pop ON apflora.pop."ApArtId" = apflora.ap."ApArtId" WHERE apflora.pop."PopHerkunft" = 201 AND apflora.pop."PopId" IN ( SELECT DISTINCT apflora.tpop."PopId" FROM apflora.tpop WHERE apflora.tpop."TPopHerkunft" = 100 ); wobei:

100 = "ursprünglich, aktuell" 201 = "Ansaatversuch" — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FNSKtZH/apflora/issues/165#issuecomment-271248097, or mute the thread https://github.com/notifications/unsubscribe-auth/AFxHTKIqBh1K_UE0UCMZgzZGkYqHQgdaks5rQgc7gaJpZM4LZgfh.

barbalex commented 7 years ago

welche der möglichen Werte wären also zulässig:

image

nur 201 = Ansaatversuch?

barbalex commented 7 years ago

übrigens: du muss das issue auf github anschauen, um das Bild mit den Werten zu sehen

barbalex commented 7 years ago

vermutlich müsste 202 auch zugelassen werden

barbalex commented 7 years ago

ich hab die Abfrage mal so angepasst - bitte sag mir, ob ich nun zuviel ausgeschlossen habe

csalzmann commented 7 years ago

Lieber Alex

Das verstehe ich nun nicht:)!

In meinen Worten:

Wenn Pop Ansaatversuch (201) ist, dann muss mind. 1 TPop auch Ansaatversuch (201) sein, wobei weitere TPops angesiedelt, erloschen (202, 211) oder potentieller Wuchsort (300) sein können. Andere Möglichkeiten für Pop (201) darf es nicht geben.

Charlotte Salzmann Dr.sc.ETH topos Marti & Müller AG Idastr. 24 8003 Zürich Tel. 044 451 52 55 salzmann@toposmm.ch

Di - Fr

Am 10.01.2017 um 21:16 schrieb Alexander Gabriel notifications@github.com:

welche der möglichen Werte wären also zulässig:

https://cloud.githubusercontent.com/assets/669783/21823088/fca06ab4-d779-11e6-805a-883c5eab88ee.png nur 201 = Ansaatversuch?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FNSKtZH/apflora/issues/165#issuecomment-271685602, or mute the thread https://github.com/notifications/unsubscribe-auth/AFxHTCGs5lONReSLUQwwAnRKrFK-6BQYks5rQ-cvgaJpZM4LZgfh.

kmarti commented 7 years ago

Liebe Charlotte

Entschuldige, wenn ich mich auch noch einmische. Vielleicht könnte man es auch so sagen: Eine Pop kann nur dann angesiedelt aktuell sein, wenn mindestens eine TP angesiedelt aktuell ist. D.h, wenn es nur Ansaatversuche oder erloschene TPs gibt (oder pot Wuchsorte), dann ist die Pop nicht angesiedelt aktuell.

Liebe Grüsse

Karin

Am 11.01.2017 um 08:41 schrieb csalzmann notifications@github.com:

Lieber Alex

Das verstehe ich nun nicht:)!

In meinen Worten:

Wenn Pop Ansaatversuch (201) ist, dann muss mind. 1 TPop auch Ansaatversuch (201) sein, wobei weitere TPops angesiedelt, erloschen (202, 211) oder potentieller Wuchsort (300) sein können. Andere Möglichkeiten für Pop (201) darf es nicht geben.

Charlotte Salzmann Dr.sc.ETH topos Marti & Müller AG Idastr. 24 8003 Zürich Tel. 044 451 52 55 salzmann@toposmm.ch

Di - Fr

Am 10.01.2017 um 21:16 schrieb Alexander Gabriel notifications@github.com:

welche der möglichen Werte wären also zulässig:

https://cloud.githubusercontent.com/assets/669783/21823088/fca06ab4-d779-11e6-805a-883c5eab88ee.png nur 201 = Ansaatversuch?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FNSKtZH/apflora/issues/165#issuecomment-271685602, or mute the thread https://github.com/notifications/unsubscribe-auth/AFxHTCGs5lONReSLUQwwAnRKrFK-6BQYks5rQ-cvgaJpZM4LZgfh.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

Karin Marti Dr.sc.nat.ETH topos Marti & Müller AG Idastr. 24 8003 Zürich Tel. 044 451 52 55 marti@toposmm.ch

barbalex commented 7 years ago

Also vorerst mal zu Charlottes Anliegen:

Ich habe die bestehende Qualitätskontrolle so angepasst, dass sie reklamiert, wenn es eine Teilpopulation gibt mit den folgenden Stati:

image

Akzeptiert werden Teilpopulationen mit diesen Stati:

image

Die Meldung lautet:

Population: Status ist "angesiedelt, Ansaatversuch", es gibt aber eine aktuelle Teilpopulation oder eine erloschene, die vor Beginn AP existierte:

Das sollte ab sofort funktionieren.

Ihr müsst dieses Issue in GitHub anschauen, damit ihr die Stati sieht.

barbalex commented 7 years ago

Nun zu Karins Input:

Entschuldige, wenn ich mich auch noch einmische. Vielleicht könnte man es auch so sagen: Eine Pop kann nur dann angesiedelt aktuell sein, wenn mindestens eine TP angesiedelt aktuell ist. D.h, wenn es nur Ansaatversuche oder erloschene TPs gibt (oder pot Wuchsorte), dann ist die Pop nicht angesiedelt aktuell.

Auf Ebene Population haben wir momentan diese Qualitätskontrollen:

Mit anderen Worten: Die von dir vorgeschlagene Kontrolle gibt es noch nicht. Ich brauche ca. 30 Minuten, um sie zu schaffen. Sagt mir, wenn ihr das wollt.