search

incf-nidash / nidm-specs

Neuroimaging Data Model (NIDM): describing neuroimaging data and provenance
nidm.nidash.org
Other
33 stars 30 forks source link

fix webpage generation #467

Closed khelm closed 6 years ago

khelm commented 6 years ago

Updated packages and modified fabfile.py and pelicanconf.py files to reflect new package requirements so that web pages are now created. See issue #461 for details.

cmaumet commented 6 years ago

Hi @khelm, thanks for those updates! It's great to see the website code working again!

I think I've found a solution to get rid of the full paths. As far as I understand, the issue was related to the fact that calls to local ignores the current conda environment. I've added optional arguments to define path to the virtual environment and the environment name:

Example of call if using a conda environment call nidm2:

fab -H 127.0.0.1 serve --ve=/path/to/anaconda2/envs/nidm2/bin/ --envname=nidm2

Without environment (i.e. when pelican is available by default):

fab -H 127.0.0.1 serve

References (maybe useful later on): prefix in fabric2, Context manager in fabric 1.

If this work on your machine, I think that's ready to go? We'll have to update the documentation on the wiki.

cmaumet commented 6 years ago

An alternative could be to use c.run instead of c.local but I could not get it to work... Let me know if you are fine with the current solution. Thanks!

khelm commented 6 years ago

Thanks @cmaumet for these updates! I also tried c.run, and had no success - I think that is used to run commands on the remote server only, but I haven't looked into this extensively.

I'll have to play around with the python aspect a bit more as I'm not using virtual environments at the moment. I should probably create one just for nidm as you have.

In 800e002 I see that you created a relative path for the pelican-plugins directory. So you are putting that directory into the nidm directory?

cmaumet commented 6 years ago

Hi @khelm! Following your comment, I've moved the pelican-plugins folder under doc (and also added the folder to .gitignore). Do you think this is now ready for merge?

khelm commented 6 years ago

Thanks for adding in the changes and re-organizing. I think it should be ready to go now. +1 to merge.

cmaumet commented 6 years ago

Thanks @khelm! I've updated the documentation in the wiki accordingly, can you double check that it looks okay? --> https://github.com/incf-nidash/nidm-specs/wiki/Generate-the-NIDM-Website

Let's merge!