Thorsten Beier comments

Results 183 comments of


                                            Thorsten Beier

Added (currently) failing test for correctness

I suggest the following: instead of these shitty calls: ``` volume[z*xyWidth + y*xWidth + x] < isoLevel ``` we use `xtensor` / `pyxtensor` and use sane calls like ``` xtensor_volume(x,y,z)...

Would you like to modify the boost::container::flat_set<size_t> to std::vector?

@stuarteberg years ago I wrote flat set on my own for OpenGM which was conserved / polished by @bjoern-andres https://github.com/bjoern-andres/random-access-set/blob/master/include/andres/random-access-set.hxx We might want to use that

Licensing Issues

any chance to chance the license to MIT. This would be really helpful

Package recipe maintainers / project contacts

consent

Parent header

@JohanMabille do you want to have a look before I continue? I might change the order of the arguments in the functions where additional arguments where introduced.

Should it be possible to `import ssl`?

> > File /lib/python3.10/ssl.py:98 Not sure, but this is on a very old stack, and the python3.11 build has some more features.

Should it be possible to `import ssl`?

we need to port this recipe to emscripten-forge https://github.com/pyodide/pyodide/blob/main/packages/ssl/meta.yaml

'emscripten' is not a valid Platform

> Didn't we patch conda in some way to allow emscripten installation? that was just for building packages with boa and conda-build. When using conda to **install** emscripten-wasm32 package we...

Using an undefined variable in the REPL will silently initialize it to 1

This issue makes it really hard to use wren in a REPL session. In particular using wren from jupyter suffers a lot from this issue. feel free to use wren...

Test on `webkit`

> cc @DerThorsten who I think has a Mac and might be able to try using the xeus python kernel in Safari directly, just to double check. on Version 16.5...