Skip to content

Reserve memory for DD calibration h5merge - #147

Open
tikk3r wants to merge 1 commit into
masterfrom
memory-ddmerge
Open

Reserve memory for DD calibration h5merge#147
tikk3r wants to merge 1 commit into
masterfrom
memory-ddmerge

Conversation

@tikk3r

@tikk3r tikk3r commented Jul 17, 2026

Copy link
Copy Markdown
Member

This PR adds a memory allocation to the merging of all DD solutions. For my run Toil reports 21.61 GB used, but Slurm reports a max RSS of 44017964K, so I went with the Slurm value to be on the safe side as I don't know what Toil's polling frequency is. This is with 37 directions.

Reserve memory for DD calibration h5merge
@tikk3r
tikk3r requested a review from jurjen93 July 17, 2026 09:34
@tikk3r

tikk3r commented Jul 17, 2026

Copy link
Copy Markdown
Member Author

@jurjen93 how many directions did you end up merging for EN1? To get a feel for how much leeway to give the memory.

@jurjen93

Copy link
Copy Markdown
Member

@jurjen93 how many directions did you end up merging for EN1? To get a feel for how much leeway to give the memory.

For EN1 I had 24 directions, for EDFN I have 41 directions

@jurjen93

Copy link
Copy Markdown
Member

For EDFN I merged the Dutch DD calibration solutions with the Dutch+Int DD calibration solutions. Because of the higher time/freq resolution from the Dutch DD calibration solutions, the h5parm becomes for 41 directions around 103GB. So, maybe constrain it to 150 GB, to be sure? (I dont think we will have more than 60 directions for a field)

Comment thread steps/multidir_merger.cwl
- class: DockerRequirement
dockerPull: vlbi-cwl
- class: ResourceRequirement
ramMin: 45000

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggestion: put this on 150000. My h5parm is for example 100 GB with ~40 directions, so for example ~60 directions you would need ~150 GB.

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.

2 participants