Skip to content

Conversation

@jamesaoverton
Copy link
Member

Replaces #1205, building on work by @pk-mitre.

  • docs/ have been added/updated
  • tests have been added/updated
  • mvn verify says all tests pass
  • mvn site says all JavaDocs correct
  • CHANGELOG.md has been updated

Add support for property-object pairs in a single cell of a ROBOT template:

AP annotation property-object pairs: If the template string is AP then ROBOT will look in each cell of that column for predicate-object pairs, with a syntax similar to Turtle, such as ex:p1 'A' or ex:p2 '1234'^^xsd:integer. Using AP SPLIT=| we can specify a list of annotations: ex:p1 'A'|ex:p2 "B"@en|ex:p3 '3'^^xsd:integer|ex:p4 ex:curie|ex:p5 <http://ex.com>.

@jamesaoverton
Copy link
Member Author

@pk-mitre and @dlutz2 I've made some changes to make this more general. Can you please look at this and give me your feedback?

@dlutz2
Copy link

dlutz2 commented Mar 7, 2025

@jamesaoverton These look good. Covering all object types very useful and we need the annotation value language tag, glad you added that.

@jamesaoverton jamesaoverton mentioned this pull request Mar 10, 2025
5 tasks
Copy link
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool feature! Implementation looks good as well.

@@ -0,0 +1,2 @@
Term Term ID Annotations
ACME Communications INC acme:s-1 "acme:p-1 " Value ZZ"|acme:p-2 "1234"^^xsd:integer|acme:p-3 'FOO BAR,BAZ,BAT'"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just noting here that I didnt know "acme:p-1 " Value ZZ"..." would work without escaping the inner " symbols!

@jamesaoverton jamesaoverton merged commit 5eb3ffb into master May 9, 2025
3 checks passed
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.

5 participants