Skip to content
Snippets Groups Projects

Added new outputs to SPHENIX scheme

Merged Josh Borrow requested to merge add-sphenix-output-#647 into master
1 unresolved thread

Adds new outputs to SPHENIX.

Implements #647 (closed).

Edited by Matthieu Schaller

Merge request reports

Merged by Matthieu SchallerMatthieu Schaller 5 years ago (Apr 15, 2020 8:28am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Josh Borrow added 1 commit

    added 1 commit

    • 5cf6dbbb - Minor change in laplacian output name

    Compare with previous version

  • Josh Borrow resolved all discussions

    resolved all discussions

  • Author Developer

    Think this is ready to be merged.

  • I forgot to ask the question I had yesterday. Are we sure the scale-factors are correct?

  • Matthieu Schaller
  • Matthieu Schaller changed the description

    changed the description

  • Happy for this to be merged with the changes above implemented,

  • Josh Borrow added 1 commit

    added 1 commit

    • 0c89e237 - Added comments about hubble term included in div_v

    Compare with previous version

  • Author Developer

    Changes implemented apart from the error call. Can't see a good way to do this without including a weird ifdef somewhere.

  • 207 209
    208 210 list[10] = io_make_output_field_convert_part(
    209 211 "DiffusionParameters", FLOAT, 1, UNIT_CONV_NO_UNITS, 0.f, parts, xparts,
    210 212 convert_diffusion, "Diffusion coefficient (alpha_diff) of the particles");
    213
    214 list[11] = io_make_output_field(
    215 "LaplacianInternalEnergies", FLOAT, 1, UNIT_CONV_FREQUENCY_SQUARED,
    216 5.f - 3.f * hydro_gamma, parts, diffusion.laplace_u,
    217 "Laplacian (del squared) of the Internal Energy per "
    218 "unit mass of the particles");
    219
    220 list[12] = io_make_output_field(
    221 "VelocityDivergences", FLOAT, 1, UNIT_CONV_FREQUENCY, 0.f, parts,
    222 viscosity.div_v,
    223 "Local velocity divergence field around the particles. Provided without "
    224 "cosmology, as this includes the Hubble flow.");
  • Josh Borrow added 1 commit

    added 1 commit

    • 3a4b9bd8 - Updated strings to include extra information

    Compare with previous version

  • mentioned in commit e1c78721

  • Please register or sign in to reply
    Loading