acurus/pnvdb

Python wrapper for the NVDB Rest API

pnvdb's Issues

print() statement i util.py

Kaller API mer enn nødvendig

Det kan sees ut som API'et blir kallet mer enn nødvendig.

egengeometri funksjonen fungerer ikke

Bare returner data rått fra API'et så funker det.

Implementere vegnett endepunkt.

http://api.vegdata.no/endepunkt/vegnett.html

Kunne hente objekt basert på navn.

Denne bør egenentlig og fungere for egenskaper og verdier. Lage en generisk funksjon som oversetter?

xy funksjonalitet i vegreferanse

Logging spytter ut for mye data

Liker ikke at logging spytter ut hele objektet. Det er ikke nødvendig. kanskje bare id og type eller noe.

const.py

Hele systemet rundt denne filen må skrives om.

Tror det er like greit å hente informasjonen ned første gang man trenger den i en instance av pnvdb.

return data, istede for retun self.data

pnvdb/pnvdb/models/objekt.py

Line 185 in 21a369c

return data['stedfestinger']

SRID er nødvendig å implementere

Trenger den når man jobber med datafangst som vil ha geografiske koordinater.

kunne vi fått alle nvdb id'r og objekttype id'r til å bli returnert som int?

objekt må hente med parameter "inkluder=alle"

Ellers får jeg ikke med vegsegmenter

installerer ikke geojson automatisk

Må legge det inn i requirements et sted. Tror det var i setup.py?

NAME2ID - fungerer ikke som den skal

Problemer med skriverettigheter, og path.

Line 73 in util.py
with open("pnvdb/const.py", 'w') as f:
endres til
with open("../const.py", 'w') as f:

Angående rettigheter, må jeg google litt..

Les.posisjon()

Denne er ikke i orden.

Exception has occurred: UnboundLocalError
local variable 'payload' referenced before assignment
File "O:\1\Res\16080 Geodata\Brukere\Jan Tore\05_Dev\00_python\vegref2utm\vegref2utm_new.py", line 5, in
print(nvdb.posisjon('0200EV18HP3M10120'))

Implementer begrensning av geometri

Legg til "inkludergeometri=utledet" i kallet til objekter.

Les.hent() - henter ikke alt på en gang.

Les.hent() burde bruke "inkluder=alle"

Legg til funksjon for å sjekke om objekt_type objekt er i objektlista.

if objekt_type(98).objekt_liste():
do_code

html response 400 gir feilmelding "API seems dead"

print statement i util.py

linje 52. Fjern denne, og vurder om det skal legges inn logging.

egengeometri som nøkkel kan mangle

Om egengeometri ikke eksisterer i api-svaret vil denne funksjonen egengeometri i models/objekt.py feile.

forbedring av hent()

Legge til rette for mulighet til å styre hva som blir hentet av objektene.

endre payload variable navn til noe mer beskrivende.

Rydd opp i bruken av _fetchData()

Tror jeg lager en funksjon per endepunkt som henter data.

Da slipper jeg så mye repeterende kode rundt dette.

Implement logging

Skulle gjerne hatt logging for hvor mange kall som blir gjort.

Legge det i _fetch_data på debug level for eksempel.

Implementer "Dybde" i hent()

Kan være 0 som default verdi, men være tilgjengelig som parameter i hent-kallet.

Kunne hente verdi direkte fra objekt ved å oppgi egenskapstype og verdi.

Egenskapstype.verdi(egenskapstype=, verdi=) # id for begge deler.

pnvdb.models.ObjektType metadata property returns None

Returnerer None om den har hentet andre egenskaper tidligere.

Lage __repr__ (__str__) for alle hoved objekter.

Mulighet til å angi flere vegreferanser som parameter for Les.hent()

Henting av barn funker ikke før data er hentet.

>>> import pnvdb
>>> nvdb = pnvdb.Nvdb()
>>> obj = nvdb.objekt_type(95)
>>> obj.barn # Feiler

>>> obj.metadata
>>> obj.barn # funker

datakatalog version er hardkodet i datafangst objekt

Vegreferanse klassen håndterer ikke strekningsreferanser.

obj = nvdb.objekt(67,86520350)
for i in obj.vegreferanser:
    print(vegref.detaljert)

https://www.vegvesen.no/nvdb/api/v2/veg?vegreferanse=0300+Rv150+hp85+m12064-12178

Traceback (most recent call last):
  File "<pyshell#10>", line 2, in <module>
    print(i.detaljert)
  File "C:\Python27\ArcGISx6410.4\lib\site-packages\pnvdb\les.py", line 326, in detaljert
    self.data = self._fetch_data('veg', payload={'vegreferanse':self.vegreferanse})
  File "C:\Python27\ArcGISx6410.4\lib\site-packages\pnvdb\les.py", line 18, in _fetch_data
    data = self._check_response(resp, format)
  File "C:\Python27\ArcGISx6410.4\lib\site-packages\pnvdb\les.py", line 32, in _check_response
    raise ApiError(read_api_error(resp))
ApiError

Legg versionsnummer i __init__.py

Dokumentasjon på readthedocs

indexfila på readthedocs er utdatert.

print statement i objekt_typer

Hent() bør gi None om den genererer en tom generator.

Når data er hentet, sjekk om data['metadata']['returnert'] == 0. Om det er det, return None