Skip to content
Snippets Groups Projects

Chemistry API changes for metal fluxes

Merged Yolan Uyttenhove requested to merge chemistry_mass_flux_updates into master

This merge request extends the chemistry API as discussed in #834 to correctly advect metals in hydro schemes with mass fluxes between particles.

The approach is very similar to @mivkov's approach to tracking the mass fractions of the ionizing species in GEAR-RT.

Current status

This is a working implementation for the EAGLE chemistry scheme. Chemistry schemes other than EAGLE and none currently throw a compile time error trying to compile with a hydro scheme that does mass fluxes (in this branch, only gizmo-mfv).

To-do

  • Implement the new functions for other chemistry schemes
  • How to treat diffusion in this case? (only necessary when implementing for GEAR-diffusion)
  • Advect other quantities tracked by chemistry as well (e.g. mass_from_SNIa, metal_mass_fraction_from_SNIa and others)?

Other thoughts

As discussed in #834, we would also need to disable metal smoothing when using this with e.g. moving mesh hydro.

Edited by Yolan Uyttenhove

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading