Skip to content

db: include backend kwargs in results metadata - #145

Merged
guillon merged 1 commit into
xtc-tools:mainfrom
guillon:dev/cguillon/db-backend-kwargs
Sep 17, 2026
Merged

guillon merged 1 commit into
xtc-tools:mainfrom
guillon:dev/cguillon/db-backend-kwargs

Conversation

@guillon

@guillon guillon commented Sep 17, 2026

Copy link
Copy Markdown
Member

Motivation

Backend-specific options can affect generated code and performance, but were not recorded in the exploration results database. This could make results produced with different backend configurations indistinguishable.

Description

Update the results database format to v0.3 and include backend keyword arguments in compiler metadata.

Exploration configuration now maintains backend-specific keyword arguments, uses them when constructing backend implementations, and forwards them to the database callback. The database query interface accepts the same arguments when matching results.

Commits

  • db: update json db to format 0.3 with backend kwargs

Discussion

The database version is bumped from v0.2 to v0.3 because compiler metadata now includes backend keyword arguments.

Testing: Not run.

@guillon guillon self-assigned this Sep 17, 2026
@guillon guillon added the bug Something isn't working label Sep 17, 2026
@guillon
guillon enabled auto-merge September 17, 2026 15:13
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.92308% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/xtc/search/explore.py 66.66% 1 Missing and 1 partial ⚠️
src/xtc/cli/query_results.py 75.00% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@guillon
guillon merged commit ee5362f into xtc-tools:main Sep 17, 2026
9 checks passed
@guillon
guillon deleted the dev/cguillon/db-backend-kwargs branch September 17, 2026 15:32
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.

1 participant