10:57 <gbruno> [github] wnc2 opened issue #5259 (Crash in opening on Mac Sequoia Beta 15.0 24A5320a) https://github.com/openscad/openscad/issues/5259

11:00 <gbruno> [github] wnc2 edited issue #5259 (Crash in opening on Mac Sequoia Beta 15.0 24A5320a) https://github.com/openscad/openscad/issues/5259

11:30 <InPhase> teepee: Looks like python is being used. Can we get indent=2 added to the json output?

11:31 <InPhase> teepee: I see it has one of those wild one-liner json outputs. :) indent=2 is the magic that makes them more nicely formatted.

11:32 <InPhase> The github display of the file will get much nicer following that.

11:32 <teepee> yeah, I've seen that. I guess that makes sense

11:32 <teepee> the minimal file size increase should not make a huge difference

11:33 <InPhase> Yep.

11:33 <InPhase> Human readability is worth it.

11:34 <InPhase> Especially as this gets much larger, as expected.

11:34 <teepee> so far I have not managed to get the client packaged, I guess I have to learn poetry a bit

11:35 <teepee> or reconsider the idea to start out with snap

11:38 <InPhase> Ah, so it requires a local python install?

11:38 <InPhase> For the user of libraries?

11:39 <teepee> in the first instance yes, either that or the snap package

11:40 <InPhase> Is snap even cross-platform?

11:41 <teepee> partially as it should work in WSL2

11:41 <teepee> the mid-term solution is probably some sort of more general python packaging

11:41 <InPhase> I recall hearing some distros have actively rejected it.

11:42 <teepee> long term I think a c++ library for a standalone binary + included in openscad gui would be the way to go

11:42 <teepee> well, then those people many need to have python installed or help getting things packaged :)

11:42 <teepee> s/many/may/

11:42 <InPhase> Well I guess every Linux user already has python anyway. :)

11:43 <InPhase> I suppose the basic requirements of client side are download of a file and json parse?

11:43 <teepee> not as trivial anymore, at least debian now forces a venv when using pip

11:43 <InPhase> And then I suppose grabbing all the library files.

11:44 <teepee> to get the list of libraries and/or search, yes

11:44 <teepee> the client also does the download and version matching for dependencies

11:44 <teepee> no complex dependencies though ;-)

11:45 <InPhase> nlohmann does json handling nicely for C++, although implementing the grab of all the library files would probably be the more annoying bit.

11:45 <teepee> it should not be hugely difficult as the version stuff is using libversion which is C already

11:46 <teepee> and download should be ok-ish via boost or curl

11:46 <InPhase> I personally do not object to the use of Python for this, and had advocated for that in the past. It just does leave that initial install problem to solve, as you say.

11:47 <teepee> as we would like the other python stuff anyway, maybe we just leave it as python

11:47 <InPhase> If we're ever to do that python branch merge, then it's a two-birds problem.

11:48 <teepee> I'm still very much for merging, it's really just a time problem for me

11:49 <InPhase> And afterwards, one could potentially go back and rethink how I had done that python testing infrastructure. Depending on the nature of the final solution this could be integrated as well. But it's fine if that stays separate, since devs can always be expected to be able to get python.

11:51 <teepee> yeah, for the dev stuff the setup could be separate

11:51 <teepee> but if it helps with the integration, why not

11:51 <InPhase> Yep.

11:52 <InPhase> Removing a step would be a nice free bonus. But the dependencies of that testing infrastructure are pretty light.

11:52 <InPhase> It's particularly unconcerned with versioning.

11:53 <InPhase> Assuming no one wildly alters basic library functioning on image loading/saving and numpy math. :)

11:54 <teepee> uh, I hope not, that sounds scary

11:55 <InPhase> I'm on guard after that random numpy deprecation of np.float and np.int. It is the python way to eventually break things that seem unlikely to break. But it should be fairly rare given the usages.

21:41 <teepee> aha! we have a snap package :)

21:55 <teepee> well, mixed signals... https://imgur.com/pADNmyC :-)

22:17 <teepee> wtf, ubuntu!

23:16 <teepee> and finally it's "released" to the snap store too