langcog / metalab2

MetaLab -- Community-augmented meta-analysis
http://metalab.stanford.edu/
MIT License
21 stars 8 forks source link

ChildDevPaper does not show up #133

Open christinabergmann opened 5 years ago

christinabergmann commented 5 years ago

No idea what's wrong, the filenames match but it says 404 not found http://metalab.stanford.edu/reports/ChildDevPaper.html

(I assume something goes wrong during generating the html, if someone could take a look, that'd be great.)

christinabergmann commented 5 years ago

Moving this comment from Mike over here:

re: paper

Quitting from lines 9-17 (ChildDevPaper.Rmd)

package or namespace load failed for 'tidyverse' in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]):

namespace 'tidyr' 0.7.0 is already loaded, but >= 0.7.2 is required

mllewis commented 5 years ago

I'm happy to spend some time debugging this, but I'm having trouble even getting the app running locally on my computer.... is the readme file the most up-to-date information about the structure of things?

Molly

On Thu, May 30, 2019 at 7:56 AM Christina Bergmann notifications@github.com wrote:

Reopened #133 https://github.com/langcog/metalab2/issues/133.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/133?email_source=notifications&email_token=ABO7NCMP2V55PI7XUAMP6YLPX7FGPA5CNFSM4HLH75G2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORW63HRY#event-2378019783, or mute the thread https://github.com/notifications/unsubscribe-auth/ABO7NCNXAU5ZBL5B5DSKBSTPX7FGPANCNFSM4HLH75GQ .

shotsuji commented 5 years ago

Hi Molly, I just looked into it, does it help if you comment out setwd("..") in main_builder.R? That did the trick for me (assuming you open from the metalab2 RProject. I published that change in a new branch called "Test" just in case. @cissard also for good code hygiene please don't leave in script parts with your own local paths (even if commented out)

mcfrank commented 5 years ago

We need to swap to using the "here" package throughout the codebase to avoid this kind of craziness. :)

On Fri, May 31, 2019 at 6:39 AM Sho Tsuji notifications@github.com wrote:

Hi Molly, I just looked into it, does it help if you comment out setwd("..") in main_builder.R? That did the trick for me (assuming you open from the metalab2 RProject. I published that change in a new branch called "Test" just in case. @cissard https://github.com/cissard also for good code hygiene please don't leave in script parts with your own local paths (even if commented out)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/133?email_source=notifications&email_token=AAI25F7LTMXV4AHTW3TZFULPYD54ZA5CNFSM4HLH75G2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWU36QI#issuecomment-497663809, or mute the thread https://github.com/notifications/unsubscribe-auth/AAI25F5VRFR3DZJG4VMT5T3PYD54ZANCNFSM4HLH75GQ .

erikriverson commented 4 years ago

@christinabergmann Building the ChildDevPaper fails if the tidyverse package is not installed (https://github.com/langcog/metalab2/blob/master/reports/ChildDevPaper.Rmd#L13)

We can either add tidyverse to the required packages list or only load the particular packages used in this Rmd. Any preference between those two?

Edit: re-reading @christinabergmann comment above, the issue arises if the tidyverse package fails to load. That can either happen if it's not installed (as was the case locally for me), or if other constraints fail, such as version compatibility issues with tidyr (which is happening on production and referenced in the comment above)

mcfrank commented 4 years ago

I'd add tidyverse...

On Tue, May 5, 2020 at 9:11 AM Erik Iverson notifications@github.com wrote:

@christinabergmann https://github.com/christinabergmann Building the ChildDevPaper fails if the tidyverse package is not installed ( https://github.com/langcog/metalab2/blob/master/reports/ChildDevPaper.Rmd#L13 )

We can either add tidyverse to the required packages list or only load the particular packages used in this Rmd. Any preference between those two?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/133#issuecomment-624149232, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI25F26TFEBCQN467BXR63RQA3CZANCNFSM4HLH75GQ .

christinabergmann commented 4 years ago

I'd agree with Mike, thanks Eric, for figuring this out.

erikriverson commented 4 years ago

This is now blocked by #135 . Once that is resolved, the ChildDevPaper will build.