Commit e5f8f09
authored
docs(processing): explain the two toolboxes and fix stale menu paths (#1929)
* docs(processing): explain the two toolboxes and fix stale menu paths
The Processing menu carries two independent tool collections whose category
names collide: the Whitebox toolbox catalog (Conversion, Hydrology, LiDAR,
Network, Projection, Raster, Remote Sensing, Terrain, Vector as direct children
of the menu) and GeoLibre's own dialogs under the GeoLibre Toolbox submenu.
Nothing in the docs said so, so 'Processing / Vector / GeoLibre (WASM)' and
'Processing / GeoLibre Toolbox / Vector' read as duplicates.
- New 'Two toolboxes in one menu' section on the Processing Tools page: the
menu tree as rendered, a side-by-side table of what each toolbox is and
where it runs, and the specific Vector-vs-Vector pair spelled out. Also
notes that the dialog's Source filter ('GeoLibre tools' / 'Whitebox tools')
splits the Whitebox catalog only, which is a third thing again.
- Reorganized the page under 'GeoLibre Toolbox' and 'Whitebox Toolbox'
headings and documented the families that had no entry at all: Network,
Spatial Statistics, DGGS, Geocode Addresses, Batch & Models, Object
Detection, Segment Everything, plus the Raster Analysis group and the
Vector Movement and Data quality groups.
- Corrected the Conversion section: every conversion tool now has a
client-side engine, so the per-tool 'Sidecar' engine column was wrong;
Raster to PMTiles was missing entirely.
- Updated the menu paths that moved under GeoLibre Toolbox across the docs,
README, and the sidecar README, and repointed the README's
#geoprocessing-toolbox anchor at #whitebox-toolbox.
- Added a Troubleshooting entry for users who land there after seeing the
same category name twice.
* docs(processing): scope the conversion allowlist claim, note colliding tool names
Two points from review.
The conversion sidecar's path allowlist is opt-in, not unconditional:
GEOLIBRE_CONVERSION_ROOTS is unset on desktop (app/conversion.py: 'Unset
means no restriction (the default for the desktop app)') and set by the
Docker image, whose sidecar is reachable same-origin through the nginx
proxy. The page inherited a flat 'hardened with a path allowlist' claim
that read as always-on; it now says which deployment sets it and why.
Reviewing a (mistaken) claim that the tutorials' Conversion paths pointed
at the wrong toolbox turned up a sharper case of this PR's own subject:
a few individual tool *names* collide across the toolboxes, not just
category names. The WASM catalog carries vector_to_pmtiles / write_pmtiles
displayed as 'Vector to PMTiles' and 'Raster to PMTiles', which are also
the names of two ConversionDialog tools, so both menus show a 'Vector to
PMTiles' leaf that opens a different dialog. Called out in 'The two paths
that look alike', with a note that these docs mean the GeoLibre Toolbox
one. The existing paths are correct and unchanged.
* docs(processing): correct the separator reference and the result-is-a-layer claim
Two minor review points, both right.
'Everything above the first separator' put AI Assistant inside the
Whitebox block; the Whitebox section is the block between the first and
second separators.
'the result is added to the map styled and ready' held for the analysis
dialogs but not for Conversion, which writes a file. Split the two.1 parent 6b854a2 commit e5f8f09
18 files changed
Lines changed: 229 additions & 67 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | | - | |
| 34 | + | |
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
| |||
106 | 106 | | |
107 | 107 | | |
108 | 108 | | |
109 | | - | |
| 109 | + | |
110 | 110 | | |
111 | 111 | | |
112 | 112 | | |
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
133 | | - | |
| 133 | + | |
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
31 | | - | |
| 31 | + | |
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
72 | 72 | | |
73 | 73 | | |
74 | 74 | | |
75 | | - | |
| 75 | + | |
76 | 76 | | |
77 | 77 | | |
78 | 78 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
132 | 132 | | |
133 | 133 | | |
134 | 134 | | |
135 | | - | |
| 135 | + | |
136 | 136 | | |
137 | 137 | | |
138 | 138 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
200 | 200 | | |
201 | 201 | | |
202 | 202 | | |
203 | | - | |
| 203 | + | |
204 | 204 | | |
205 | 205 | | |
206 | 206 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
79 | 79 | | |
80 | 80 | | |
81 | 81 | | |
82 | | - | |
| 82 | + | |
83 | 83 | | |
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
87 | 87 | | |
88 | | - | |
| 88 | + | |
89 | 89 | | |
90 | 90 | | |
91 | 91 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
48 | | - | |
| 48 | + | |
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
371 | 371 | | |
372 | 372 | | |
373 | 373 | | |
374 | | - | |
| 374 | + | |
375 | 375 | | |
376 | 376 | | |
377 | 377 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
36 | | - | |
| 36 | + | |
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
| 43 | + | |
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| |||
0 commit comments