ajnisbet / opentopodata

Open alternative to the Google Elevation API!
https://www.opentopodata.org
MIT License
323 stars 70 forks source link

Incorrect example in docs #40

Closed CramericaIndustries closed 3 years ago

CramericaIndustries commented 3 years ago

Hello and thank you for that wonderful project!

I found a mistake in your documentation where you describe how to define a multi dataset in config.yaml file. https://www.opentopodata.org/notes/multiple-datasets/

This didn't work for me:

# Hi-res New Zealand.
- name: nzdem8m
  path: data/nzdem8m/
  filename_tile_size: 65536
  filename_epsg: 2193

# Mapzen global.
- name: mapzen
  path: data/mapzen/

# NZ with mapzen fallback.
-name: nz-global
  - child_datasets:
    - nzdem8m
    - mapzen

I had to change the nz-global configuration part to:

# NZ with mapzen fallback.
- name: nz-global
  child_datasets:
    - nzdem8m
    - mapzen

There was a space missing between the first dash and name: -name: nz-global => - name: nz-global and I had to remove the dash before child_datasets

ajnisbet commented 3 years ago

Thanks for finding that! yaml is so fussy, fixed.

CramericaIndustries commented 3 years ago

- child_datasets: row also must be altered to child_datasets: (no dash)

Blackfishbox commented 3 years ago

In the instructions to install without docker it has to be pip3 install -r requirements.txt instead of pip3 install requirements.txt (you forgot -r)

ajnisbet commented 3 years ago

Fixed, thanks!