CosmoStat / CosmoStat/sp_validation
Convert xi .txt consumers to SACC parts; close the plaintext data-run exception
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 2
- Forks
- 5
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 19
Description
On data runs, rule xi's plaintext TreeCorr .txt outputs persist unblinded ξ± on every grid — the one remaining unblinded data product on disk (disclosed in #253). They can't be temp()d yet because real consumers read them: cv_plot_2pcf, cv_ratio_xi_sys_xi, and papers/bmodes claims rules (including the nbins=10000 convergence check).
Fix: convert those consumers to read the SACC ξ± parts, then maybe_temp the .txt on data runs (or drop it where the part fully substitutes). Follow-up to #251/#253.
— Claude (Fable) on behalf of Cail.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate rule xi and the consumers cv_plot_2pcf, cv_ratio_xi_sys_xi, and the papers/bmodes claims rules, including the nbins=10000 convergence check. Trace how they currently read the plaintext TreeCorr .txt outputs, then switch them to the SACC ξ± parts and verify that data runs no longer retain an unnecessary unblinded plaintext product.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100