Closed yhoriuchi closed 1 year ago
I don't understand -- do you want me to remove the print() method? Is that it?
I don't quite understand the merit of print() for predict_tau(). Users only want to show the estimated IRR and plot the figure. Wha does print() do?
On Sun, Aug 13, 2023 at 2:02 PM Aaron R. Kaufman @.***> wrote:
I don't understand -- do you want me to remove the print() method? Is that it?
— Reply to this email directly, view it on GitHub https://github.com/yhoriuchi/projoint/issues/18#issuecomment-1676425681, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC332BE4ZB4YHNFUY7ROZQLXVEJCNANCNFSM6AAAAAA3OY6UWE . You are receiving this because you authored the thread.Message ID: @.***>
We can definitely remove it, but we can also keep it in if we decide to expand on it later. Currently, it is not harmful and it contains different information from summary()
OK! Then, can you revise the vignette and close this issue?
On Sun, Aug 13, 2023 at 2:09 PM Aaron R. Kaufman @.***> wrote:
We can definitely remove it, but we can also keep it in if we decide to expand on it later. Currently, it is not harmful and it contains different information from summary()
— Reply to this email directly, view it on GitHub https://github.com/yhoriuchi/projoint/issues/18#issuecomment-1676427152, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC332BEZA5B34B2UENRA7XLXVEJ4BANCNFSM6AAAAAA3OY6UWE . You are receiving this because you authored the thread.Message ID: @.***>
We should use
summary()
andplot()
. Do we needprint()
forpredict_tau()
?