dajmcdon / rtestim

https://dajmcdon.github.io/rtestim/
Other
4 stars 0 forks source link

Remove `echo = FALSE` in line 249 of delay-distributions vignette #69

Closed rachlobay closed 2 months ago

rachlobay commented 3 months ago

The code chunk starting on line 249 of delay-distributions.Rmd should be shown so that the reader can use the resulting can_pred_class to construct the time-varying delay matrix starting on line 395 of the vignette (so I think that removing echo = FALSE in the 249 code chunk should do the trick to fix this). I'm not a collaborator on this repo, so I don't see that I can do this directly on the repo. When someone gets a chance, have a look at this and fix if necessary

jiapivialiu commented 2 months ago

I think the code can be accessed from the source file here. The users should be able to clone the repo and run the code by themselves to get the results.

Making the code chunk in line 249 alone won’t be helpful, since it uses the results from the previous chunk, which is invisible and very long. And making both code chunks visible will take too much space in the vignette.

So I save the relevant dataset to the vignettes directory, so users can directly use it without running the code.