simonsobs / simonsobs/mnms

Possible incorrect power for tiles in last column

Open
#12 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Python
Stars
3
Forks
1
Avg merge
1m
Merged PRs (30d)
1

Description

I believe the linked comment also applies to the highest-ell column of a sample in the case that the real tile size is even.

https://github.com/simonsobs/mnms/blob/b58790f6b57871b95533151b7e564500c2dcf76f/mnms/tiled_noise.py#L536

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in mnms/tiled_noise.py around line 536 and read the linked comment for the related case. Compare the power assigned to the highest-ell column when the real tile size is even, then determine the expected behavior and verify the correction against the existing noise-model workflow.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.