You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
At the moment it seems that we only test the summary method for the pymc_experiments.DifferenceInDifferences class, and that is in the test_pymc_experiments.py file...
We should make sure that we are testing the summary method for all of the pymc_experiment classes. Though I believe the InstrumentalVariable class doesn't have a summery method.
This could be done by adding more to the current test_pymc_experiments.py file, but it could be worth thinking about simply adding to existing tests in test_integration_pymc_examples.py. My thinking is that this would involve less setup / computation time as we wouldn't be doing any additional model fitting.
The text was updated successfully, but these errors were encountered:
At the moment it seems that we only test the
summary
method for thepymc_experiments.DifferenceInDifferences
class, and that is in thetest_pymc_experiments.py
file...CausalPy/causalpy/tests/test_pymc_experiments.py
Lines 10 to 23 in 3bb81a7
We should make sure that we are testing the summary method for all of the
pymc_experiment
classes. Though I believe theInstrumentalVariable
class doesn't have asummery
method.This could be done by adding more to the current
test_pymc_experiments.py
file, but it could be worth thinking about simply adding to existing tests intest_integration_pymc_examples.py
. My thinking is that this would involve less setup / computation time as we wouldn't be doing any additional model fitting.The text was updated successfully, but these errors were encountered: