Fork me on GitHub
Eric Scott14:02:14

Wow @kelvin063 this is great!


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.

Eric Scott14:02:30

I noticed that you're using # inst. The ont-app/vocabulary module defines an # lstr reader macro. I know @U4P4NREBY 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?


@UB3R8UYA1 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.

Eric Scott13:02:53

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


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

Eric Scott13:02:46

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


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

Eric Scott13:02:46

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

Eric Scott16:02:27 v. 0.1.6 is now committed! Thanks Simon!

👍 1
Eric Scott16:02:04

Strangely (s) is not supported in cljs


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


In that case, let me make another PR


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.


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


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

Eric Scott17:02:49

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

🙏 1

No rush 😄

Eric Scott15:02:54

Done. Thanks again!

👍 1