Closed zhafen closed 7 years ago
Original comment by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)
Fixed as of the latest commit.
Original comment by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)
The above is what happens when I increase neg
to 10. It looks okay.
Honestly, though, the best option might be honesty: we don't know what happens to gas at high redshifts... This is something we should be aware of, going forward.
Original comment by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)
While boundary conditions should be applied as described above, after doing so there is still a suspicious high-z population, the "merger-gas" population in the above figure.
Particles in this population were in a galaxy for at least 100 Myr before accreting but were not in the main galaxy in the first few snapshots after the main galaxy was officially resolved. This means that they have the potential to actually be fresh accretion/recycled gas. In fact, given previous knowledge, they likely are (especially because there's no accompanying merger-star component).
There are two ways I can think of addressing this:
neg
(the number of snapshots to which we apply the boundary condition) to a larger value.Original comment by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)
Boundary conditions need to be applied in two places:
neg
snapshots it's resolved should be counted as pristine gas.
Originally reported by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)
At some point, early in the simulation, the halo finder successfully resolves what galaxy is the main galaxy. However, under our classification scheme, the moment the halo finder resolves our main galaxy everything that was previously actually inside the galaxy (which the halo finder couldn't resolve) is suddenly classified as "merger". This clearly isn't correct.
The proposed solution:
n_star_main_gal >= 100
.neg
snapshots after the galaxy is resolved as pristine gas.