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
Mss 0001 #290
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good. Note that the trivial whitespace changes in C++ code are due to a version upgrade of the uncrustify tool. Some default spacing option changed between 0.66 and 0.68, but we confirmed that the changes in this branch are made with the newer version, so fine to merge that update.
examples/templates/ground.in
Outdated
@@ -122,6 +122,9 @@ export TOAST_NUM_THREADS=${OMP_NUM_THREADS} | |||
echo "OpenMP # threads: ${OMP_NUM_THREADS}" | |||
echo "TOAST # threads: ${TOAST_NUM_THREADS}" | |||
|
|||
export NUMBA_THREADING_LAYER=tbb | |||
export NUMBA_NUM_THREADS=${OMP_NUM_THREADS} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These numba things are no longer needed. They are set automatically on the first import of the toast module:
https://github.com/hpc4cmb/toast/blob/master/src/toast/mpi.py#L40
@@ -955,6 +991,68 @@ def write_common_flags(self, local_start=0, flags=None, **kwargs): | |||
self._put_common_flags(local_start, flags, **kwargs) | |||
return | |||
|
|||
@function_timer | |||
def read_hwp_angle(self, local_start=0, n=0, **kwargs): | |||
"""Read half-wave plate angle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for moving the HWP motion into the TOD class where it belongs.
Merging after @tskisner fixed the Travis test setup. |
Several fixes arising from the mss-0001 simulation effort.