nd-ball / py-irt

Bayesian IRT models in Python
MIT License
118 stars 44 forks source link

Add 1PL Amortized Model #28

Closed jplalor closed 2 years ago

jplalor commented 2 years ago

@EntilZha All tests are passing, but implementing amortization required some changes to other files as well. Could you take a look at those specifically (cli, config, etc.)?

codecov[bot] commented 2 years ago

Codecov Report

Merging #28 (58e99b9) into master (53b7b84) will increase coverage by 1.22%. The diff coverage is 75.96%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   67.19%   68.42%   +1.22%     
==========================================
  Files          14       15       +1     
  Lines         942     1121     +179     
==========================================
+ Hits          633      767     +134     
- Misses        309      354      +45     
Impacted Files Coverage Δ
py_irt/cli.py 0.00% <0.00%> (ø)
py_irt/models/multidim_2pl.py 100.00% <ø> (ø)
py_irt/models/two_param_logistic.py 62.29% <ø> (ø)
py_irt/models/amortized_1pl.py 79.57% <79.57%> (ø)
py_irt/config.py 100.00% <100.00%> (ø)
py_irt/dataset.py 84.93% <100.00%> (+1.59%) :arrow_up:
py_irt/models/abstract_model.py 86.04% <100.00%> (ø)
py_irt/models/four_param_logistic.py 80.68% <100.00%> (ø)
py_irt/models/one_param_logistic.py 56.77% <100.00%> (+0.74%) :arrow_up:
py_irt/models/three_param_logistic.py 89.28% <100.00%> (-1.04%) :arrow_down:
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 53b7b84...58e99b9. Read the comment docs.