Skip to content

Remove note on total accreted masses#65

Merged
jchelly merged 1 commit into
jchelly:masterfrom
robjmcgibbon:accreted_mass_units
May 26, 2026
Merged

Remove note on total accreted masses#65
jchelly merged 1 commit into
jchelly:masterfrom
robjmcgibbon:accreted_mass_units

Conversation

@robjmcgibbon

Copy link
Copy Markdown
Contributor

The note about the "Incorrect units for TotalAccretedMasses" was wrong. This is only a problem for the original jet runs, I must have looked at them by mistake. Since we do not release the original jet runs, there is no reason to keep this note.

import os
import h5py

base_dir = '/cosma8/data/dp004/flamingo/Runs/L1000N1800'
runs = os.listdir(base_dir)

for run in runs:
    snap = f'{base_dir}/{run}/snapshots/flamingo_0077/flamingo_0077.hdf5'
    with h5py.File(snap, 'r') as file:
        if 'PartType5' in file:
            print(run)
            print(file['PartType5/TotalAccretedMasses'].attrs['U_t exponent'])

returns

HYDRO_JETS_old
[-1.]
HYDRO_STRONGEST_AGN
[0.]
HYDRO_STRONG_JETS_old
[-1.]
HYDRO_PLANCK_LARGE_NU_VARY
[0.]
HYDRO_FIDUCIAL_HiResDM
[0.]
HYDRO_FIDUCIAL
[0.]
HYDRO_PLANCK_LARGER_NU_FIXED
[0.]
HYDRO_JETS_published
[0.]
HYDRO_STRONG_SUPERNOVA
[0.]
HYDRO_WEAK_AGN
[0.]
HYDRO_PLANCK_DCDM_24
[0.]
HYDRO_PLANCK_LARGE_NU_FIXED
[0.]
HYDRO_PLANCK
[0.]
HYDRO_LOW_SIGMA8_STRONGEST_AGN
[0.]
HYDRO_STRONG_AGN
[0.]
HYDRO_STRONGER_AGN
[0.]
HYDRO_LOW_SIGMA8
[0.]
HYDRO_STRONG_JETS_published
[0.]
HYDRO_PLANCK_DCDM_12
[0.]
HYDRO_STRONGER_AGN_STRONG_SUPERNOVA
[0.]

@jchelly

jchelly commented May 26, 2026

Copy link
Copy Markdown
Owner

Thanks Rob. I'll go ahead and merge this.

@jchelly jchelly merged commit fd09529 into jchelly:master May 26, 2026
1 check passed
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