Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Visning av enkeltminner for en lokasjon #132

Open
havardgj opened this issue Oct 5, 2015 · 23 comments
Open

Visning av enkeltminner for en lokasjon #132

havardgj opened this issue Oct 5, 2015 · 23 comments
Labels
Milestone

Comments

@havardgj
Copy link

havardgj commented Oct 5, 2015

Ved visning av polygone(er) for lokasjon - også vise polygoner for enkeltminner for samme lokasjon.

Ved klikk på polygon - gå til detaljvisning for enkeltminne med navn, beskrivelser og kategori (enkcatlabel)

Spørring for å hente enkeltminner for en lokasjon:

SELECT ?enk ?name ?desc ?area ?enkcatlabel
where {
?enk a https://data.kulturminne.no/askeladden/schema/Enkeltminne .
?enk rdfs:label ?name .
?enk https://data.kulturminne.no/askeladden/schema/i-lokalitet https://data.kulturminne.no/askeladden/lokalitet/173268 .
?enk https://data.kulturminne.no/askeladden/schema/beskrivelse ?desc .
?enk https://data.kulturminne.no/askeladden/schema/geo/area/etrs89 ?area .
?enk https://data.kulturminne.no/askeladden/schema/enkeltminnekategori ?enkcat .
?enkcat rdfs:label ?enkcatlabel .
}

https://sparql.kulturminne.no/?default-graph-uri=&query=SELECT+%3Fenk+%3Fname+%3Fdesc+%3Farea+%3Fenkcatlabel%0D%0Awhere+%7B%0D%0A%3Fenk+a+%3Chttps%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Fschema%2FEnkeltminne%3E+.%0D%0A%3Fenk+rdfs%3Alabel+%3Fname+.%0D%0A%3Fenk+%3Chttps%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Fschema%2Fi-lokalitet%3E+%3Chttps%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Flokalitet%2F173268%3E+.%0D%0A%3Fenk+%3Chttps%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Fschema%2Fbeskrivelse%3E+%3Fdesc+.%0D%0A%3Fenk+%3Chttps%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Fschema%2Fgeo%2Farea%2Fetrs89%3E+%3Farea+.%0D%0A%3Fenk+%3Chttps%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Fschema%2Fenkeltminnekategori%3E+%3Fenkcat+.%0D%0A%3Fenkcat+rdfs%3Alabel+%3Fenkcatlabel+.%0D%0A%7D&format=application%2Fsparql-results%2Bjson&timeout=0&debug=on

@havardgj havardgj added this to the Pri 2 milestone Oct 5, 2015
@havardgj havardgj modified the milestones: Pri 3, Pri 2 Oct 19, 2015
@atlefren
Copy link
Contributor

La meg se om jeg har forstått dette riktig.

  • Dette gjelder riksantikvaren sparql?
  • VI beholder logikken i dag med å vise polygoner for enkeltminner knyttet til lokasjonen
  • Klikk på lokasjonen (punkt) fungerer som i dag
  • Vi endrer følgende: ved klikk på enkeltminne-polygon viser vi i dag data for lokasjonen, men vi skal nå kjøre en spørring med enkeltminne-id og vise informasjon om enkeltminnet?

@havardgj
Copy link
Author

  • Ja - dette gjelder kun riksantikvaren SPARQL
  • Det vi gjør i dag er vel ikke å vise polygoner for enkeltminner, men alle polygoner for en lokasjon? Her er et eksempel på en lokasjon (blå) med enkeltminnepolygoner (rosa) (Trondenes russefangeleir (174603)):
    image001
  • Logikken blir vel at en ved på lokasjons-polygon viser henter enkeltminner og viser disse.

Så lenge begge er polygoner og de er organisert sånn noen lunde som i eksempelet over, så er det en ting. Men jeg tror at det også kan være tilfeller der det bare finnes punktinformasjon. Hvis en også skal håndtere dette - hva er realistiske alternativen da?

atlefren added a commit that referenced this issue Nov 11, 2015
@havardgj
Copy link
Author

Dette ser veldig, veldig lovende ut. Såpass detaljerte områdedata gir jo visningen en ny dimensjon.

For å rydde litt sånn rent visuelt - kunne det vært en ide å fjerne punktmarkører nå polygon vises?

@havardgj
Copy link
Author

Og da må kanskje valgt polygon - enkeltminne eller lokasjon - få en eller annen visuell markering? Når en mister fargeendringer på punktmarkøren?

atlefren added a commit that referenced this issue Nov 12, 2015
@atlefren
Copy link
Contributor

@havardgj det er en tanke!

Med "når polygon vises", tenker du da lokalitetspolygon eller enkeltminne-polygoner?

@havardgj
Copy link
Author

Lokalitetspolygon i utgangspunktet. Og logikken må vel da knyttes til om polygon faktisk finnes. Det samme gjelder vel i prinsippet for enkeltminner, der det i teorien også kan væe enkeltminner som ikke har polygoner.

