Skip to content
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

feat: Results analysis #103

Merged
merged 48 commits into from
Oct 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
48 commits
Select commit Hold shift + click to select a range
e103df6
feat: Results analysis
ThetaSinner Sep 30, 2024
8e9f2fd
Basic data queries
ThetaSinner Oct 1, 2024
2f9f973
Initial summarizer tool
ThetaSinner Oct 4, 2024
ebd245f
Summarise more scenarios
ThetaSinner Oct 7, 2024
61a4180
Summary for local signals
ThetaSinner Oct 7, 2024
81d6bff
Summary for remote call rate
ThetaSinner Oct 10, 2024
43cbee2
Summary for single write many read
ThetaSinner Oct 10, 2024
928e307
Summary for write validated
ThetaSinner Oct 10, 2024
ffdf10e
Summary for validation receipts
ThetaSinner Oct 11, 2024
d8ba218
Summary for write query
ThetaSinner Oct 11, 2024
8d9a76b
Summary for write read
ThetaSinner Oct 11, 2024
0479c52
Summary for write validated
ThetaSinner Oct 11, 2024
5a4928e
Summary for zome call single value
ThetaSinner Oct 11, 2024
9dec3da
Fetch test data for use in tests
ThetaSinner Oct 11, 2024
c1cec85
Add first test for app install
ThetaSinner Oct 14, 2024
fef9aa2
Add test for app install large
ThetaSinner Oct 14, 2024
81286b5
Tidy up and add test for first call
ThetaSinner Oct 15, 2024
4906d4a
Local signals test
ThetaSinner Oct 15, 2024
5e98818
Add trends and partition timings
ThetaSinner Oct 18, 2024
d84a47e
Tidy
ThetaSinner Oct 18, 2024
eaf7b2c
TryCP write validated to respect partitioning
ThetaSinner Oct 18, 2024
ad0e2d4
Two party countersigning analysis
ThetaSinner Oct 18, 2024
044cfd7
Validation receipts summary
ThetaSinner Oct 18, 2024
5945df6
More docs about usage
ThetaSinner Oct 18, 2024
ef4bab7
Lint
ThetaSinner Oct 18, 2024
51511d6
Summary for dht sync lag
ThetaSinner Oct 21, 2024
a3485f7
Add summary tests for the remaining scenarios
ThetaSinner Oct 21, 2024
8c72507
Add readme for the summariser
ThetaSinner Oct 21, 2024
5cc28c1
Fix after rebase
ThetaSinner Oct 21, 2024
4c5c7d3
Update dashboards
ThetaSinner Oct 21, 2024
f0c3c76
Format TOML
ThetaSinner Oct 21, 2024
42fc706
Include test data files in workspace build
ThetaSinner Oct 22, 2024
285bdbd
First round of review changes
ThetaSinner Oct 23, 2024
29d349b
Update test data after review changes
ThetaSinner Oct 23, 2024
91620c3
Remove unused test data
ThetaSinner Oct 23, 2024
6416310
Update summariser/README.md
ThetaSinner Oct 23, 2024
e586916
Update summariser/src/analyze.rs
ThetaSinner Oct 23, 2024
6c6bd3e
Update summariser/src/lib.rs
ThetaSinner Oct 23, 2024
3f087d8
Fix test data name collisions
ThetaSinner Oct 23, 2024
9e3db9f
Revert "Remove unused test data"
ThetaSinner Oct 23, 2024
a4204dc
Fix missing test data for first_call
ThetaSinner Oct 23, 2024
abbc921
Remove test data that actually isn't used
ThetaSinner Oct 23, 2024
7e50770
Format
ThetaSinner Oct 24, 2024
28b7f4c
Update summariser/src/main.rs
ThetaSinner Oct 25, 2024
29174c0
Update framework/summary_model/src/lib.rs
ThetaSinner Oct 25, 2024
a2c1ac4
Update framework/summary_model/src/lib.rs
ThetaSinner Oct 25, 2024
9174b2e
Update changelog
ThetaSinner Oct 25, 2024
78d023f
Update CI config to match renamed role
ThetaSinner Oct 25, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ jobs:
nix develop .#ci -c bash -c "hc s clean && echo "1234" | hc s --piped create && echo "1234" | hc s --piped -f 8888 run &"

# Run the scenario for 5 seconds
RUST_LOG=info nix run .#first_call -- --connection-string ws://localhost:8888 --agents 1 --behaviour local:1 --duration 5 --no-progress
RUST_LOG=info nix run .#first_call -- --connection-string ws://localhost:8888 --duration 5 --no-progress

pkill hc && pkill holochain && pkill lair-keystore

Expand Down
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -30,3 +30,5 @@ logs/**/*.log
*.key
/data
scenarios/*/logs
run_summary.jsonl
summariser-report-*.json
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Check in the `happ_builder` whether `hc` and `cargo` are available. This is used by the scenario build script to skip
building happs if build tools are not available. This allows the project to be loaded in an environment where the
tools aren't available.
- A new tool for summarising scenario outcomes. This is called the `summariser` which is possibly a working title! The
tool is specific to the scenarios in this project but does have some re-usable pieces. It remains to be decided whether
we will separate those parts out and publish them as a crate. For now, this is private to the project.

### Changed
- Updated to new Holochain client version 0.5.0-alpha.4 which allowed `&mut self` to be replaced with `&self` in admin
Expand All @@ -26,6 +29,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
have any subscribers then that should not be considered a fatal error. It will now log a warning instead.
- Metrics now automatically include `run_id` and `scenario_name` tags.
- Update `trycp_client` and `trycp_api` dependencies to `v0.17.0-dev.6`. [#117](https://github.com/holochain/wind-tunnel/pull/117)
- When making zome calls with the TryCP client bindings, the `agent` is now reported on the metrics, taken from the target
cell_id for the call. For the wrapped `holochain_client`, this is only done when the call target is `CellId`. Or in
other words, the `agent` is not reported when calling a clone cell.
- All metrics are now reported in seconds, as an `f64`. There were some types still using milliseconds which made reporting
across scenarios more complex.

### Deprecated
### Removed
Expand Down
Loading