Keep main GPU dev branch updated to main deployed branch #51
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed the actual start time of the data processed by calculating offset between full observation start time and the GPS time of first sample processed. This occurs within the geometry calculations and associated structs, thus both VDIF and PSRFITS are corrected with this one fix.
Co-authored-by: Chia Min Tan <[email protected]>
* changed calibration read for Offringa to use requested number of channels to process * fix GPU memory address alignment errors, add comments/debug code * included extra debug info in Offringa-style solution reads * fix C-standard issue in DEBUG block * added debug output * add debug output * update variable names to match fix from PR #43, removed commented statements * fixed mildly pedantic int vs. unsigned int string formatting warning * fixed the formatting for real this time...
* changed calibration read for Offringa to use requested number of channels to process * fix GPU memory address alignment errors, add comments/debug code * included extra debug info in Offringa-style solution reads * ad-hoc fix to time stamp and calibration reader * Remove innocuous stdout messages * Remove innocuous stdout messages * re-fix printed types * Remove innocuous stdout messages --------- Co-authored-by: Chia Min Tan <[email protected]>
Only relevant if compiling with the `-DDEBUG` statement.
* Updated function names to compile under mwa_hyperbeam 0.7 * Make runnable with latest DSPSR and small GPUs - Re-added 'nchunks' option to divide up the data into chunks to fit onto gpu - Added NPOL to VDIF header * Fixed VDIF output bug - The calculation of file_no (to decide which half of the buffer to fill) had been moved to a spot after one of the dependent values had changed. The way this is calculated has now been changed. * Reverted to Hyperbeam <0.7.0 function calls to compile on Garrawarla * Added more info to the VDIF ASCII header file * Flattened the detected_beam buffer, and renamed it to data_buffer_fine to be less confusing * Removed UTC_START sfrom ASCII header s that dspsr will run * Removed whitespace * Changed length modifier on size_t string formatter --------- Co-authored-by: Sam McSweeney <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the title suggests, and given at this moment we're embarking on porting the code to be GPU-agnostic via a Pawsey Uptake project, we want to start from the best working version we have.