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

Create an all-inclusive example unit test for ground-based data. #796

Merged
merged 4 commits into from
Nov 15, 2024

Conversation

tskisner
Copy link
Member

This work aims to solve two problems:

  • Lack of an example unit test for ground telescope data which can be used as a starting point (by removing various things) for future unit tests.

  • Inconsistent techniques for "scanning from a map" across many unit tests. Basically I have removed ad-hoc code for generating fake sky signal and scanning into timestreams and centralized that in several unit test helper functions.

This work aims to solve two problems:

- Lack of an example unit test for ground telescope data which
  can be used as a starting point (by removing various things)
  for future unit tests.

- Inconsistent techniques for "scanning from a map" across many
  unit tests.  Basically I have removed ad-hoc code for generating
  fake sky signal and scanning into timestreams and centralized
  that in several unit test helper functions.
@tskisner tskisner marked this pull request as ready for review November 1, 2024 14:31
Copy link
Collaborator

@AERAdler AERAdler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made comments on ops_example_ground and the commented out section of helpers. I tried running some of the tests and it looks to me like everything is in order.

@tskisner tskisner merged commit 1985fff into toast3 Nov 15, 2024
7 checks passed
@tskisner tskisner deleted the example_unittest branch November 15, 2024 19:25
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