28 locit solver for antenna positions #140
Merged
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
…in dict that contain info on their name are ignored
…till not accessible from user
…a and scipy fits. scipy and linear algebra agree for up to 5 or 6 decimal places!
…nes, scipy or linear algebra
… Now the polarization scheme is inferred from the telescope being used.
…ons for the data fit
…apping for slope term in scipy fits
…is is the first step in making locit capable of combining DDIs in the fit, plotting routines not yet updated
…ing locit capable of combining DDIs in the fit, plotting routines not yet updated
…s is the third step in making locit capable of combining DDIs in the fit, plotting routines not yet updated
…utines not yet updated, error bars in linear algebra fit meaningless
… the position mds
plot_delays, which supports the two possible structures for the position mds
This was referenced Aug 31, 2023
Closed
Closed
…pdate soem incorrect docstrings.
|
Perhaps we should make the default writing plots to disk as opposed to the notebook cell. As you can see below even with the test file we are running into memory issues and the cell actually truncates the output after the first 50 outputs.
A viewer could be built is needs be. Anyways, I think this can be merged tomorrow. So far everything seems to work as I would naively expect. |
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.
LOCIT prototype.
This prototype is capable of fitting the delays to produce antenna position corrections.
Some tests have been done where it achieves similar (but not equal) results to AIPS and is able to recover simulated antenna position shifts.
It still needs several improvements which will be added as issues.