simonsobs / simonsobs/sodetlib

Make `frac_pp` more persistent

Open
#484 2 comments 0 reactions 1 assignee View on GitHub

@tristpinsm is already working on this.

Since Dec 5, 2025.

Dominant language
Python
Stars
5
Forks
0
Avg merge
3d 17h
Merged PRs (30d)
4

Description

Occasionally setup_tracking_params will fail to converge for some bands (see plot below). The cause is often that the value of frac_pp it found is off, and the solution is to inspect the frac_pp for good bands and manually set it to their mean for the bad ones. Although the underlying pysmurf algorithm could be made more reliable, this is a parameter that changes very little and having it be set dynamically is unnecessary.

Max in Slack:

Ya this was brought up by Suzanne, basically it doesn't change unless something goes wrong (like you develop a short). She was requesting we split the dev_cfgs and don't allow such long lived parameters to be overwritten unless done so by hand in a git controlled way or something

I think this configuration is already possible by setting fraction_full_scale in the pysmurf config, which is tracked in the simonsobs/ocs-deployment-configs. Then we would need to make the sodetlib optionally not update that parameter (and not track it in the device config). We'd need to think a bit about operationally how this would get set.

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.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.