[AUTOTUNER] Use hosted default FlagTune Manifest URL - #1103
Open
k6699k wants to merge 38 commits into
Open
Conversation
Reorganize FlagTune into contract, runtime, and training namespaces; update public imports and tests to use the new boundaries; and remove the obsolete migration module. Introduce an architecture-neutral event/replay benchmark protocol with complete cache identity and audit metadata. Extend driver capabilities so NVIDIA and AMD backends can expose replay timing without leaking device-specific APIs into callers. Update the FlagTune autotuner to consume resolved benchmarkers, preserve event-cache compatibility, and add unit coverage for protocol validation, replay fallback, and backend capability behavior.
Apply the Code-Format-Check pre-commit output to FlagTune contracts, runtime, training, and their focused tests without changing behavior. The resulting files pass the configured Ruff, YAPF, clang-format applicability, and mypy checks.
Replace gpu_key identity with platform_key while keeping architecture as device metadata. Add flat versioned platform packages, model migration without retraining, loading, caching, validation, and tests.
A platform package covers only the operators, variants, and dtype combinations it was built for, so a package that resolves but carries no bundle for one identity is a statement about coverage rather than a fault. Integration layers need to tell that apart to fall back, but it previously raised the same IncompatibleModelError as a corrupt archive, an incompatible FlagTune version, and an identity mismatch. ModelBundleMissingError is raised only for that case and subclasses IncompatibleModelError, so existing handlers keep working. Also name the cause when download-latest narrowed a cache lookup: asking for the Manifest's newest package while forbidding downloads reports only that this version is uncached, which sends the reader after the network when an older cached package was simply not eligible. Behaviour is unchanged; that pair remains a supported offline configuration.
k6699k
requested review from
Galaxy1458,
menchunlei,
sunnycase and
zhzhcookie
as code owners
September 4, 2026 12:45
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
FLAGTUNE_MANIFEST_URL.FLAGTUNE_MANIFEST_URLvalues as overrides.