-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(source_removal,beam): updates to holography-based source removal [WIP] #121
base: master
Are you sure you want to change the base?
Conversation
@tristpinsm This PR contains most of the changes that I made to enable proper handling of polarisation in the source subtraction. It also contains a few new tasks that I found necessary to apply to the data prior to stacking over observations. I have a bunch of configuration files that I can share to create stacked holography templates and then fit them to the data. I can also point you to the already existing templates that I have on disk. |
ok, thanks for posting this here! I will try and get the pipeline running myself, so if you could point me to your config files / templates that would be great. |
@tristpinsm The config files can be found here: Let me know if you have any access issues. Unfortunately it appears that the templates were deleted from my scratch space. I'm going to go ahead and re-generate them now, but they probably won't be ready until after the workshop. There is a version of the config files that attempts to subtract a single observation. I think you could use this to get things running if you want to work on this problem now. |
I also just remembered that I had a similar need for a container that is an array of containers in my gated holography pipeline, so I created a generic |
thanks! I'll take a look |
I've been looking through the various config files, and here's what I think I could try running:
However when I started with the first step, I wasn't able to submit a job because the task |
That sounds reasonable. I included ApplyInputFlags in PR #114. I think we can probably merge that if you are happy with it. |
Oh I see. Yeah I don't have any more comments for that PR. |
If subtracting sources from a sidereal stack, then average the source coordinates over sidereal days that went into the sstack at each RA.
To enable output of dictionaries. Also fixes a few other bugs in the source_removal module.
No description provided.