Skip to content

docs: add a platform comparison page - #1730

Merged
giswqs merged 5 commits into
mainfrom
docs/platform-comparison
Aug 6, 2026
Merged

docs: add a platform comparison page#1730
giswqs merged 5 commits into
mainfrom
docs/platform-comparison

Conversation

@giswqs

@giswqs giswqs commented Aug 6, 2026

Copy link
Copy Markdown
Member

Adds docs/comparison.md, a page comparing GeoLibre to the platforms people most often ask about: QGIS, ArcGIS Pro, ArcGIS Online, Felt, and kepler.gl.

What's in it

Five comparison tables:

  1. At a glance — license, cost, browser/desktop/mobile/Jupyter availability, offline support, where data lives, project file format
  2. Data and formats — format breadth, cloud-native vector and raster, STAC, OGC and Esri services, databases, 3D and point clouds, planetary basemaps
  3. Analysis and processing — tool counts, where analysis runs, vector and raster tools, spatial SQL, model chaining, spatial statistics, network analysis, AI/ML
  4. Cartography, styling, and layout — renderers, data-defined styling, labeling, print layout, style interchange, project import, story maps, dashboards
  5. Automation, extensibility, and sharing — scripting, notebooks, plugins, AI assistant, real-time collaboration, embedding, self-hosting, standalone export

Then "Where each one is the right choice" (a paragraph per platform, including when not to pick GeoLibre) and "Where GeoLibre is not the strongest option", which states the real limits: the browser memory ceiling on large local datasets, GDAL/OGR's wider format coverage, Esri's specialized toolboxes, enterprise geodatabase management, and relative maturity.

That last section is deliberate. A comparison written by the project gets discounted entirely if it reads as all-wins, so the honest limits are what make the tables above them credible.

Framing

The intro carries a dated caveat (August 2026), scopes the non-GeoLibre columns to mainstream out-of-the-box behavior rather than anything reachable with every add-on, credits separate products as separate (QField for QGIS in the field, ArcGIS Field Maps / Dashboards / StoryMaps for Esri) instead of folding them into the base product, and links to the issue tracker for corrections.

Wiring

  • Added to the mkdocs.yml nav as a top-level Comparison entry after Features
  • Cross-linked from docs/index.md (Project status) and docs/features.md (intro)

Verification

  • zensical build — no issues; emits site/comparison/index.html with all 5 tables
  • Every table row rendered with exactly 7 cells (verified by rendering the markdown, not by eye)
  • All internal cross-link anchors resolve
  • pre-commit run --files <changed> — passing

Docs only; no application code touched.

Summary by CodeRabbit

  • Documentation
    • Added a comprehensive comparison of GeoLibre with QGIS, ArcGIS Pro, ArcGIS Online, Felt, and kepler.gl.
    • Documented capabilities, limitations, licensing, collaboration, embedding, self-hosting, and export options.
    • Added the Comparison page to the documentation navigation and linked to it from relevant project pages.

Add docs/comparison.md, comparing GeoLibre against QGIS, ArcGIS Pro,
ArcGIS Online, Felt, and kepler.gl across five tables: platform basics,
data and formats, analysis and processing, cartography and layout, and
automation/extensibility/sharing.

Follow the tables with per-platform guidance on when each one is the
right choice, and a section stating plainly where GeoLibre is not the
strongest option (large local datasets, GDAL format breadth, specialized
Esri toolboxes, enterprise data management, maturity) so the rest of the
page is worth trusting.

Scope the non-GeoLibre columns to mainstream out-of-the-box behavior as
of August 2026, credit separate products as separate (QField, ArcGIS
Field Maps, ArcGIS Dashboards/StoryMaps), and invite corrections via
GitHub issues.

Wire the page into the mkdocs nav after Features, and cross-link it from
index.md and features.md.
Copilot AI lite review requested due to automatic review settings August 6, 2026 00:02

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 638531b3-86af-4323-8529-8ad77010e17c

📥 Commits

Reviewing files that changed from the base of the PR and between cf17289 and 180f3e9.

