refactor!: move sensitivity analysis to fasterrecipes (BREAKING)#42
Open
nathanhubens wants to merge 1 commit into
Open
refactor!: move sensitivity analysis to fasterrecipes (BREAKING)#42nathanhubens wants to merge 1 commit into
nathanhubens wants to merge 1 commit into
Conversation
nathanhubens
force-pushed
the
feature/sensitivity-migration
branch
2 times, most recently
from
July 1, 2026 15:52
f9932bb to
438ca68
Compare
analyze_sensitivity, SensitivityAnalyzer, SensitivityResult, LayerSensitivity moved out of fasterai into the higher-level FasterAI workflow package (decisions and allocation logic live there; fasterai holds compression mechanisms only). fasterai.analyze.sensitivity now raises a helpful ImportError via a PEP-562 __getattr__ stub. _modidx pruned; quarto sidebar entries and the sensitivity tutorial removed. Validated: nbdev-test stub green; import fasterai OK; stub raises with guidance.
nathanhubens
force-pushed
the
feature/sensitivity-migration
branch
from
July 2, 2026 10:06
438ca68 to
4974e75
Compare
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
1 participant
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
BREAKING. Removes the sensitivity-analysis code from fasterai. Per the architecture split, fasterai holds the compression mechanisms; the sensitivity decision/allocation logic now lives in the higher-level workflow package.
Removed from
fasterai.analyze.sensitivity:analyze_sensitivity,SensitivityAnalyzer,SensitivityResult(incl.to_layer_targets),LayerSensitivityfasterai.analyze.sensitivityis now a PEP-562 stub — importing any of the moved names raises a helpfulImportErrorwith the new import path. fasterai gains no new dependency (it does not import the downstream package).Validation
ImportError; no downstream import, so it passes in CI)._quartosidebar entries + the moved sensitivity tutorial removed.