Skip to content

[DuckDBPyType] Add __hash__ method overload#61

Merged
evertlammerts merged 5 commits into
duckdb:v1.4-andiumfrom
Tishj:pytype_hash
Sep 18, 2025
Merged

[DuckDBPyType] Add __hash__ method overload#61
evertlammerts merged 5 commits into
duckdb:v1.4-andiumfrom
Tishj:pytype_hash

Conversation

@Tishj

@Tishj Tishj commented Sep 17, 2025

Copy link
Copy Markdown
Member

This PR fixes #78

@Tishj Tishj changed the base branch from main to v1.4-andium September 17, 2025 08:15
@Tishj

Tishj commented Sep 17, 2025

Copy link
Copy Markdown
Member Author

I think we stumbled into another windows define..

    python_conversion.cpp
  D:\a\duckdb-python\duckdb-python\src\duckdb_py\include\duckdb_python\pytype.hpp(33,10): error C3646: 'Hash': unknown override specifier [C:\Users\RUNNER~1\AppData\Local\Temp\tmpfoywn8np\build\src\duckdb_py\native\python_native.vcxproj]
    (compiling source file 'D:/a/duckdb-python/duckdb-python/src/duckdb_py/native/python_conversion.cpp')

@evertlammerts evertlammerts added this pull request to the merge queue Sep 18, 2025
Merged via the queue into duckdb:v1.4-andium with commit 134b784 Sep 18, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[1.4.0] TypeError: unhashable type: _duckdb.typing.DuckDBPyType

2 participants