Closed nawtrey closed 2 months ago
As an additional test for this branch, I have consolidated all the code from the KDA manuscript and executed it on this branch to ensure it does not break the current code.
The above script executes and runs to completion on this branch. However, it actually does not run on the current master
due to a missing key="name"
... in one of the very first lines of code in the paper... 🤦♂️
So I guess this is serving multiple purposes 🦝
Description
In anticipation of the KDA API (issue #83) I'm updating the calculation functions to default to symbolic outputs. Since this is really KDA's biggest appeal, it's a little awkward that symbolic outputs are not the default. Eventually it would be nice to deprecate
output_strings=True
in lieu ofsymbolic=True
, but for now this should be good.Changes
Changes the default function calls for all
kda
calculation functions (e.g.calc_state_probs()
,calc_net_cycle_flux()
) tokey="name"
andoutput_strings=True
.Fixes function calls in
test_kda.py