Skip to content

DIRBE DR2.1 run code#224

Open
unfunfunt wants to merge 221 commits intomasterfrom
precond
Open

DIRBE DR2.1 run code#224
unfunfunt wants to merge 221 commits intomasterfrom
precond

Conversation

@unfunfunt
Copy link
Collaborator

Pushing the version of the code that ran DR2.1 to master

unfunfunt and others added 30 commits June 9, 2022 13:50
@unfunfunt unfunfunt requested a review from dncnwtts January 22, 2026 10:45
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we remove this?

@@ -1045,6 +1094,18 @@ subroutine read_star_catalogue(self, cpar, id, id_abs)
!load in the beam information
call init_beam_templates(self, cpar, id, id_abs)

! Update mixing matrix -- HKE: GAIN IS MISSING
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another thing that could be deleted

write(58,*)
do i = 1, size(s_tot,1)
write(58,*) i, tod_arr(i, 1)
open(58,file='gain_delta_'//itext//'.dat', recl=1024)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This really should be deleted, since it will save a file to the write directory

end do

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another thing we need to delete

binmap%A_map(off+3,pix_) = binmap%A_map(off+3,pix_) + tod%sin2psi(psi_) * inv_sigmasq
end if

! if(pix_ == 1 .and. det==2) then
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this kind of comment still?

ncut = 0
if (output_scan == self%scanid(scan)) open(58, file='flag_stage2.dat')
do iter = 1, 1
!!$ ! Compute full-scan, masked rms0
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another set of comments to remove

Copy link
Collaborator

@dncnwtts dncnwtts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks good. Some comments could be removed (see what I wrote before)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants