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

smurf: Fix EXT discontinuities when using threads #31

Open
timj opened this issue Apr 3, 2014 · 0 comments
Open

smurf: Fix EXT discontinuities when using threads #31

timj opened this issue Apr 3, 2014 · 0 comments
Assignees

Comments

@timj
Copy link
Member

timj commented Apr 3, 2014

It has been noted that the EXT model causes slightly different maps to be created by SMURF makemap when the number of threads changes. The discrepancy is of the order of the noise and so it quite noticeable.

I think that this is caused by the code that splits up the data into chunks not splitting on WVM measurement boundaries. The WVM is readout every 1.2 seconds or so and so there can be 200 TCS readings for every WVM reading. The WVM calculation depends on the airmass of the telescope but once calculated should not change as the telescope moves until the next reading is made. If we split the data between threads between readings the tau reading for the start of the next chunk will be different to the reading from the end of the previous chunk despite them being nominally the same reading. The number of threads will therefore control how many of these jumps there are (and the telescope can move quite a long way in a second).

After an estimate for the thread boundaries has been calculated the actually boundary should be finessed by running forward until the WVM measurement changes.

To be pedantic we should really be reading the telescope position for WVM_TIME but at the moment we simply assume that the TCS_AIRMASS reading is appropriate for the associated WVM_* readings despite the WVM_TIME indicating that an earlier telescope reading should be used (by looking at TCS_TAI).

@timj timj added the bug label Apr 3, 2014
@timj timj added the SMURF label Apr 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants