search

simonsobs / sotodlib

Simons Observatory: Time-Ordered Data processing library.
MIT License
16 stars 19 forks source link

preproc_or_load_group in core #1025

Open chervias opened 3 hours ago

chervias commented 3 hours ago

This function currently lives in the site_pipeline section, but probably is better to be in the library side, for example should be in core next to get_obs

msilvafe commented 2 hours ago

Ya, thanks for creating an issue. @mmccrackan, and I have been speaking about the number of functions in site_pipeline that are being called outside of just the main script. I agree we need to move things out, we can discuss where the best place to do that is. We had moved some of the functions into site_pipeline/util.py however do you think that it's better that this goes into preprocess/common.py or preprocess/util.py. @mhasself since you raised this in @chervias's PR review perhaps you have an opinion?