Wow @kelvin063 this is great!
Done. Thanks again!
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.
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?
@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.
Oh. Thanks!
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
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.
Yeah, maybe I'll let that season for a bit.
https://github.com/ont-app/vocabulary v. 0.1.6 is now committed! Thanks Simon!
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 .
This should fix it https://github.com/ont-app/vocabulary/pull/17
Awesome! Thanks! Clearly your regex-fu is top-notch. I'll incorporate this when I get back to my desk.
No rush 😄