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.

Sign up for GitHub

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

Jump to bottom

Execution Block ID uid://A001/X15a0/X16e Sgr_A_st_ai_03_7M #143

Open
14 of 15 tasks
keflavich opened this issue May 20, 2022 · 9 comments
Open
14 of 15 tasks

Execution Block ID uid://A001/X15a0/X16e Sgr_A_st_ai_03_7M #143

keflavich opened this issue May 20, 2022 · 9 comments
Assignees
@d-l-walker
Labels
7m Delivered Done! EB Execution Block

Comments

@keflavich
Copy link
Contributor

keflavich commented May 20, 2022 •
edited
Loading

Sgr_A_st_ai_03_7M
uid://A001/X15a0/X16e

Product Links:

Reprocessed Product Links:
@keflavich keflavich added 7m EB Execution Block labels May 20, 2022
@d-l-walker
Copy link
Contributor

d-l-walker commented Jun 1, 2022 •
edited
Loading

Summary

Red warning about data flagged in one EB:

  • QA 45.35% data in uid___A002_Xf934b1_X127c0.ms flagged by online, shadow, qa0, qa2, before and template flagging agents
  • I don't think this is a major issue. It's only 1/6 EBs, and the final sensitivity looks OK.
  • No significant issues otherwise prior to imaging stage.

Cubes

SPW 24 diverged and will need to be recleaned using cyclefactor/cycniter parameter. @d-l-walker will do this.
uid___A001_X15a0_X16e s38_0 Sgr_A_star_sci spw24 cube I iter1 image pbcor sky

The continuum channel selection in SPW 20 is ... questionable. It clearly includes significant line emission, which is evident in the 'line-free' moment 8 map. Since I'll be reprocessing this data for SPW 24, I may as well re-do SPW 20 and the continuum, using a more suitable channel range for SPW 20.

uid___A001_X15a0_X16e s38_0 Sgr_A_star_sci spw20 cube I iter1 image spectrum

uid___A001_X15a0_X16e s38_0 Sgr_A_star_sci spw20 cube I iter1 image mom8_fc sky

@d-l-walker d-l-walker self-assigned this Jun 1, 2022
@d-l-walker d-l-walker added the Needs Reimaging: Divergence Needs reimaging b/c of divergence label Jun 1, 2022
@d-l-walker
Copy link
Contributor

d-l-walker commented Aug 3, 2022 •
edited
Loading

I've finished the re-imaging and uploaded the products to /upload/Sgr_A_st_ai_03_7M_reclean/ @keflavich (target destination is /rawdata/2021.1.00172.L/science_goal.uid___A001_X1590_X30a8/group.uid___A001_X1590_X30a9/member.uid___A001_X15a0_X16e/)

There were only two real changes -- I simply re-ran the PL steps for imaging, with a refined cont.dat file to minimise the line contamination in the continuum selection for SPW 20. The continuum + SPWs 20 & 24 were imaged, and cyclefactor=1.5 was used to avoid the divergence in SPW 24. All products look good, so I'm setting this as 'Done'.

@keflavich
Copy link
Contributor Author

keflavich commented Aug 3, 2022 •
edited
Loading

Todo:

  • add continuum selection spw command to override_tclean_parameters.json
  • add cyclefactor to same
  • add cont.dat to (new) continuum_selection folder

@d-l-walker
Copy link
Contributor

@keflavich to address your comment ...

  • I actually didn't use the spw parameter to clean the continuum. I simply re-ran the PL imaging steps with the new cont.dat file in the directory and let the PL do the rest.
  • I updated the cyclefactor params for SPWs 20 & 24 via PR Adding parameters for Sgr_A_st_ai_03_7M #143 #207.
  • I noticed that the /~/upload/Sgr_A_st_ai_03_7M_reclean/ folder is still present, so I just added the new cont.dat file there. I'm not sure where the new continuum_selection folder is? So just thought I'd chuck it in the relevant upload folder for now.

d-l-walker added a commit that referenced this issue Aug 23, 2022
@d-l-walker
Merge pull request #207 from d-l-walker/update_tclean_params
a366725 
Adding parameters for Sgr_A_st_ai_03_7M #143. Checks looks good, no obvious syntax issues. Merging.
@keflavich
Copy link
Contributor Author

TODO: check whether cont.dat file is in the right place to be used if we re-run everything

@keflavich keflavich removed the Needs Reimaging: Divergence Needs reimaging b/c of divergence label Feb 15, 2023
@eacmills eacmills mentioned this issue Mar 23, 2023
Closed
@pyhsiehATalma
Copy link
Contributor

@d-l-walker @keflavich

Thanks for reclean spw 24. I downloaded spw 24 cube under /working for CS2-1 feathering and found the cube still shows divergence at ch 602. Perhaps the cube is not updated correctly after reclean?

X15a0_X16e_ch602

@d-l-walker
Copy link
Contributor

d-l-walker commented May 3, 2023 •
edited
Loading

Hi @pyhsiehATalma,

Could you grab the cube from /rawdata/2021.1.00172.L/science_goal.uid___A001_X1590_X30a8/group.uid___A001_X1590_X30a9/member.uid___A001_X15a0_X16e/Sgr_A_st_ai_03_7M_reclean/uid___A001_X1590_X30a8.s3_0.Sgr_A_star_sci.spw24.cube.I.iter1.reclean.image.fits

I think the one that you used was the old cube.

@d-l-walker
Copy link
Contributor

I've confirmed that channel 602 looks OK in the recleaned version. Could you try again using the other cube @pyhsiehATalma?

Screenshot 2023-05-11 at 15 00 00

@pyhsiehATalma
Copy link
Contributor

Hi @pyhsiehATalma,

Could you grab the cube from /rawdata/2021.1.00172.L/science_goal.uid___A001_X1590_X30a8/group.uid___A001_X1590_X30a9/member.uid___A001_X15a0_X16e/Sgr_A_st_ai_03_7M_reclean/uid___A001_X1590_X30a8.s3_0.Sgr_A_star_sci.spw24.cube.I.iter1.reclean.image.fits

I think the one that you used was the old cube.

Hi @d-l-walker thanks for the re-imaging of spw 24. The map looks pretty good.
I just found that the file names under /Sgr_A_st_ai_03_7M_reclean should be replaced to "...uid___A001_X15a0_X16e...."

@pyhsiehATalma pyhsiehATalma mentioned this issue Jul 4, 2023
Open
@pyhsiehATalma pyhsiehATalma mentioned this issue Jan 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@d-l-walker d-l-walker

Labels
7m Delivered Done! EB Execution Block
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@keflavich @d-l-walker @pyhsiehATalma