Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/235
> Does this complete JENA-1292?
yes.
In JENA-1292's discussion I had suggested a way to automatically remove
from the @context the properties that belong to the @vo
Github user fpservant closed the pull request at:
https://github.com/apache/jena/pull/211
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/211
closing this, as more work is still needed
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user fpservant opened a pull request:
https://github.com/apache/jena/pull/235
jsonld: @vocab in @context for "" ns prefix
"" prefix are not allowed in jsonld's @context. "" ns prefix defined in a
model were therefore ignored. This pull req
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/211
@afs sorry for the time before coming back to this. I just tried in class
``JenaRDF2JSONLD`` to replace (first line of constructor)
```
NodeToLabel labels
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/211
@afs OK, that was a quick hack, based on the existing. I'll need more time,
then
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/211
@afs yes
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user fpservant opened a pull request:
https://github.com/apache/jena/pull/211
writing jsonld: preserve id of blanknodes if they are valid jsonld
identifiers
current process to output jsonld computes identifiers for the blanknodes:
_:b0, _:b1, etc.
Ids of blanknodes
GitHub user fpservant opened a pull request:
https://github.com/apache/jena/pull/207
JsonLDWriter.toJsonLDJavaAPI: create a JsonLD API object from jena data
small refactoring of JsonLDWriter to expose a static method that allows to
convert jena RDF data to JsonLD-java API's object
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/203
@afs jsonld-java 0.9.0 solves the issue (and I have been able to run maven
install on jena-arq with it).
How do you want to handle jsonld-java's version change? Other PR? Shouldn't
I wait
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/203
>> current jena has a problem reading JSON-LD
> jsonld-java is reporting an error, all Jena does, and can do, is pass
that error upwards.
reformulating: I have problem wit
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/203
Hi,
there are very different points in the discussion, all around one test that
attempts to connect to an external resource.
1) It must first be noted that the test in question
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/203
@ajs6f are you speaking of the "org.apache.jena.riot.RiotException: loading
remote context failed" thing?
1) note that in the context of a test, jena doesn't want to depend on a
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/203#discussion_r95216878
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/out/JenaRDF2JSONLD.java ---
@@ -0,0 +1,100 @@
+/**
+ * Licensed to the Apache Software
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/203#discussion_r95170644
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/lang/JsonLDReader.java ---
@@ -45,9 +45,20 @@
import org.apache.jena.riot.system.* ;
import
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/203
@afs I think that @ajs6f refers to code:
```
InputStream in = new
ByteArrayInputStream(jsonld.getBytes(StandardCharsets.UTF_8));
+Dataset ds = DatasetFactory.create
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/203
@ajs6f thanks for the review. I made most (all ?) of the changes that you
suggested. Remains the problem "loading remote context failed:
http://schema.org/; - unrelated to this PR p
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/203#discussion_r95095721
--- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java
---
@@ -0,0 +1,81 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/203#discussion_r95095597
--- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java
---
@@ -0,0 +1,81 @@
+/*
+ * Licensed to the Apache Software Foundation
GitHub user fpservant opened a pull request:
https://github.com/apache/jena/pull/203
JsonLDReader: possibility to override the @context
following a question on the user list: "[Overriding @context when loading
JSON-LD](http://jena.markmail.org/message/uf3zkzybv7gb4crt)",
Clone URL (Committers only):
https://cms.apache.org/redirect?new=anonymous;action=diff;uri=http://jena.apache.org/documentation%2Fio%2Frdf-output.mdtext
fpservant
Index: trunk/content/documentation/io/rdf-output.mdtext
===
--- trunk
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
@ajs6f ah, OK, thank you. So I'll do it shortly
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
@afs thanks to you.
> What should I link to to say "it's new - read this"?
good question. I think that the best is to link to the sample code, where
I've tried to exp
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
@afs as I said in y previous mess, I don't seem to be able to get rid of
the conflict (but I am not an expert with git and github) The only solution I
see that would probably work would mean to move
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/139#discussion_r81478977
--- Diff: jena-arq/pom.xml ---
@@ -85,8 +84,9 @@
- com.github.jsonld-java
- jsonld-java
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
@afs,
> If possible, please see if the PR can be alighed to the current Jena
master branch for a clean merge
do you mean: "see if I can do something to get rid of the
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
@stain yes, JSONLD-java can download a context from a URL, and cache it.
The problem is, there is no way that I know of that would allow me to use that
context in a write operation initiated from
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
I updated this with some sample code, and it seems to me that this is as
good as *I* can. The sample code is the best place to learn how it works and
what it does.
Andy, answering your
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
Andy,
I have reverted to an unmodified version of RDFDataMgr (without write
methods with a context param).
I am sorry that I take so long to finalize this stuff. Remember that one
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
Hi Andy,
getting back to this after, well, a long summer.
I have a remark regarding your following comment:
> RDFDataMgr
>
> The way to set things specially for a writing
Github user fpservant commented on the issue:
https://github.com/apache/jena/pull/139
Andy
> Apologies that this has taken so long.
no problem, really. Thanks for this answer. Give me some time to take all
your valuable comments into account.
just one remark:
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/139#discussion_r62686179
--- Diff:
jena-arq/src/test/java/org/apache/jena/riot/writer/TestJsonLDWriter.java ---
@@ -0,0 +1,457 @@
+/*
+ * Licensed to the Apache Software
Github user fpservant commented on a diff in the pull request:
https://github.com/apache/jena/pull/139#discussion_r62686055
--- Diff: jena-arq/src/main/java/org/apache/jena/riot/out/JsonLDWriter.java
---
@@ -79,26 +130,85 @@ public void write(OutputStream out, DatasetGraph
GitHub user fpservant opened a pull request:
https://github.com/apache/jena/pull/139
JSON-LD output
Hi,
here's my attempt at improving user's control over JSON-LD output and
therefore, my request for comments.
Following discussions on jena-users list :
[[1]](http
34 matches
Mail list logo