Repository navigation
Blinding: refuse tomography at the producers, check stamps on consumption, test the guarantees - #420
Open
cailmdaley wants to merge 9 commits into
Open
cailmdaley wants to merge 9 commits into
cailmdaley wants to merge 9 commits into
Conversation
…hey were papers/bmodes runs only on public (blind: none) catalogues, so threading a blind through its readers and adding Snakemake adapters is out of scope here. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Pushed by the lint gate.
Contributor
|
🤖 autofix pushed |
This branch has not been deployed
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
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.
Closes #419.
This hardens the non-tomographic blinding from #253 before a catalogue is declared
blind: <name>. The design is unchanged, and onblind: nonecatalogues, which is all of them today, every product is identical.What changes
calculate_pseudo_cl_mapandcalculate_pseudo_cl_catalogrefuse a named blind for integer bin pairs before reading the catalogue. Previously only their orchestrator did. Tomographic sealing waits on Feature/305 adapt tomographic 2pcf script to sacc io format #306 / Tomographic SACC writers: one tracer and n(z) per bin #376.sacc_io.load(path, blind=...)raises if the part's stamp differs from the declared blind, andsacc_io.expectdoes the same for a part held in memory. It is used in the ξ± part reuse, the reporting-part loader, COSEBIs, pure-E/B, the pseudo-Cℓ plots, the B-mode summary and assembly, so a staleblind: nonepart can't feed figures or NPZ under a named blind. The consuming rules passblind=blind_of(version).blind_ofcomparesrealpath, so symlinks and/automnt/n17datavs/n17datacan't put one catalogue under two blinds.test_blinding.py:papers/bmodesreaders are left alone: Paper II runs only on public catalogues.Checks.
test_config_paths_exist×2, HDF5 star-loader tmp cleanup ×2).— Claude (Opus 5.5) on behalf of Cail
🤖 Generated with Claude Code