rdf

2022-02-18T14:35:14.176859Z

Wow @kelvin063 this is great!

2022-02-21T15:24:54.163239Z

Done. Thanks again!

👍 1
Kelvin 2022-02-18T19:36:16.068089Z

Thank you! Indeed, IGraph has been an inspiration for Flint: • It has inspired some of Flint's syntactic conventions, namely the triple normal form map. • More broadly, Flint and IGraph both share the idea of being a Clojure data layer on top of RDF/SPARQL.

2022-02-19T14:33:30.344189Z

I noticed that you're using # inst. The ont-app/vocabulary module defines an # lstr reader macro. I know @simongray has also found that useful. =ont-app/vocabulary has a bunch of other stuff that might be a bit more weight than you'd like to take on. Would it be of use to you if I broke # lstr into its own project?

simongray 2022-02-20T13:18:07.530219Z

@eric.d.scott Yeah, ont-app/vocabulary is really nice, as are your other RDF libs. Speaking of… I just submitted a PR to ont-app/vocabulary with a tiny fix to a bug I found.

2022-02-20T13:38:25.716569Z

Oh. Thanks!

2022-02-20T13:40:53.071879Z

It had not occurred to me that a lstr would have \n's in them.

simongray 2022-02-20T13:41:28.612379Z

me neither, but I’ve found it in both ontolex and vann so far

2022-02-20T13:42:46.967019Z

It does seem that # lstr could live independently of the rest of ont-app/vocabulary, yeah?

simongray 2022-02-20T13:44:22.791359Z

I guess it’s a trade-off. I do like mini-libs, but they can also get too small sometimes.

2022-02-20T13:57:46.422889Z

Yeah, maybe I'll let that season for a bit.

2022-02-20T16:08:27.048319Z

https://github.com/ont-app/vocabulary v. 0.1.6 is now committed! Thanks Simon!

👍 1
2022-02-20T16:09:04.349689Z

Strangely (s) is not supported in cljs

simongray 2022-02-20T16:39:48.748749Z

it isn’t? hm… that doesn’t solve my issue then… :S

simongray 2022-02-20T16:43:16.903339Z

In that case, let me make another PR

simongray 2022-02-20T16:44:44.934799Z

it is an easily solvable problem after all. Initially, I just made a non-capturing group of (?:.|\s), but then I decided to use the flag instead as it is the “cleaner” solution.

simongray 2022-02-20T16:45:27.066569Z

perhaps you could put (?:.|\s)instead of . in the CLJS regex?

simongray 2022-02-20T16:46:19.126369Z

that would match any character as well as any whitespace, so essentially the same as a DOTALL .

simongray 2022-02-20T16:52:40.371199Z

This should fix it https://github.com/ont-app/vocabulary/pull/17

2022-02-20T17:11:49.360349Z

Awesome! Thanks! Clearly your regex-fu is top-notch. I'll incorporate this when I get back to my desk.

🙏 1
simongray 2022-02-20T18:28:06.232149Z

No rush 😄