Skip to content

docs(dem): v2 (MAPCO-8937)#163

Open
shimoncohen wants to merge 13 commits into
masterfrom
new-dem
Open

docs(dem): v2 (MAPCO-8937)#163
shimoncohen wants to merge 13 commits into
masterfrom
new-dem

Conversation

@shimoncohen

Copy link
Copy Markdown
Contributor

No description provided.

@shimoncohen shimoncohen self-assigned this Feb 17, 2026
@shimoncohen shimoncohen added the documentation Improvements or additions to documentation label Feb 17, 2026
@shimoncohen shimoncohen changed the title docs(dem): v2 docs(dem): v2 (MAPCO-8937) Feb 17, 2026
@github-actions

github-actions Bot commented Feb 17, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://MapColonies.github.io/developer-portal/pr-preview/pr-163/

Built to branch gh-pages at 2026-06-04 16:01 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

<WCS_SERVICE_URL>/wcs?request=GetCapabilities&token=<token>
```

<details>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove mimad from GetCapabilites response

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? I want to show a response with a few layers.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shimoncohen it's better to include only publicly available resources in docs and since mimad is not one of them it should be removed.
furthermore, it may confuse clients between the deployed/served data to data depicted in docs which may have different values (outdated/errors/...)

Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/ogc/protocols/ogc-wcs.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Services/catalog/profile_v2_0.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Services/catalog/profile_v2_0.md
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
Comment thread docs/MapColonies/DEM/Services/download/README.md
asafmas-rnd
asafmas-rnd previously approved these changes Feb 24, 2026
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md
| mc:links | text | Available links for different services available for the current product |
| mc:classification | enum | product classification / confidentiality <br /> Classification values |
| mc:producerName | text | manufacturer / organization that produced / supplied the product |
| mc:noDataValue | enum | **Valid Values**: <br/> -32768 |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noDataValue is a number (double)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be int?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, it's double

Comment thread docs/MapColonies/DEM/Services/catalog/profile_v2_0.md Outdated
Comment thread docs/MapColonies/DEM/Services/catalog/profile_v2_0.md Outdated
Comment thread docs/ogc/protocols/ogc-wcs.md Outdated
Comment thread docs/MapColonies/DEM/Guides/height_extraction.md Outdated
Comment thread docs/ogc/protocols/ogc-wcs.md Outdated
Comment thread docs/ogc/protocols/ogc-wcs.md Outdated
Comment thread docs/ogc/protocols/ogc-wcs.md Outdated
Comment thread docs/ogc/protocols/ogc-wcs.md Outdated
@shimoncohen shimoncohen requested a review from asafmas-rnd June 4, 2026 15:16
url:
'<3D_CATALOG_SERVICE_URL>/csw'

body (XML):

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider remving the terrain guide "Get terrain provider URI (Step 2.1, optional)" from here and direct to DEM V1 terrain_provider guide

The following guide will help you understand ***Step-by-step*** how to work with the Map Colonies Height Extraction service, from the [Catalog](/docs/MapColonies/DEM/Services/catalog/dem-catalog) to the WCS protocol.

:::warning
This guide is **not** for viewing purposes.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for algo... (be positive)

@@ -0,0 +1,965 @@
---

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Decide if and where to put the WFS section

import Tabs from '@theme/Tabs';
import TabItem from '@theme/TabItem';

## Step by step guide

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please emphasize that "coverage" refers to V2 catalog

Query **DEM CSW catalog** service to find item(s) according to desired filter [examples are here](/docs/ogc/protocols/ogc-csw/ogc-csw-examples)

Filter should be based on [DEM profile](/docs/MapColonies/DEM/Services/catalog/catalog-profile-v1)
Filter should be based on [DEM profile](/docs/MapColonies/DEM/Services/catalog/dem-catalog-profile-v1)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please emphasize that "coverage" refers to the V1 catalog only

@vitaligi vitaligi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please also fix Catalogue typo in '../DEM/Services/catalog/dem-catalog'
ignore

```bash
curl --location --request POST '{DEM_CATALOG_SERVICE_URL}/csw?token=<token>' \
--header 'Content-Type: application/xml' \
--data-raw '<?xml version="1.0" encoding="UTF-8"?>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this md file contains mixed version requests align to a specific version and add a notice on the given version at the top

| mc:links | text | Available links for different services available for the current product |
| mc:classification | enum | product classification / confidentiality <br /> Classification values |
| mc:producerName | text | manufacturer / organization that produced / supplied the product |
| mc:noDataValue | enum | **Valid Values**: <br/> -32768 |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, it's double

@@ -189,14 +189,14 @@ Now, after you got all product metadata that you need by querying our Catalog, l
// **Optional** add to Cesium terrain provider in order to clamp 3d models to the ground or investigate terrain

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove blank space at EOL

:::warning
New version [v2](/docs/MapColonies/Raster/services/catalog/raster-catalog-profile-v2) is coming!
<p style={{color:"red"}}><b>The following parameters are depracted an will be removed in v2</b></p>
<p style={{color:"red"}}><b>The following parameters are deprecated and will be removed in v2</b></p>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove space at EOL

| mc:type | enum | type of the catalog <br/> **Value**: RECORD_DEM |
| mc:keywords | text | list of key words relevant for product |
| mc:verticalType | enum | **Valid Values**: Orthometric, Ellipsoidal |
| mc:verticalSrsId | text | reference System ID (EPSG), <br/> ex: 4326 / 3857 |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vertical srs ids examples are not vertical. use 5773 / 3855 / 4979.
plus, system can be lowercase

| mc:description | text | the product description |
| mc:footprint | geojson | geographical delineation of the product / model trace |
| mc:BoundingBox | bbox | two points that represent the record extent |
| mc:srsId | text | reference System ID (EPSG), <br/> ex: 4326 / 3857 |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

system can be lowercase

The following guide will help you understand ***Step-by-step*** how to work with the Map Colonies Height Extraction service, from the [Catalog](/docs/MapColonies/DEM/Services/catalog/dem-catalog) to the WCS protocol.

:::warning
This guide is **not** for viewing purposes.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the purpose of this warning?

- `numberOfRecordsReturned` indicating the amount of records returned for this request (may be less than `maxRecords` given in the request)
- `nextRecord` the value that should be passed for `startPosition` in the following request

In case we are fetching the full profile without any filters our next request should look like this:

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is almost identical to the request above. consider removing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants