Accommodate per-bias-group shunt resistance and bias_line_resistance

Open
  • Is something wrong?
#69 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
python
Domain
data, testing-qa

Research direction

Start by locating functions that use shunt resistance, including IV-curve handling, and trace how band/channel pairings are represented. Review how routing-wafer measurements from Cornell could map values to bias groups; done means all relevant functions accept per-group resistance values and IV curves identify the applicable bias group.

Written by the indexing model from the issue text.

Description

mustfix

Any function that requires using the shunt resistance should take in a value for each separate bias group, which can be pulled from measurements done on the routing wafers at Cornell. This includes but is not limited to IV curves, which need a way to be clear which bias group a given (band,ch) pairing is on.

Dominant language
Python
Stars
5
Forks
0
No merged PRs in 30d

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

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.

More from simonsobs/sodetlib

All issues in simonsobs/sodetlib

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.