krisrs1128 / treelapse

timelapse + trees = treelapse
http://krisrs1128.github.io/treelapse
8 stars 0 forks source link

webshot package #154

Open spholmes opened 7 years ago

spholmes commented 7 years ago

For the vignettes it seems the package webshot is needed.

krisrs1128 commented 7 years ago

I'm not sure I want to require this, because I don't want there to be too many dependencies. The screenshots should have already been included in the vignettes folder, are they not appearing for you?

On a side note -- this seems to be better than taking screenshots, because you can convert the SVGs into PDFs.

spholmes commented 7 years ago

No I don't need webshot, somehow the Vignette complained because I didn't have that package installed.

On Mon, Dec 12, 2016 at 6:28 AM, krisrs1128 notifications@github.com wrote:

I'm not sure I want to require this, because I don't want there to be too many dependencies. The screenshots should have already been included in the vignettes folder, are they not appearing for you?

On a side note -- this http://nytimes.github.io/svg-crowbar/ seems to be better than taking screenshots, because you can convert the SVGs into PDFs.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/krisrs1128/treelapse/issues/154#issuecomment-266301772, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJcvdjnVlkdGkgMBjGZ2Z7CclhR6p88ks5rHE7ygaJpZM4LJ-sr .

-- Susan Holmes Professor, Statistics and BioX John Henry Samter Fellow in Undergraduate Education Sequoia Hall, 390 Serra Mall Stanford, CA 94305 http://www-stat.stanford.edu/~susan/

krisrs1128 commented 7 years ago

Hm, that's strange, I don't have webshots installed, but I can compile the vignettes. Can you copy the error message you see?