desihub / desihub/LSS

Minimize columns used by altMTL

Open
#152 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
22
Forks
39
PR merge metrics
No merged PRs in 30d

Description

Notes from the CAI call on 2025-01-16:

Full fiberassign calls inflate_ledger to get additional targeting columns like fluxes to propagate to the offline pipeline, but these aren't needed for the altMTL process. Investigate the minimal required set of columns and whether those can be achieved using just the MTLs without needing inflate_ledger.

From @araichoor in zoom chat:
```
For what is worth, I tried a call with fba_run with only those 18 columns, and it seems to run:
"TARGETID", "RA", "DEC", "REF_EPOCH", "PARALLAX", "PMRA", "PMDEC", "DESI_TARGET",
"BGS_TARGET", "MWS_TARGET", "SUBPRIORITY", "OBSCONDITIONS", "PRIORITY_INIT",
"NUMOBS_INIT", "SCND_TARGET", "NUMOBS_MORE", "NUMOBS", "PRIORITY”;
it s very likely that those are in the ledgers.
```

To which @geordie666 replied
```
Yep, those 18 columns are all in the ledgers already!
```

@LNapolitano @ashleyjross heads up

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.