Add readable and fast compiled model modes - #191
Draft
VEZY wants to merge 187 commits into
Draft
Conversation
Added generated GraphSimulation compatibility validation in src/compiled_model.jl (line 68). Compiled scripts now embed a model signature and fail early if called with a different mapping/model set or wrong same-rate vs multirate mode. Added the compatibility signature to generated script headers for inspection. Hoisted multirate per-node setup out of the timestep loop in src/compiled_model.jl (line 338): node metadata, model, model spec, and model clock are now generated as stable locals like _mr_Leaf_carbon_assimilation_model_clock. Kept temporal input resolution, meteo sampling, output publishing, and requested exports delegated to the existing runtime helpers. Kept the GraphSimulation(...; type_promotion=...) constructor fix in src/mtg/GraphSimulation.jl (line 37). Added tests for compatibility validation and multirate hoisting in test/test-compiled-model.jl (line 113).
Contributor
Benchmark Results (Julia v1)Time benchmarks
Memory benchmarks
|
Member
Author
|
Comment: use https://github.com/JuliaDebug/CodeTracking.jl for a helper to retrieve code after the functions were created (in the case that the modeler didn't use the macro on the run! Method) |
- Added `Domain`, `SimulationMapping`, and `DomainSimulation` for composing plant, soil, scene, and future environment domains. - Added cross-domain stream/value dependencies with `AllDomains(...)` and explicit `Route(...)` materialization. - Added hard-domain dependencies with `HardDomains(...)`, `dependency_targets(...)`, `model_target(...)`, and `run_target!(...)` so scene models can manually run plant/soil targets, including iterative workflows. - Added MTG-backed domain support with selectors, multi-plant domains, graph-domain output publication, dynamic topology registration, organ removal, and reparenting. - Added `Updates(:var; after=...)` for intentional same-scale variable updates by later models. - Added environment/microclimate backend protocol with `AbstractEnvironmentBackend`, `GlobalConstant`, `meteo_inputs_`, `meteo_outputs_`, sampling, scattering, and validation. - Extended multirate support with `Dates`-based timesteps, inferred/explicit bindings, temporal output policies, scoped streams, meteo aggregation, and requested output export. - Added MAESPA-style example in [examples/maespa_domain_example.jl](/Users/rvezy/Documents/dev/PlantSimEngine/examples/maespa_domain_example.jl) with two plant species, shared soil, scene-scale iterative energy balance, and hard-domain targets. - Added docs for domain simulation, model traits, hard-domain target design, and implementation plan/handoff notes. - Added focused tests for domains, environment backends, meteo traits, updates, and the MAESPA example. Verification already run: - `julia --project=test test/runtests.jl` passed: `1844 / 1844` - `julia --project=docs docs/make.jl` passed - `git diff --check` passed Files to remember are untracked too, especially: - `src/domains/domain_simulation.jl` - `src/time/runtime/environment_backends.jl` - `src/dependencies/update_dependencies.jl` - `examples/maespa_domain_example.jl` - new docs under `docs/src/dev/` - new tests under `test/test-domain-simulation.jl`, `test/test-environment-backends.jl`, `test/test-maespa-domain-example.jl`, `test/test-meteo-traits.jl`, `test/test-updates.jl`
- remove ModelList compatibility
- replace Symbol("") mapping sentinels
- refactor repeated multirate input-resolution flow
- split the large domain runner into scheduler/routes/environment/graph/publication pieces
- replace assertions by more standard errors
Shared Status / StatusView interface helpers. Extracted repeated test graph comparison runner. Extracted toy tutorial MTG helpers. Typed export plans, reverse mappings, temporal stream aliases. Preserved renamed producer source variables in binding inference. Added explicit domain DAG run ordering. Added topology reindexing for RefVectors after add/remove/reparent. Split MAESPA scene solver math from side effects and added validation tests.
…r status anymore)
Fix recursive call leading to stack overflow
compile_scene (line 638) now validates required inputs_(model) variables. A required input is valid if it has a compiled Inputs(...)/inferred binding or already exists on the target object Status. Missing inputs now error with application id, object id, and input name. Added tests for missing inputs, ambiguous same-object producers, and status-provided inputs.
…cies. Unscoped Inputs(...) / Calls(...) dependency selectors now infer scope from the consumer object: scene consumers default to SceneScope() non-scene consumers default to Self() Direct public resolve_object_ids(scene, Many(...)) remains scene-wide unless within=... is explicit. Shared cross-scope dependencies from organs now need explicit within=SceneScope(), as in the updated shared-soil test. Added tests showing plant-level unscoped leaf inputs are plant-local, while scene-level unscoped leaf inputs are scene-wide.
Exceptional-organ Override(...). Instance and object-level model replacement validation. Concrete dispatch preserved for same-type parameter overrides. explain_instances(scene) and instance membership in explain_objects. New organs inherit instance kind and species. Hard-called overridden models use the correct per-object implementation.
# Conflicts: # .gitignore # src/PlantSimEngine.jl # test/runtests.jl
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.
Summary
mode=:readable | :fastto compiled model generation and preserveStatus-backed execution semanticsTesting
ModelMappingcoverage passesgit diff --checkpasses