📒 Files selected for processing (1)
  • docs/comparison.md

📝 Walkthrough

Walkthrough

Added a GeoLibre comparison guide. The guide compares mapping platforms, documents selection guidance and limitations, and links to related documentation. The guide is available from the main navigation and key documentation pages.

Changes

Documentation comparison

Layer / File(s) Summary
Comparison guide
docs/comparison.md
Added product scope, capability comparisons, selection guidance, interoperability details, limitations, and related documentation links.
Documentation wiring
mkdocs.yml, docs/features.md, docs/index.md
Added the Comparison page to navigation and linked it from the features and project status pages.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • opengeos/GeoLibre#1451: Both PRs update docs/features.md and mkdocs.yml; this PR adds comparison links and navigation.

Poem

A rabbit found a comparison chart,
With maps and tools set part by part.
New links guide each careful hop,
From project page to knowledge stop.
GeoLibre’s guide is ready to start.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main documentation change: adding a platform comparison page.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/platform-comparison

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Code review

I reviewed this docs-only PR (docs/comparison.md + small cross-links in docs/features.md, docs/index.md, mkdocs.yml). No inline comments were posted — I didn't find issues rising to that bar. Summary of what was checked:

Bugs: None found. All internal cross-link targets and anchors resolve (features.md, index.md#recently-added, roadmap.md, getting-started.md, user-guide/projects.md#importing-a-qgis-project, #importing-an-arcgis-pro-project). Spot-checked table rows in all five comparison tables — each has 7 pipe-delimited cells, no truncated/misaligned rows.

Security: None applicable — no code, no user input handling, no secrets.

Performance: N/A — static markdown content only.

Quality: Content is well-structured and internally consistent (e.g., the "Format breadth" claim in the top table is echoed, not contradicted, by the "Where GeoLibre is not the strongest option" section). Self-referential claims about GeoLibre's own features (Android app via Tauri v2, .geolibre.json project format, MIT license, embed maponly/layout=viewer URL params + postMessage API, real-time collaboration MVP with anchored review comments) were cross-checked against docs/features.md, docs/collaboration.md, docs/user-guide/embedding.md, and LICENSE — all accurate. Comparative claims about QGIS/ArcGIS/Felt/kepler.gl are inherently subjective marketing/editorial content, but the doc itself flags this bias upfront ("written by an interested party") and includes an honest limitations section, which is good practice. Confidence: high.

CLAUDE.md: !!! note admonition syntax is valid — admonition extension is enabled in mkdocs.yml. Nav entry "Comparison" correctly placed after "Features" as described in the PR body. No i18n concerns — the repo's react-i18next convention applies to in-app UI strings, not this docs site's markdown. No other CLAUDE.md guidance applies to docs/**.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

🔍 Cloudflare PR preview

Item Value
Site https://ded4a79d.geolibre-preview.pages.dev
Demo app https://ded4a79d.geolibre-preview.pages.dev/demo/
Commit 2f56409

@coderabbitai coderabbitai Bot left a comment

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.

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/comparison.md`:
- Line 48: Update the Esri services comparison row so it no longer lists “.gdb”
folders as a network service; either move that format to the format comparison
or rename the row to clearly cover Esri services and file geodatabases.
- Line 47: Update the OGC services row in the comparison table to distinguish
GetCapabilities discovery for WMS/WFS/WMTS from landing-page and /conformance
discovery for OGC API Features/Tiles, preserving the existing service coverage
and comparison values.
- Line 57: Update the kepler.gl entries in the comparison table, including the
analysis and AI assistant cells at the referenced sections, to reflect DuckDB
SQL Data Explorer and visualization/data-exploration capabilities rather than
“none.” Identify the AI Assistant as model-dependent, and keep the descriptions
concise and consistent across all affected rows.
- Around line 31-36: Update the Felt column entries in the comparison table and
the referenced sections to name the separate Felt Field App for mobile, state
that offline areas are available through that app on eligible paid plans, and
mention Felt’s self-hosted VPC deployment option wherever data hosting or
deployment scope is described. Replace the current “No”, “—”, and
hosted-cloud-only wording without changing unrelated product comparisons.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: ad943a5c-4e53-4858-a995-e5dc6d35fa31

📥 Commits

Reviewing files that changed from the base of the PR and between 67eeffc and 673a2e5.

📒 Files selected for processing (4)
  • docs/comparison.md
  • docs/features.md
  • docs/index.md
  • mkdocs.yml

Comment thread docs/comparison.md Outdated
Comment thread docs/comparison.md Outdated
Comment thread docs/comparison.md Outdated
Comment thread docs/comparison.md Outdated
- Felt: name the separate Felt Field App for iOS/Android in the Mobile
  row rather than calling it responsive web, record its offline areas
  with sync-on-reconnect, and note the single-tenant AWS deployment in
  the data-residency and self-hosting rows. Verified against
  felt.com/platform/field-app and help.felt.com/administration/self-hosted-felt.
- Felt: the Dashboards row said "—", but Felt ships dashboard
  components (statistics, bar, histogram, time series). Corrected.
- kepler.gl: the analysis, spatial SQL, and AI assistant rows understated
  it. It ships a DuckDB-backed SQL Data Explorer over loaded data and
  remote URLs, spatial joins through its assistant, and an AI assistant
  supporting OpenAI, Gemini, DeepSeek, and local Ollama with a
  user-supplied key. Verified against the kepler.gl user guides.
- OGC services row: GetCapabilities is the WMS/WFS/WMTS discovery
  mechanism; OGC API Features/Tiles discovers from a landing page,
  collection, or items URL. Split the two rather than attributing
  GetCapabilities to both.
- Esri row: `.gdb` is a local File Geodatabase format, not a network
  service. Renamed the row to "Esri services and geodatabases" and
  labeled the entry accordingly.
- Updated the "Choose Felt" and "Choose kepler.gl" paragraphs to match
  the corrected tables — Felt's self-hosting option and field
  collection, and kepler.gl's SQL/AI analysis path.
Comment thread docs/comparison.md Outdated
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Code review

This is a docs-only PR adding docs/comparison.md plus small cross-links from docs/features.md/docs/index.md and an mkdocs.yml nav entry. No application code is touched, so most review categories (bugs, security, performance) don't really apply here.

Bugs: None found. Markdown tables are well-formed (every row has 8 pipe delimiters / 7 columns, consistent with the PR description's claim). All internal links/anchors checked resolve: features.md, index.md#recently-added, roadmap.md, getting-started.md, and user-guide/projects.md#importing-a-qgis-project / #importing-an-arcgis-pro-project all exist in the current tree.

Security: None. No user input, no code execution, no secrets.

Performance: N/A — static documentation content.

Quality:

  • The "Real-time collaboration" table cell describes the feature as "Yes (MVP, self-hosted server)", but per docs/collaboration.md the default/production path is a hosted Cloudflare Worker relay (collab.geolibre.app); self-hosting (workers/collab-node) is an optional alternative, not the default. This slightly overstates the self-hosting requirement. (Medium confidence — posted inline.)
  • Cross-checked numerous other specific factual claims about GeoLibre itself (Gaussian splats/3D Tiles/I3S, Apache Sedona/PGlite, Emerging Hot Spot Analysis, maponly/layout=viewer/@geolibre/embed, SLD/QML/Mapbox GL style interchange, .qgs/.aprx project import, standalone HTML export, Docker self-hosting) against docs/features.md and they all check out accurately.

CLAUDE.md: No violations found. This changes only docs/*.md and mkdocs.yml, which are outside the scope of the i18n (react-i18next) requirement — that applies to in-app user-facing strings in apps/geolibre-desktop/src/i18n/locales/*.json, not the mkdocs/zensical documentation site. The nav entry placement (Comparison after Features) matches the PR description and existing nav conventions.

One inline comment posted on the minor "self-hosted server" phrasing nit; everything else in this PR looks solid and well fact-checked against the existing docs.

@coderabbitai coderabbitai Bot left a comment

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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/comparison.md`:
- Around line 45-46: Update the ArcGIS Pro entries in the comparison table:
replace “Zarr via extensions” with wording that identifies Zarr as a natively
supported multidimensional raster format, and replace “Via add-ins” with wording
describing the built-in STAC connection and Explore STAC pane workflow. Leave
the other product entries unchanged.
- Line 88: Update the ArcGIS Online cell in the comparison table from “—” to
concise wording that indicates organization-enabled AI assistants are available,
including Arcade assistant support, subject to administrator enablement and beta
restrictions being disabled.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: cdd682c8-268c-4673-b3fd-3375b4eb4967

📥 Commits

Reviewing files that changed from the base of the PR and between 673a2e5 and 03893bb.

📒 Files selected for processing (1)
  • docs/comparison.md

Comment thread docs/comparison.md Outdated
Comment thread docs/comparison.md Outdated
- Real-time collaboration row said "self-hosted server", implying
  self-hosting is required. Per docs/collaboration.md the default relay
  is the hosted Cloudflare Durable Object at collab.geolibre.app
  (workers/collab), with workers/collab-node as the self-hostable
  option. Now reads "hosted relay, optionally self-hosted".
- ArcGIS Pro reads Zarr as a native multidimensional raster type, not
  "via extensions" — corrected against the Esri multidimensional raster
  types documentation.
- ArcGIS Pro STAC support is built in (STAC connections and the Explore
  STAC pane, since Pro 3.2), not "via add-ins".
- Same row, not flagged but the same error: QGIS reads Zarr through
  GDAL's built-in Zarr driver (core since GDAL 3.4), so
  "plugin-assisted" was wrong.
- ArcGIS Online AI assistant cell said "—", but it ships the Arcade
  assistant (beta) across Map Viewer, Field Maps Designer, Experience
  Builder, and Scene Viewer, plus the wider AI assistants family.
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

🔍 GitHub Pages PR preview

Item Value
Site Deploy failed. See the job log.
Demo app Unavailable
Commit 2f56409

@coderabbitai coderabbitai Bot left a comment

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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/comparison.md`:
- Line 46: Update the QGIS entry in the STAC comparison row to describe support
as built in, replacing “Via plugins” with concise wording that mentions STAC
connections through the Browser panel and Data Source Manager.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 3dc8f726-aaf5-419d-a3dd-e363255b6451

📥 Commits

Reviewing files that changed from the base of the PR and between 03893bb and 2f56409.

📒 Files selected for processing (1)
  • docs/comparison.md

Comment thread docs/comparison.md Outdated
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Code review

This is a docs-only PR (docs/comparison.md new page, plus small cross-link edits to docs/features.md, docs/index.md, and a mkdocs.yml nav entry). No application code is touched, so most CLAUDE.md conventions (i18n, package boundaries, WASM/version mirrors, etc.) don't apply here.

Bugs: None found. Spot-checked every internal cross-link and anchor referenced from the new page — features.md, roadmap.md, getting-started.md, index.md#recently-added, user-guide/projects.md#importing-a-qgis-project, and #importing-an-arcgis-pro-project — all resolve to real headings/files in the repo. Markdown tables are well-formed (7 cells per row, no stray unescaped | in cell content).

Security: None. No secrets, no external script inclusion, only a standard GitHub issues link.

Performance: N/A (docs only).

Quality: The GeoLibre-side factual claims (Android app, real-time collaboration architecture, AI assistant BYO-key, Dashboard panel, Whitebox "1,000+" tools, PGlite/Sedona SQL engines, planetary basemap list, .aprx/.mapx project import, PWA/offline support) all cross-check accurately against docs/features.md, docs/collaboration.md, and docs/user-guide/projects.md. The mkdocs.yml nav addition and the two intro-paragraph edits in features.md/index.md read naturally and match the surrounding prose style. Claims about competitor products (QGIS, ArcGIS, Felt, kepler.gl) aren't independently verifiable from this repository/sandbox, but the page's own caveat note (dated, scoped to mainstream out-of-the-box behavior, inviting corrections via issue) appropriately hedges that risk.

CLAUDE.md: No violations — this change doesn't touch any of the mirrored-constant, catalog-generation, or CSP-allowlist areas CLAUDE.md calls out, and it doesn't modify translatable UI strings.

No inline comments posted.

- QGIS STAC support is built in, not "via plugins": QGIS creates STAC
  connections from the Browser panel and the Data Source Manager's STAC
  tab, covering both static catalogs and STAC APIs. Verified against the
  QGIS user manual's OGC client support page. Left the version
  unqualified since the page compares current releases.
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Code review

This is a docs-only PR (docs/comparison.md + three small cross-link/nav additions) with no application code touched, so most standard review dimensions (bugs, security, performance) don't really apply. I verified:

  • All internal links/anchors resolve: comparison.md's links to features.md, roadmap.md, getting-started.md, and user-guide/projects.md#importing-a-qgis-project / #importing-an-arcgis-pro-project all match real headings in those files, as does index.md#recently-added.
  • The new mkdocs.yml nav entry (Comparison: comparison.md) is placed consistently with the PR description (after Features, before Demos) and matches existing nav conventions.
  • The five comparison tables render as well-formed 7-column Markdown tables with no stray unescaped | characters in cell content that would misalign columns.
  • Claims about GeoLibre's own capabilities (collaboration relay architecture, AI assistant "bring your own key" model, planetary basemaps, 1,000+ tools, project import, etc.) cross-check against docs/features.md, docs/collaboration.md, and docs/index.md and are accurate and consistent.
  • docs/index.md and docs/features.md edits are minimal, additive, and don't disturb surrounding content.

Bugs: None found.

Security: None found (no code, no secrets, no injected content).

Performance: N/A — static Markdown docs page.

Quality: No issues rising to the level of a finding. One very minor, low-confidence observation not worth an inline comment: the teaser sentences added to docs/features.md and docs/index.md refer to the comparison collectively as "QGIS, ArcGIS, Felt, and kepler.gl," while the comparison page itself treats ArcGIS Pro and ArcGIS Online as distinct columns — a harmless shorthand, not a defect.

CLAUDE.md: No violations. This change doesn't touch translatable UI strings (i18n applies to the app's apps/geolibre-desktop/src/i18n catalogs, not the mkdocs documentation site), and none of the other repo-specific mirror/lockfile/CSP rules apply to a docs-only change.

One caveat: I have no network access, so I could not independently fact-check the specific claims made about third-party products (QGIS, ArcGIS Pro/Online, Felt, kepler.gl feature details as of "August 2026"). The PR description states these were verified by the author; a reviewer with current knowledge of those products should spot-check the more specific/recent claims (e.g., kepler.gl's AI assistant provider list, Felt's Enterprise AWS single-tenant offering) before merging.

@coderabbitai coderabbitai Bot left a comment

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.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)
docs/comparison.md (2)

57-57: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Separate configured QGIS providers from out-of-the-box tools.

The scope states that the table compares mainstream out-of-the-box behavior, but this cell counts SAGA and GRASS with native and GDAL tools. QGIS documents that providers relying on third-party applications require additional configuration. Qualify these providers or exclude them from the out-of-the-box count. (docs.qgis.org)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/comparison.md` at line 57, Update the QGIS geoprocessing-tools cell in
the comparison table to distinguish SAGA and GRASS providers that require
additional configuration from tools available out of the box; either qualify
those providers explicitly or exclude them from the out-of-the-box count.

Source: MCP tools


7-11: 🔒 Security & Privacy | 🟠 Major | ⚡ Quick win

Sensitive Data Exposure (CWE-200): Exposure of Sensitive Information to an Unauthorized Actor

Reachability: External

Qualify the local-data pledge.

docs/comparison.md:7-11 and docs/comparison.md:96-100 state that GeoLibre keeps data on the user’s machine without mentioning optional network transfers. Use “local by default” and list the exceptions: AI assistant prompts plus layer metadata, and real-time collaboration project snapshots via the relay.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/comparison.md` around lines 7 - 11, Update the local-data claim in the
introductory comparison text around GeoLibre’s browser experience to say “local
by default” and explicitly identify the exceptions: AI assistant prompts and
layer metadata, plus real-time collaboration project snapshots sent through the
relay; apply the same qualification to the corresponding statement near the
later comparison section.

Source: MCP tools

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/comparison.md`:
- Line 48: Update the “Esri services and geodatabases” row to distinguish ArcGIS
Online hosted layers from direct local File Geodatabase support; replace the
ArcGIS Online geodatabase capability labeled “Native” with hosted-layer wording,
or split the hosted services and local .gdb entries into separate rows while
preserving accurate native support only for local geodatabases.

---

Outside diff comments:
In `@docs/comparison.md`:
- Line 57: Update the QGIS geoprocessing-tools cell in the comparison table to
distinguish SAGA and GRASS providers that require additional configuration from
tools available out of the box; either qualify those providers explicitly or
exclude them from the out-of-the-box count.
- Around line 7-11: Update the local-data claim in the introductory comparison
text around GeoLibre’s browser experience to say “local by default” and
explicitly identify the exceptions: AI assistant prompts and layer metadata,
plus real-time collaboration project snapshots sent through the relay; apply the
same qualification to the corresponding statement near the later comparison
section.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 35c5f8a0-3d40-4339-8e23-e98963dfea59

📥 Commits

Reviewing files that changed from the base of the PR and between 2f56409 and cf17289.

📒 Files selected for processing (1)
  • docs/comparison.md

Comment thread docs/comparison.md Outdated
- The "Esri services and geodatabases" row left ArcGIS Online as
  "Native", which after the row was renamed in 03893bb read as direct
  file geodatabase support. ArcGIS Online serves hosted layers, so the
  cell now reads "Hosted feature, tile, imagery, and scene layers",
  matching the documented hosted web layer types. ArcGIS Pro keeps
  "Native", where it is accurate for both services and geodatabases.
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Everything checks out cleanly. This is a well-crafted, self-consistent docs PR with no code changes. I did not find issues rising to the level of an inline comment — the tables are well-formed, internal links/anchors resolve, and the factual claims about GeoLibre's own capabilities match features.md/collaboration.md. No create_inline_comment calls are warranted.

Code review

No issues found. This is a docs-only PR (docs/comparison.md + small cross-links in docs/features.md, docs/index.md, mkdocs.yml) with no application code changes.

Checks performed:

  • Bugs: N/A — no code, only Markdown tables/prose.
  • Security: N/A — no code, no secrets, no untrusted input handling.
  • Performance: N/A — static docs content, no build-time cost of note.
  • Quality: Table structure is consistent (every row in all 5 tables has the same 7-cell shape matching the header). Cross-links (docs/features.md, docs/index.md, mkdocs.yml nav entry) are correctly wired, and the anchors used (user-guide/projects.md#importing-a-qgis-project, #importing-an-arcgis-pro-project, index.md#recently-added) all resolve to real headings in the target files. High confidence.
  • Factual accuracy (spot-checked against repo docs): Verified several specific claims in comparison.md against docs/features.md and docs/collaboration.md — AI Segmentation (SamGeo/SAM 3) + in-browser ONNX/YOLO, 3D/point-cloud formats (LiDAR, 3D Tiles, I3S, Gaussian splats, glTF/GLB), style interchange (SLD/QML/Mapbox GL JSON), QGIS/ArcGIS Pro project import, and real-time collaboration (MVP, self-hostable relay, anchored review comments) — all match the current documented feature set. Medium-high confidence given the sampling wasn't exhaustive over every table cell, but no discrepancies were found.
  • CLAUDE.md: No conventions in CLAUDE.md apply to this docs-only change (it targets app/package source, i18n strings, and mirrored constants — none of which are touched here).

@giswqs
giswqs merged commit 4a95543 into main Aug 6, 2026
17 checks passed
@giswqs
giswqs deleted the docs/platform-comparison branch August 6, 2026 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants