simonsobs / simonsobs/SOOPERCOOL

Simplify geometry inference

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

Nobody has claimed this yet.

Dominant language
Jupyter Notebook
Stars
6
Forks
5
Avg merge
5d 22h
Merged PRs (30d)
2

Description

We should improve user friendliness with regards to geometry selection from maps. This includes but is not restricted to these two tasks:

  1. Remove car_template in the yaml and directly read the geometry from the maps.
  2. Get rid of this patch and similar ad-hoc solutions in other scripts.

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 by reading pipeline/compute_pseudo_cells.py around the linked patch and inspect the YAML configuration that defines car_template. Search the other pipeline scripts for similar geometry workarounds and determine how maps expose their geometry. Done means geometry is inferred directly from maps, car_template is removed, and the ad-hoc patches are no longer needed.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-engineering
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.