From 6afe0f90937bbcec33561228ba1c440bedcbff24 Mon Sep 17 00:00:00 2001 From: Marco Hutter Date: Sat, 25 May 2024 15:11:32 +0200 Subject: [PATCH 1/3] Added note about CMPT-to-GLB in upgrade --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 51a41fb..9e1b686 100644 --- a/README.md +++ b/README.md @@ -101,11 +101,11 @@ When `--targetVersion 1.1` is given, then this will upgrade legacy tilesets to c - The `refine` value will be converted to be in all-uppercase. - glTF 1.0 models in B3DM or I3DM will be upgraded to glTF 2.0. - The `3DTILES_content_gltf` extension declaration will be removed. -- PNTS, B3DM, and I3DM content will be converted to glTF. +- PNTS, B3DM, I3DM, and CMPT content will be converted to glTF. > Implementation note: > -> The conversion of the legacy tile formats to glTF should be considered as a _preview feature_. There are corner cases where the conversion is not possible generically - for example, when I3DM tile content contains glTF data that contains _animations_. The conditions under which the conversion is possible may be specified more explicitly in the future. +> The conversion of the legacy tile formats to glTF should be considered as a _preview feature_. There are corner cases where the conversion is not possible generically - for example, when I3DM tile content contains glTF data that contains _animations_, or when a CMPT (indirectly) contains multiple glTF assets that already use the `EXT_structural_metadata` extension. The conditions under which the conversion is possible may be specified more explicitly in the future. From 4addb918e52c56a4af0b697da78a3e5bb0e96c9f Mon Sep 17 00:00:00 2001 From: Marco Hutter Date: Sat, 25 May 2024 15:12:32 +0200 Subject: [PATCH 2/3] Added notes about 'merge' and 'combine' --- README.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/README.md b/README.md index 9e1b686..d8b828a 100644 --- a/README.md +++ b/README.md @@ -52,6 +52,11 @@ npx 3d-tiles-tools ungzip -i ./specs/data/TilesetOfTilesets-gzipped/ -o ./output #### combine Combines all external tilesets into a single tileset. + +> **Note** about the difference between `merge` and `combine`: The `combine` command takes a tileset that already _refers_ to _external_ tilesets. And it creates a new tileset where the former _external_ tilesets are "inlined". So the result will be a single tileset, without external references. + +Example: + ``` npx 3d-tiles-tools combine -i ./specs/data/combineTilesets/input -o ./specs/data/combineTilesets/output ``` @@ -59,6 +64,10 @@ npx 3d-tiles-tools combine -i ./specs/data/combineTilesets/input -o ./specs/data #### merge Merge multiple tilesets into a single one that refers to the input tilesets as external tilesets. + +> **Note** about the difference between `merge` and `combine`: The `merge` command creates a _new_ tileset that _refers_ to the given ones as _external_ tilsets. This means that the resulting tileset is not complete _without_ the ones that are used as the inputs for the `merge` command. + +Example: ``` npx 3d-tiles-tools merge -i ./specs/data/mergeTilesets/TilesetA -i ./specs/data/mergeTilesets/sub/TilesetA -o ./specs/data/mergeTilesets/output ``` From ba63bd2c2289a778ac93e18d55799a10b3bd0064 Mon Sep 17 00:00:00 2001 From: Marco Hutter Date: Sat, 25 May 2024 15:13:15 +0200 Subject: [PATCH 3/3] Added note about case sensitive names --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index d8b828a..51e385e 100644 --- a/README.md +++ b/README.md @@ -14,6 +14,8 @@ If you want to work directly with a clone of the Git repository, see [Developer ## Command Line Usage +> **Note:** All input- and output paths for the 3D Tiles Tools are _case sensitive_. Even though some operating systems (like Windows) usually do not differentiate between uppercase and lowercase, the file- and directory names that are given at the command line must exactly match the actual names. + #### Common command line options for each function: |Flag|Description|Required|