simonsobs/sotodlib

Splits in make_map does the opposite of what is expected

Open

#1.141 geöffnet am 8. März 2025

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (23 Forks)auto 404
good first issue

Repository-Metriken

Stars
 (17 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I was debugging why for single detector maps setting detector weights to isolate one det shows a different map than using data_splits. @skhrg pointed out that this section of the code does the opposite of what is expected: https://github.com/simonsobs/sotodlib/blob/11ed2ac345ef0a007daef9518007c576ac0ca4eb/sotodlib/coords/planets.py#L744-L759

I.e. it is cutting (group_cuts) samples marked in splits, while we would expect those to be kept and the rest cut.

Can be fixed by just adding an inversion, but unclear if this is how the code is intended to be used and I am just misunderstanding splits vs cuts. Thanks!

Contributor Guide

Research-Richtung
Untersuche die Logik von `make map` im Abschnitt zur Split Verarbeitung (Zeilen 744-759) von `sotodlib/coords/planets.py`. Der aktuelle Code schneidet Samples, die als Splits markiert sind, obwohl sie behalten werden sollten. Implementiere eine Inversion, um das Verhalten zu korrigieren, aber überprüfe auch, ob die beabsichtigte Verwendung das Schneiden oder Behalten von Splits ist. Teste die Korrektur mit einer Einzeldetektor Karte, um das erwartete Verhalten zu bestätigen.
Tech Stack
python
Domain
data
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75