Skip to content

Recorded cat porc #84

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.

Sign up for GitHub

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

Jump to bottom
Draft
frankier wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Recorded cat porc #84

frankier wants to merge 7 commits into from

Conversation

@frankier
Copy link
Member

@frankier frankier commented Aug 30, 2025

No description provided.

@frankier frankier marked this pull request as draft August 30, 2025 11:33
@coveralls
Copy link
Collaborator

coveralls commented Aug 30, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 20424194867

Details

  • 14 of 399 (3.51%) changed or added relevant lines in 25 files are covered.
  • 6 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-5.8%) to 32.982%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Aggregators/optimizers.jl 0 1 0.0%
src/ConfigBase.jl 0 1 0.0%
src/NextItemRules/strategies/randomesque.jl 0 1 0.0%
src/Sim/Sim.jl 0 1 0.0%
src/TerminationConditions.jl 0 1 0.0%
src/Aggregators/tracked.jl 0 2 0.0%
src/NextItemRules/strategies/balance.jl 0 2 0.0%
src/NextItemRules/strategies/pointwise.jl 0 2 0.0%
src/Responses.jl 0 2 0.0%
src/NextItemRules/prelude/next_item_rule.jl 0 3 0.0%
Files with Coverage Reduction New Missed Lines %
src/Aggregators/tracked.jl 1 27.27%
src/Sim/recorder.jl 5 0.0%
Totals Coverage Status
Change from base Build 20410093924: -5.8%
Covered Lines: 626
Relevant Lines: 1898
💛 - Coveralls

@github-actions
Copy link

github-actions bot commented Aug 30, 2025 •
edited
Loading

Benchmark Results

main 06f3c03... main / 06f3c03...
next_item_rules/4pl/mean_point_mepv_10 2.4 ± 0.011 ms 2.36 ± 0.01 ms 1.02 ± 0.0065
next_item_rules/4pl/mean_point_mepv_bare 0.923 ± 0.015 ms 0.931 ± 0.015 ms 0.991 ± 0.023
next_item_rules/4pl/mode_point_mepv_10 1.95 ± 0.01 ms 1.9 ± 0.0097 ms 1.02 ± 0.0075
next_item_rules/4pl/mode_point_mepv_bare 0.846 ± 0.015 ms 0.844 ± 0.015 ms 1 ± 0.025
time_to_load 1.86 ± 0.013 s 1.87 ± 0.037 s 0.994 ± 0.021

Benchmark Plots

A plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR.
Go to "Actions"->"Benchmark a pull request"->[the most recent run]->"Artifacts" (at the bottom).

Frankie Robertson added 6 commits December 21, 2025 16:52
Ignore claude local settings
571798c
Add show(...) for CatLoop
9d76255
Add RecordedCatLoop
7f4d7b6
Fix spelling error Criteron => Criterion
58f7203
Add item bank to power_summary of RecordedCatLoop
ab954d1 
* Clear recorded data on run_cat startup
Improve printing of RecordedCatLoop
69a8e66 
  * Use power_summary(...) as main method for all printing
  * Add StdDevEstimator as a std dev measure
  * Add way to make recording entry before any items are administered
  * Add all kinds printing for Recorder and Recording
  * Format numbers to 3dp in Recording
Add all sorts of mean and dispersion estimation stuff
06f3c03 
 * Delete a lot of comments
 * Remove vestigial dependency on RDataGet
 * Use derivatives stuff from PsychometricsBazaarBase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@frankier @coveralls