Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Frequency Map (1/2): Query database for frequency slot map #163

Merged
merged 4 commits into from
Mar 16, 2023

Conversation

ljgray
Copy link
Contributor

@ljgray ljgray commented Feb 13, 2023

Query the database for the frequency map that was active when a dataset was collected. Lookup is done by dataset_id, so older datasets which do not have a dataset_id will just use the default mapping that we assume was active at the time.

Changes:

  1. Add a task to query the database for the frequency map corresponding to a dataset_id.
  2. Add a container type to hold a single frequency map. This does not modify the existing FrequencyMap container for backwards compatibility.
  3. Modify the MaskDecorrelatedCylinder task to use the new container
  4. Modify the daily config to use the new task(s)

Requires chime-experiment/ch_util#50

@ljgray ljgray force-pushed the ljg/freq-map-lookup branch 11 times, most recently from bb3c626 to fb42fe3 Compare February 14, 2023 20:38
@ljgray ljgray force-pushed the ljg/freq-map-lookup branch 2 times, most recently from b5f7daf to 8479089 Compare February 27, 2023 21:12
@ljgray ljgray changed the title Query database for frequency slot map Frequency Map (1/2): Query database for frequency slot map Feb 27, 2023
@ljgray ljgray marked this pull request as ready for review February 27, 2023 21:13
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
ch_pipeline/core/dataquery.py Outdated Show resolved Hide resolved
@ljgray ljgray force-pushed the ljg/freq-map-lookup branch 4 times, most recently from 1d021f3 to 5daea2f Compare March 8, 2023 22:04
@ljgray ljgray requested a review from jrs65 March 8, 2023 22:41
@ljgray ljgray merged commit d14edd6 into master Mar 16, 2023
@ljgray ljgray deleted the ljg/freq-map-lookup branch March 16, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants