Skip to content

Deterministic SQL result ordering#109

Open
jbcoe wants to merge 3 commits intoNetHack-LE:mainfrom
jbcoe:jbcoe/sort-sql-results
Open

Deterministic SQL result ordering#109
jbcoe wants to merge 3 commits intoNetHack-LE:mainfrom
jbcoe:jbcoe/sort-sql-results

Conversation

@jbcoe
Copy link
Collaborator

@jbcoe jbcoe commented Mar 21, 2026

Added explicit ORDER BY .gameid to SQL queries to ensure deterministic ordering.

Possibly closes #88

jbcoe added 3 commits March 20, 2026 20:05
This reverts commit 0b96483.

Fast rebuilds with uv and scikit-build and use of Python tooling
within a CMake build are incompatible. The isolated environment
used for the build means that the Python executable cached in the
build directory is not valid when the build directory is used for
a rebuild.
@StephenOman
Copy link
Collaborator

I like this idea - let's hope it fixes the issue 🤞

@StephenOman StephenOman added the bug Something isn't working label Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test_dataset failing intermittently

2 participants