Det kan være et praktisk problem at lokasjonspolygonene som dukker opp i noen få tilfeller kan være veldig små og derfor vanskelige se rett og slett - det har vært snakk om at f.eks. en flaggstang finnes som polygon, men hypotesen er at dette er et mindre problem som en evt. .må håndtere senere.

@atlefren
Copy link
Contributor

Jeg har prøvd noen timer nå på å få til dette med å skjule markører når polygon vises. Det viste seg å ikke være så lett, mest på grunn av at vi også bruker clustring på markører, og markercluster-pluginen skaper en del problemer.

Jeg har dermed gitt opp dette for nå, men noe fikk jeg gjort: koden for å håndtere enkeltminner og polygoner for lokasjoner har blitt ryddet opp og lettere å forstå. Du får ta en avveiing på om jeg skal bruke mer tid på dette med å skjule polygoner. Et problem med å skule markører sånn rent praktisk er også at noen polygoner ligger helt under andre, og blir dermed ikke klikkbare.

En annnen ide er jo å gjøre med lokasjonspolygoner som med enkeltminner: kun vise dem ved klikk. Da unngår vi å ha mye overlappende data.

@havardgj
Copy link
Author

Her er en spørring som gjør det mulig å hente ut polygoner for flere lokasjoner i en spørring:

