Skip to content

Pass halo indices from a file - #218

Merged
robjmcgibbon merged 2 commits into
masterfrom
pass_halo_indices
Aug 14, 2026
Merged

Pass halo indices from a file#218
robjmcgibbon merged 2 commits into
masterfrom
pass_halo_indices

Conversation

@robjmcgibbon

Copy link
Copy Markdown
Collaborator

Adds a --halo-indices-file argument to compute_halo_properties, so that large numbers of halo indices can be passed in a text file. tests/COLIBRE/run_L0025N0188_Thermal.sh now uses the new flag with tests/COLIBRE/halo_indices_0092.txt

SOAP/core/combine_args.py gains COMMAND_LINE_ONLY_PARAMETERS, a deny-list which makes SOAP exit with an error if any of those arguments appear in the Parameters section of the config file.

In SOAP/core/halo_centres.py rank 0 reports how many of the requested indices were matched and writes any unmatched ones to <catalogue_name>_unmatched_halo_indices.txt in the working directory.

flush=True added to every print immediately preceding a comm.Abort across SOAP/, misc/ and tests/ (stops messages being lost when MPI killed the process)

README gains a command line argument reference plus a halo-selection section

@robjmcgibbon
robjmcgibbon merged commit 15e5299 into master Aug 14, 2026
1 check passed
@robjmcgibbon
robjmcgibbon deleted the pass_halo_indices branch August 14, 2026 12:02
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.

1 participant