Repository navigation
perf(runtime): share derived defaults and verdicts between occurrences of one shape #324
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
ef8a3f8
feat(runtime): share derived defaults and verdicts between occurrence…
devin-ai-integration[bot] c6531ed
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] c780905
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] e687cd9
Merge branch 'feature/fleet-mode-stress-model' into feature/fleet-spa…
devin-ai-integration[bot] 1fdd567
perf(runtime): gather declared-scope coverage only when a later type …
devin-ai-integration[bot] 3ab614c
test(runtime): cover shared defaults over cyclic and failing derivations
devin-ai-integration[bot] c97dd12
test(runtime): cover every shared scalar kind and check bindings by t…
devin-ai-integration[bot] 0369fdc
docs(runtime): record shared defaults and verdicts, their measurement…
devin-ai-integration[bot] 2ade6b1
Merge branch 'feature/fleet-mode-stress-model' into feature/fleet-spa…
devin-ai-integration[bot] b758f6b
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 1282d3e
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] fae2c05
fix(runtime): key shared read paths by segment so a quoted dotted nam…
devin-ai-integration[bot] 026798d
fix(runtime): keep traces and held images exact under shared defaults
devin-ai-integration[bot] 7229233
docs(runtime): note that a traced context shares no derived default o…
devin-ai-integration[bot] febd9c6
docs(compliance): record that traced contexts evaluate every check
devin-ai-integration[bot] 4fd42ad
docs: refresh generated test counts
devin-ai-integration[bot] 7c34dc8
fix(runtime): owe every lazy element a shared default read through a …
devin-ai-integration[bot] e65e75c
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 8559339
test(runtime): name the shared object by its first path in the identi…
devin-ai-integration[bot] 3c1ccec
fix(runtime): keep values and verdicts decided over an extent out of …
devin-ai-integration[bot] 9ceb509
fix(runtime): take a failed image's shared records off the destination
devin-ai-integration[bot] ab4b00a
docs: refresh generated test counts
devin-ai-integration[bot] d549710
fix(runtime): rewind the shared-defaults-taken count with a snapshot
devin-ai-integration[bot] 980062a
fix(runtime): journal the shared-defaults-taken count with the take
devin-ai-integration[bot] 78eb683
fix(runtime): carry a feature value's assumed population through a he…
devin-ai-integration[bot] 4405226
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] d081133
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 4f9419a
fix(runtime): owe nothing for a taken value derived again
devin-ai-integration[bot] dce585a
fix(runtime): leave a default or verdict derived over a lifetime unsh…
devin-ai-integration[bot] de575e9
docs: recount test figures for the lifetime sharing regressions
devin-ai-integration[bot] 73d62a4
test(runtime): an extent after taken verdicts counts every occurrence
devin-ai-integration[bot] 8285a9f
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 97418b0
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] dcff1ef
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] a5db33d
fix(runtime): key shared verdicts by the kind of check
devin-ai-integration[bot] 3e5095c
docs(runtime): shorten the verdictKey comment
devin-ai-integration[bot] de653d1
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 4bb3a33
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 64964f9
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 3168694
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 32b3883
fix(runtime): keep a random draw from being shared between occurrences
devin-ai-integration[bot] a8fb862
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] e240e5d
fix(runtime): index subsetters under every subsetted name, not only t…
devin-ai-integration[bot] d8c39a7
Merge branch 'feature/fleet-mode-stress-model' into feature/fleet-spa…
devin-ai-integration[bot] afbdb61
fix(runtime): derive clock-dependent defaults and checks per occurrence
devin-ai-integration[bot] 77c68fa
Merge branch 'feature/fleet-mode-stress-model' into feature/fleet-spa…
devin-ai-integration[bot] 7cc9282
Merge remote-tracking branch 'origin/feature/fleet-mode-stress-model'…
devin-ai-integration[bot] 758c9c2
fix(runtime): refuse shared defaults and verdicts along a destroyed o…
devin-ai-integration[bot] 26882b1
test(runtime): run the sparse differential per file in parallel and s…
devin-ai-integration[bot] 8d5b9eb
test(runtime): read the fleet differential through the parts the cons…
devin-ai-integration[bot] ca487da
chore(runtime): merge the fleet-mode stress model branch
devin-ai-integration[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
16 changes: 16 additions & 0 deletions
16
changes/unreleased/occurrence-shared-defaults.performance.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| - **Occurrences of one shape share their derived defaults and their verdicts.** A `=` default | ||
| that one pristine occurrence of a type derives from nothing but declared values under itself | ||
| is now recorded against the occurrence's shape — its type, classifiers and holding feature — in | ||
| a side table of the runtime context, and every other pristine occurrence of that shape reads | ||
| the recorded value instead of deriving it again and materializing the component tree the | ||
| derivation walked; an occurrence that states, writes, binds or classifies anything the | ||
| derivation read derives on its own, and a write under an occurrence invalidates what it took. | ||
| Within one `-satisfy` or `-validate=<object>` report, a check over occurrences of one shape is | ||
| evaluated once per distinct set of inputs and its verdict fanned out to each occurrence, which | ||
| still reports its own verdict, message and path in the same order. Values, verdicts and | ||
| diagnostics are unchanged, as `TestSparseValuesDifferential` asserts with sharing on and off | ||
| (`OPENSYSML_SHARED_DEFAULTS=0` turns it off; a context recording a trace shares nothing, so | ||
| the trace lists every evaluation). On the 12 800-satellite fleet constellation, | ||
| checking its 2 412 satisfaction assertions drops from 8.84 s and 23.4 GiB allocated to 4.59 s | ||
| and 7.2 GiB, and reading one summed attribute over every occurrence from 42.7 s and 141.3 GiB | ||
| to 3.85 s and 2.7 GiB. |
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.