select ?lok ?poly where {
?lok https://data.kulturminne.no/askeladden/schema/geo/area/etrs89 ?poly
filter (?lok in (https://data.kulturminne.no/askeladden/lokalitet/173268,https://data.kulturminne.no/askeladden/lokalitet/103673 ,https://data.kulturminne.no/askeladden/lokalitet/155051))}

Forhåpentligvis kan denne effektivisere eksisterende løsning og kanskje også gjøre det mulig å hente alle polygoner for alle lokasjoner på laveste zoomnivå - selv om det i noen tilfeller kan være mer enn grensen på 10 som er brukt i eksisterende logikk.

@havardgj
Copy link
Author

Hovedproblemet med punkter og polygoner er at det er vanskelig å forstå hvilke polygoner som hører til hvilke punkt. Og et annet sentralt problemet er som du sier at noen polygoner ikke er klikkbare ettersom de ligger bak andre polygon.

Samtidig ser det ut til at punktmarkørene på en måte løser det andre problemet - de er alltid klikkbare og så vidt jeg forstår så gjør de det da praktisk mulig å nå alle lokasjoner - selv om ikke alle polygoner kan klikkes på.

Vil det da være en løsning å se litt er på det visuelle i den løsningen du har nå? Hvis en kan markere både polygoner og punkt så blir det mulig å se sammenhengen mellom den. Og da skal det være mer forståelig å bruke punktene for valg av polygoner.

  • Endre farge på polygon ved klikk på polygon (grønn - opasitet som nå?)
  • Endre farge på tilhørende punkt ved klikk på ploygon
  • Endre farge på polygon ved klikk på tilhørende punkt
  • Reversere fargeendring ved klikk på annet polygon eller punkt

atlefren added a commit that referenced this issue Nov 27, 2015
atlefren added a commit that referenced this issue Nov 27, 2015
@atlefren
Copy link
Contributor

Da har jen en første versjon av dette på plass

Se

http://development.atlefren-kulturraadet.divshot.io/demonstratorer/config.html?datasets=riksantikvaren&bbox=15.9521484375%2C68.58183319828414%2C18.720703125%2C69.38225823683517&layer=norges_grunnkart_graatone#18/68.8245/16.563:kulturminnedataSparql_https%3A%2F%2Fdata.kulturminne.no%2Faskeladden%2Fenkeltminne%2F174603-54

Jeg har måttet gjøre om litt på clustringa: under zoomnivå 13 clustres ikke riksantikvaren-laget

Det gjenstår også noen ting

  • enkeltminner blir værende igjen
  • Logikk når man skrur av kartlaget med polygoner
  • Kodeopprydding

Spørringen for å hente polygoner for en rekke lokasjoner var "god-sent" her!

Tar gjerne imot feedback her

@havardgj
Copy link
Author

Synes dette begynner å fungere bra - basert på forutsetningene. Og selv om det kanskje er tekniske grunner til å fjerne clustering på noen nivå, så er det jo også med på å redusere den visuelle kompleksiteten - en type innhold mindre å forholde seg til.

Rent visuelt så blir en del av gruppene med punktmarkører veldig kompakte og det kan være litt vanskelig å se/forstå hva som er de enkelte markørene. Er det mulig å sette en viss gjennomsiktighet på disse markørene - når det ikke clustres?

Et annet spørsmål er hvordan dette i praksis fungerer med bilder for noen av lokalitetene - ref #135? Her jobber jeg med å få tilpasset spørringen slik at en kun får ett objekt for hver lokasjon - selv om det finnes flere bilder. Ser at det i alle fall fort blir forvirrende om en får flere lokalitetspunkt for en lokalitet, så kanskje greit å vente med å teste dette til denne spørringen er på plass?

@havardgj
Copy link
Author

Ser nå at dette er satt opp og fungerer med bilder - det var bare ingen bilder i dette området. Så da kan dette med bilder og justering av spørring heller håndteres i #135.

@havardgj
Copy link
Author

Har testet logikken litt rundt Oscarsborg, for å sjekke med det som ser ut til å være et av de vanskeligste casene.

http://development.atlefren-kulturraadet.divshot.io/demonstratorer/config.html?datasets=riksantikvaren&bbox=15.9521484375%2C68.58183319828414%2C18.720703125%2C69.38225823683517&layer=norges_grunnkart_graatone#13/59.686/10.6062

Som du ser i denne filmen så er det et par litt uheldige ting som det kanskje er mulig å se på?

  • Polygon for lokalitet forsvinner når man zoomer inn (og punkt for lokalitet forsvinner ut av synlig område?).
  • Lokalitet ser ut til å velges automatisk når man zoomer litt inn og ut. Ser kanskje ut til at valg av lokalitet på et zoomnivå henger igjen på en eller annen måte?

Litt usikker på hva som er løsningen og hvor mye logikk man bør legge inn for det som vel strengt tatt burde vært en geografisk spørring mot datasettet av typen gi meg alle lokaliteter som har punkt, hele eller deler av området innenfor denne boksen. Men hadde det vært mulig å beholde lokalitetspolygoner som er vist - sammen med enkeltminnepolygonene som vises nå - f.eks. helt til man zoomer ut til nivået der clustering aktiveres igjen?

https://www.dropbox.com/s/2qgqsaxv4gf5n72/Screen%20Recorded2015-11-30%2012_35_34.mov?dl=0

@atlefren
Copy link
Contributor

atlefren commented Dec 7, 2015

Denne issuen var ikke enkel. Har knøla rundt med mye nå, men fikk tilslutt fikset feilene du oppdaget.

Jeg synes løsningen er relativt treg, men det skal ihvertfall fungere. Fint om du kan re-teste @havardgj

@havardgj
Copy link
Author

havardgj commented Dec 8, 2015

Synes det interaksjonsmessig fungerer mye bedre nå. Men ser også at det kanskje virker litt treigt. Har du noen ide om hvor dette ligger? Er det datamengde, bearbeiding av data eller rendring på en eller annen måte?

Rent visuelt så er det eneste jeg lurer på muligheten for å sette en gjennomsiktighet for punktmarkørene for å gi et litt riktigere bildet av innholdet ved overlappende markører. Var det mulig?

@havardgj
Copy link
Author

havardgj commented Dec 8, 2015

Når logikken for visninger av polygoner nå er litt mer forutsigbar - vil det da hjelpe noe å sette ned igjen grensen for antallet punkter før polygoner skal vises?

@atlefren
Copy link
Contributor

atlefren commented Dec 8, 2015

Bra!

Jeg skrev om ganske mye, slik at nå hentes polygoner for alle punkter synlige i (og 20% utenfor) kartet. Dette gjør at det blir ganske mye lasting av data (polygoner på hver zoom/pan).

Jeg mistenker tregheten skyldes dette, pluss at vi nå ikke clustrer når man er zooma inn. Dette medfører mange punkter i kartet.

Nå har vi jo en spørring for alle polygoner, så logikken med antall punkter er fjernet, jeg går kun på zoomnivå. Tror nok det kan være en ide å gjen-innføre antall punkter for å få ting til å gå litt raskere.

Skal teste litt mer!

atlefren added a commit that referenced this issue Dec 9, 2015
@atlefren
Copy link
Contributor

atlefren commented Dec 9, 2015

Da har jeg gjort en sjau her!

Det var som sagt mye rart, men nå har jeg skrevet om slik at den "unclustrer" og viser polygoner når det er mindre enn X (der x by default er 20) "features" i kartet.

Dette gjorde ting mye raskere, og jeg klarer ikke å se at jeg har "ødelagt" annen funksjonalitet heller. Kan testes på divshot, men om du ikke finner noe veldig graverende har jeg lyst til å parkere denne nå.

(cc @havardgj )

@atlefren atlefren added the review label Dec 9, 2015
@havardgj
Copy link
Author

Litt usikker på hva som ble siste versjon her. Er det nå en kombinasjon av zoom-nivå og antall punkt? Eller gikk du tilbake til kun antall "features"?

@havardgj
Copy link
Author

For å kompensere litt for de manglende geospørringene hadde det vært interessant å se om det var mulig å prøve å justere innslagspunktet for visning av polygoner - enten ved å øke antall fra 20 til f.eks. 40 eller sette visningen for ett zoomnivå høyrer - avhengig av hva som er mulig og hva som er best med den logikken som er der nå.

@atlefren
Copy link
Contributor

Pr nå er det basert på zoomnivå, siden vi nå kan hente en rekke polygoner i samme spørring er det ikke noe problem med antall mer

@havardgj
Copy link
Author

Så vidt jeg kan se vises polygoner nå på zoomnivå 16. Hadde det da vært mulig å teste om det fungerer å vise dem allerede på nivå 16?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants