Skip to content

Path-related bugfix for broken prior images.#19

Merged
kemball merged 6 commits intomasterfrom
brokenpriors
Aug 1, 2013
Merged

Path-related bugfix for broken prior images.#19
kemball merged 6 commits intomasterfrom
brokenpriors

Conversation

@kemball
Copy link
Copy Markdown
Contributor

@kemball kemball commented Jul 31, 2013

Hopefully these changes will insulate the code against path changes in the future. The priors page now also uses ruby tags instead of hardcoded urls and added public/images/prev to the commit itself, while ignoring .png files in there.

@ghost ghost assigned kemball Jul 31, 2013
kemball added a commit that referenced this pull request Aug 1, 2013
Path-related bugfix + tag changes to prevent it happening again + folder addition to repo to prevent permissions problems in the future.
@kemball kemball merged commit 9bcba77 into master Aug 1, 2013
@dlebauer
Copy link
Copy Markdown
Member

The images are not showing up now on betydb.org. Is there a simple fix, or should I open a new bug?

@kemball
Copy link
Copy Markdown
Contributor Author

kemball commented Aug 23, 2013

It looks like PEcAn is not installed on ebi-forecast. Is this intentional? The prior images rely on PEcAn R code for DRY reasons, but I can change it if need be.

@dlebauer
Copy link
Copy Markdown
Member

Okay. Boris installed the PEcAn packages using the following code:

export R_LIBS_USER="/usr/local/R-2.15/lib64/R/library"
wget https://github.com/PecanProject/pecan/archive/v1.3.2.tar.gz
tar -xvf v1.3.2.tar.gz
./pecan/scripts/build.sh

But the image regeneration still does not work.

I delete the images, touch restart.txt, and relaod. But the pictures do not show up.

rm /usr/loca/ebi/images/prev/*.png
touch /usr/local/ebi/tmp/restart.txt
## refresh browser
## expect images

to get the images back for now:

rsync -routi pecandev:/usr/local/ebi/public/images/prev/*.png /usr/local/ebi/public/images/prev/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants