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
I was fixing an edge-case bug in ard_categorical() when the variable was all NA. To correct this, I made an update to the way the denominator N was calculated. After the update, there is now a space for an efficiency gain when denominator = c("column", "row", "cell"), the most common cases.
Instead of handling denominator = c("column", "row", "cell") entirely separately, we can instead include useNA="always" for the primary variable tabulation, then calculate the Ns by performing a grouped sum of "n".
The other cases allowed in denominator will still be handled in the same way they are now. But as these are less common calculations, I expect an overall gain in efficiency.
The text was updated successfully, but these errors were encountered:
I was fixing an edge-case bug in
ard_categorical()
when the variable was all NA. To correct this, I made an update to the way the denominator N was calculated. After the update, there is now a space for an efficiency gain whendenominator = c("column", "row", "cell")
, the most common cases.denominator = c("column", "row", "cell")
entirely separately, we can instead includeuseNA="always"
for the primary variable tabulation, then calculate the Ns by performing a grouped sum of"n"
.denominator
will still be handled in the same way they are now. But as these are less common calculations, I expect an overall gain in efficiency.The text was updated successfully, but these errors were encountered: