Develop#151
Merged
Merged
Conversation
…orm roadmap note The README described features that live in the separate DisSModel Platform project (REST reproduce endpoint, Docker/FastAPI/Redis deployment), which could mislead users and reviewers of the published package. - Rewrite the 'Cloud-ready' feature bullet as 'Platform-ready contract', linking to the new roadmap section - Replace the curl /reproduce example with a description of what the locally generated record.json actually provides - Add a 'Roadmap: DisSModel Platform' section stating explicitly that the platform is a separate project under development and not part of this package Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Replace the 'Anisotropy / GPM Support' row in the comparison table: DisSModel provides neighborhood definition via libpysal spatial weights (Queen, Rook, KNN, custom adjacency), not a GPM implementation - Remove the runtime 'TOML-based model registry' claim from the executor description (platform-side feature); keep what the package provides: executors as standard Python packages with parameters resolved from a version-controllable TOML specification - Clarify that Streamlit dashboards live in the satellite packages; the core ships Streamlit-compatible input widgets - Mark the DisSModel Platform explicitly as a separate project in the Research Impact section Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…chmark results BR-MANGUE now lives in the brmangue-dissmodel package, which holds both the raster and vector substrates plus two validation levels (raster vs TerraME and vector vs raster). coastal-dynamics is being retired. - paper: cite brmangue-dissmodel (@BRMangue) instead of @CoastalDynamics; describe the two-level validation — raster vs TerraME on the Maranhão Island dataset (50,496 cells, 20 steps: uso 99.90%, solo 99.85%, alt 90.67% within 1 mm) and vector vs raster on the 60x60 synthetic grid (10 steps, 100% match on all bands, raster 5.0 ms/step vs vector 124.4 ms/step) - paper: refresh Game of Life benchmark table with current measurements (1M cells at ~27 ms/step) - paper.bib: replace the CoastalDynamics entry with BRMangue - README and docs/examples.md: point the ecosystem tables to brmangue-dissmodel (install verified via pip git+ and smoke imports) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
JOSS requires papers between 750 and 1750 words; after the BR-MANGUE two-level validation was added the text reached 1795 words. Three surgical cuts bring it to 1745 without losing citations or required content (non-specialist explanation and research applications kept): - drop the redundant 'hybrid data types' sentence in Statement of Need (dual-substrate architecture is already described in the Summary) - compress the TerraME/LUCCME counterpart sentence in Software Design - tighten the ExperimentRecord provenance sentence in Performance Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Doc/readme platform scope
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.