Thijs

Results 416 comments of Thijs

I tried replacing it with `locals()` and `globals()` and it made no difference Those probably just call `currentframe().f_locals` under the hood anyways The created dictionary/proxymapping is supposedly stored in the...

Thanks for the report, this is unfortunately an expected limitation currently, because we don't have dependencies on columns, only tables. Since the table is referenced by the index, it can't...

Ah, looks like https://github.com/pybind/pybind11/pull/3402 fixes this if I read it correctly 👍

I think actually the problem is that `py::arg(...)` should not be provided if the argument is `py::args` Which makes some sense, but since we're making a definition, and for python...

> @Mytherin sure, I'm looking into the Thread Sanitizer errors. Looks like they appeared because I used std::ostringstream at the same time with the std::cerr. I think the ThreadSanitizer output...

> > `GetSummary` should be a class, which has a mutex, and we need to lock that mutex whenever we write to the summary. > > Thank you! This makes...

Next issue: https://github.com/duckdb/duckdb-wasm/pull/2016#issuecomment-2888968820

Next issue: ``` yarn yarn install v1.22.22 [1/4] 🔍 Resolving packages... success Already up-to-date. ✨ Done in 0.14s. yarn install yarn install v1.22.22 [1/4] 🔍 Resolving packages... success Already up-to-date....

What does exist in that directory is: `packages/duckdb-wasm/src/bindings/duckdb-mvp.d.ts` Not `packages/duckdb-wasm/src/bindings/duckdb-mvp.js` Contents of that file: ```ts import { DuckDBModule } from './duckdb_module'; export function DuckDB(moduleOverrides?: Partial): Promise; export default DuckDB; ```...

Even running `make wasm_star` before running `make serve` did not help to resolve the issue that errors on: ```js (async () => { // Don't attempt to bundle NodeJS modules...