[GitHub] jena issue #299: Turtle Star

2017-11-22 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 Thanks Andy! I agree with what you write. Do you think there is a chance for such a separate maven module to become part of the official family of Apache Jena maven modules? ---

[GitHub] jena issue #299: Turtle Star

2017-11-10 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 Thanks @mschroeder-github for taking the initiative of writing an RDF* parser for Jena! @afs Before responding to some of your comments, I should mentioned that I am still actively working

[GitHub] jena issue #299: Turtle Star

2018-05-06 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 I agree. Making `checkTriple` protected in `ParserProfileStd` would be an even better idea. Please do it. While we are at it, can you add another constructor to `TurtleShell` that allows me

[GitHub] jena pull request #418: pass a custom NodeFormatter to TurtleShell

2018-05-09 Thread hartig
GitHub user hartig opened a pull request: https://github.com/apache/jena/pull/418 pass a custom NodeFormatter to TurtleShell … added another constructor that can be used to You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] jena issue #299: Turtle Star

2018-05-09 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 > That does not work because the `ParserProfileStd` supplied to `TurtleStarReaderRIOT` has already been created, which is why the suggestion is to use a forwarding proxy object (the wrap

[GitHub] jena issue #299: Turtle Star

2018-05-09 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 Add the PR #418 ---

[GitHub] jena issue #418: pass a custom NodeFormatter to TurtleShell

2018-05-09 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/418 This PR provides the extension proposed in https://github.com/apache/jena/pull/299#issuecomment-386918751 ---

[GitHub] jena issue #418: pass a custom NodeFormatter to TurtleShell

2018-05-16 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/418 @afs thanks for merging this PR, and also for your work on #416. Now I will be waiting eagerly for v.3.8.0 ;-) ---

[GitHub] jena issue #299: Turtle Star

2018-06-11 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 Agreed. ---

[GitHub] jena issue #299: Turtle Star

2017-12-23 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 Great Andy, thanks! I will check it out after the holidays. (I'm a bit overloaded here at the moment) ---

[GitHub] jena issue #299: Turtle Star

2018-04-21 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 Ah, one more thing: @afs, would it be possible to extend the `checkTriple` method of `ParserProfileStd` to permit the subject and the object of a triple to be `Node_Triple`? This is the only thing

[GitHub] jena issue #299: Turtle Star

2018-04-21 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/299 I finally have implemented a Turtle* parser that hooks into Jena's RIOT parser framework and uses the `Node_Triple` class that @afs had added to Jena in PR #327. Find the code at https://github.com

[GitHub] jena pull request #473: Extendable versions of classes for printing result s...

2018-10-01 Thread hartig
GitHub user hartig opened a pull request: https://github.com/apache/jena/pull/473 Extendable versions of classes for printing result sets This PR modifies the classes `CSVOutput` and `TSVOutput` to make it possible to extend them and override their behavior in subclasses. I want

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-10-03 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/473 > When Jena 3.9.0 is completed, I can come back and look at this PR. In principle, there is no reason not have have them extensible. Sounds good. Thanks! > What syntax are you

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-12-12 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/473 > Where and how are you wanting to output text format? Where? In the command line tool for executing SPARQL* queries (see [`ExecuteSPARQLStar` in RDFstarTools](https://github.com/RDFs

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-12-06 Thread hartig
Github user hartig commented on the issue: https://github.com/apache/jena/pull/473 Thanks Andy!! I will be waiting for Jena 3.10 to use this in my code. What about the other point mentioned in my initial comment in this PR? You have been hesitant to add