Repository navigation
BrasilData 0.1.24: MapBiomas land cover for any year, vegetation disturbance alerts, Sentinel-1 radar, official municipal clips and complete drainage - #49
Conversation
… large areas Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. Walkthrough
Merge Risk:
|
Code reviewBugs
Security
Performance
Quality
CLAUDE.md
Plugin packages
Assessment of Change sizeCompared: the whole PR against main (merge base cc60d40 to PR head d5293fe).
|
🧩 Plugin previewRemoved: the pull request is closed. |
… fetched again Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
| "url": "https://github.com/elpidioassinaturas/BrasilData/releases/download/v0.1.16/brasildata-0.1.16.rgplugin", | ||
| "sha256": "b25d9e244594141f370f2b248c4a8552d8e544b2bf457d693601a2360922afaa" | ||
| "url": "https://github.com/elpidioassinaturas/BrasilData/releases/download/v0.1.19/brasildata-0.1.19.rgplugin", | ||
| "sha256": "9cb8ba93385e06afdf33c12d9bd83c7d1904e75f0c628dc24fb818b1be08fb98" |
There was a problem hiding this comment.
The package at this URL/SHA-256 was inspected: the hash matched and the manifest (id, name, version 0.1.19) is consistent with the entry. A maintainer should still confirm the network permission is accepted by the registry and that the author's source tree matches the released wasm, because the module (6.7 MB) can't be read here. Confidence: medium.
Code reviewBugs
Security
Performance
Quality
CLAUDE.md
Plugin packages
Assessment: the package was downloaded and its hash verified. The manifest is consistent with the registry entry. The commands and URLs match the stated purpose of fetching public Brazilian data and Sentinel-1 radar. The module is compiled, so its code can't be reviewed here. The source is reachable at the linked GitHub repository. The permissions are unchanged from 0.1.16. No blocking issues found. The only open question for a maintainer is the unreadable-module caveat above. Change sizeThe whole PR against main (merge base cc60d40 to PR head d398ffe).
|
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Code reviewBugs
Security
Quality
CLAUDE.md
Plugin packages
brasildata: the package was inspected and the hash verified. The interface is valid, and the URLs point to public Brazilian government and open-data services (IBGE, ANA, ANEEL, ANP, MMA/ICMBio, INCRA, FUNAI, IBAMA, ANM, DNIT, BCB, MapBiomas), plus Copernicus DEM on S3 and NASA ASF/OPERA. Nothing suspicious turned up. The WebAssembly code can't be read here, so behaviour rests on the upstream source. A maintainer may want to spot-check the upstream repository at the v0.1.20 tag. Change sizeCompared: the whole PR against main (merge base cc60d40 to PR head eddc8db).
|
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
| "url": "https://github.com/elpidioassinaturas/BrasilData/releases/download/v0.1.16/brasildata-0.1.16.rgplugin", | ||
| "sha256": "b25d9e244594141f370f2b248c4a8552d8e544b2bf457d693601a2360922afaa" | ||
| "url": "https://github.com/elpidioassinaturas/BrasilData/releases/download/v0.1.21/brasildata-0.1.21.rgplugin", | ||
| "sha256": "8c0636596d2ce0a19950beada10eae95740f15fbf1158bd1175c33c27e154bda" |
There was a problem hiding this comment.
The version jumps from 0.1.16 to 0.1.21, so five releases ship in one review, and the new package is a compiled 6.7 MB module whose code can't be read here. Before merging, a maintainer should confirm the tag v0.1.21 in elpidioassinaturas/BrasilData builds to this SHA-256. The new strings name destructive commands (layer.remove, table.remove, edit.deleteField, style.remove, map.removeTiles) that don't seem needed for the stated purpose, and gp.run is a new addition too. The PR says style.remove and temp-layer cleanup are used, but nothing here proves layer.remove/edit.deleteField only touch layers or fields the plugin created. Confidence: low-medium.
Code reviewBugs
Security
Performance
Quality
CLAUDE.md
Plugin packages
Assessment of
Change sizeCompared: the whole PR against main (merge base cc60d40 to PR head a166e17).
|
…n IBGE's full-detail boundary Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Code reviewBugs
Security
Performance
Quality
CLAUDE.md
Plugin packages
Assessment: The manifest id, name and version (0.1.22) match the registry entry, and the module imports are only the three allowed ones. The commands named in strings match the description: data fetch, raster processing, styling, and replacing the plugin's own layers. The new Change sizeCompared: the whole PR against main (merge base cc60d40 to PR head d731989).
|
…nd a fix for two-zone areas Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Code reviewBugs
Security
Performance
Quality
CLAUDE.md
Plugin packages
Assessment: the manifest has id Change sizeCompared: the whole PR against main (merge base cc60d40 to PR head afbdcc1).
|
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Code reviewBugs
Security
Performance
Quality
CLAUDE.md
Plugin packages
Assessment of
Change sizeCompared: the whole PR against main (merge base cc60d40 to PR head 053b20e).
|
Plugin: BrasilData (
brasildata), elpidioassinaturas/BrasilDataWhat it does: this updates
registry/brasildata.jsonfrom 0.1.16 to 0.1.24. Nothing else changes. Versions 0.1.17 to 0.1.23 were released while this PR was open, so their changes are included here.Unchanged: the permissions (
edit,files,network), the minimum RustGIS (0.5.0) and the categories.0.1.24: MapBiomas land cover for any year
{"ano": 2003}still works in scripts.0.1.23: clips follow IBGE's official municipal boundary, and a fix for two-zone areas
layer.addUrl. Parsing it inside the plugin ran out of fuel for large municipalities (Manaus, 457 KB; Rio de Janeiro, 754 KB), so the plugin never reads it: it tags the layer with amalhafield and checks the fields throughinfo.project. The API's full-quality mesh stays as the fallback. Measured in Viçosa against the official line: the embedded intermediate mesh was up to 815 m off, the API's full-quality one up to 144 m.gp.run clip), so along the border they follow the line instead of 30 m cell steps.geolibre:raster_to_vector_polygonscannot write EPSG:10857 (IBGE's Albers, which the grid uses when an area spans two UTM zones), so 0.1.22's status command failed there. The patches now come from a copy in degrees, and if they still fail, the raster stays visible. Above about 5 million grid cells the plugin keeps only the raster (vectorising a whole mesoregion took minutes).0.1.22: alert patches with outlines, and municipal clips on IBGE's full-detail boundary
geolibre:raster_to_vector_polygons), each filled with its class colour and outlined in a darker shade of that colour, so light patches stay readable on a light basemap. A text field gives each class its name, so the legend reads "6 · Confirmada, alta confiança" instead of a bare code. The gridded raster stays in the project, hidden, for analysis.0.1.21: vegetation disturbance alerts (OPERA DIST-S1)
Four new commands under Desmatamento: the alert status, the date of first detection, the confidence and the disturbance metric, from JPL's OPERA DIST-S1 (Sentinel-1), found through the ASF DAAC search API.
{"paleta": "oficial"}. The palette goes in throughstyle.importQgis(paletted); no-disturbance (0) is hidden withraster.mask, because 0.5.0 draws an alpha-0 palette entry as white.0.1.20: paged themes stay within the apps' download queue
layer.addUrldownload, and the app refuses the 17th waiting one ("16 downloads are already waiting"). The ANA drainage of a mesoregion needs up to 30 pages of 1,000 segments. The command failed, and the pages that had already started were never merged. I found this by testing 0.1.19 on web.rustgis.app; the CLI downloads synchronously, so it never hit the limit.0.1.19
Sentinel-1 radar. Three new commands under Uso e cobertura: Radar VV, Radar VH and the VH/VV ratio.
earthdata.login). The plugin only checksearthdata.statusand never sees the token.Automatic grid: when the area spans more than one UTM zone, the grid now uses IBGE's Albers (EPSG:10857). Before, it used the Polyconic (EPSG:5880), which
raster.projectin RustGIS 0.5.0 does not support.Layer style is kept. When a theme is fetched again for another area or view, the new layer keeps the style the user gave the old one: legend, colours, labels and transparency.
style.new). At the end, it applies that style to the layer with the same name (style.apply) and removes it (style.remove).0.1.18: drainage by zoom
The ANA drainage layer (BHO 2017) shows the main rivers when zoomed out and adds smaller ones as the user zooms in.
layer.displayFilters, keyed on each segment's upstream drainage area (nuareamont, km²). Strahler order is not used: it grows with mapping density and ranks the Amazon below rivers in the Southeast.0.1.17: complete BC250 drainage
startIndex/count) and merges the pages.service.addreports a cut, the plugin shows that notice in Portuguese.How I tested it
0.1.24, built from the same source:
0.1.23, built from the same source:
scripts/smoke.py0 failures.0.1.22, built from the same source:
"malha": "maxima"); the status gave 3,183 patches with names; the soils theme was clipped to the new boundary; the radar output is unchanged;scripts/smoke.py0 failures.0.1.21, built from the same source:
scripts/smoke.pyin rustgis-cli 0.5.0 (0 failures, also locally).Earlier versions:
python scripts/build.py --checkwith this entry passes (Python 3.12): 1 new package, 12 already published. The release asset's SHA-256 matches the entry.scripts/smoke.pyin rustgis-cli 0.5.0, and the same smoke test passes locally (0 failures).Built from the same source, in rustgis-cli 0.5.0:
area_km2, 4 quantiles, Ember) kept its field, classes and ramp.Tested in web.rustgis.app (0.1.19, from this PR's preview registry): BC250 drainage complete over two mesoregions, including one that needed 2 pages; the layer style kept across two area changes; no temporary style left in the project.
Not tested: the desktop window.
🤖 Generated with Claude Code
Summary by CodeRabbit