search

incf-nidash / nidm-api

A restful api and client for NIDM
http://nidm-api.readthedocs.org/
BSD 3-Clause "New" or "Revised" License
0 stars 3 forks source link

Include json-ld support #17

Closed nicholsn closed 9 years ago

nicholsn commented 9 years ago

https://github.com/RDFLib/rdflib-jsonld

vsoch commented 9 years ago

The files are jsonld, I just used a different extension because nobody sees those in the wild.

nicholsn commented 9 years ago

right but for serializing/parsing you'll want to include the library as a dependency. For example, with content negotiation you'll need this for API responses

vsoch commented 9 years ago

oh yes, I see. Let me see if adding it breaks readthedocs again :O)

vsoch commented 9 years ago

It breaks it, but I can add it as a pip command post install, see here. Can you think of a better way to do this?

Here is an example of a successful build: https://readthedocs.org/projects/nidm/builds/3444416/ (look under the setup.py to see that not having the curl installed doesn't stop the build), but on a user machine this would work fine.

and an unsuccessful build: https://readthedocs.org/projects/nidm/builds/3444406/

nicholsn commented 9 years ago

I think solution is fine, don't understand why tho. Does rtd not have curl available?

On Fri, Oct 30, 2015 at 3:15 PM, Assenav Sochat notifications@github.com wrote:

It breaks it, but I can add it as a pip command post install, see here https://github.com/vsoch/nidm-api/blob/master/nidm/script/post_install.py. Can you think of a better way to do this?

Here is an example of a successful build: https://readthedocs.org/projects/nidm/builds/3444416/ (look under the setup.py to see that not having the curl installed doesn't stop the build), but on a user machine this would work fine.

and an unsuccessful build: https://readthedocs.org/projects/nidm/builds/3444406/

— Reply to this email directly or view it on GitHub https://github.com/incf-nidash/nidm-api/issues/17#issuecomment-152662996 .

vsoch commented 9 years ago

Nope, it appears not.

vsoch commented 9 years ago

Let me try running it outside of a venv, that might do the trick.

nicholsn commented 9 years ago

is wget available? you could do something with miniconda like: https://github.com/nipy/nipype/blob/master/.travis.yml#L11

On Fri, Oct 30, 2015 at 3:25 PM, Assenav Sochat notifications@github.com wrote:

Let me try running it outside of a venv, that might do the trick.

— Reply to this email directly or view it on GitHub https://github.com/incf-nidash/nidm-api/issues/17#issuecomment-152664382 .

vsoch commented 9 years ago

I think the solution is fine for now, it's not worth it (at least now) to try and come up with a second solution to do the same thing. I'm going to close this for now!