Skip to content
Snippets Groups Projects

Sink accretion

Merged Loic Hausammann requested to merge sink_accreation into master

In this merge request, I am adding two new type of tasks: sink_ghost and sink_accretion. The ghost is an implicit task between the mergers and the accretion and the accretion is an interaction between a sink particle and a hydro particle that is done on active sink particles (not necessary active hydro).

Due to the "recent" addition of the union for the cell structure, I had to define new files for GEAR. They are simply a copy/paste from the default ones with a constant probability of creating sink particles. This is done for debugging and will change once I will start to implement the physics.

I had some issues with the sink formation tasks that were only created in top level cells that contain hydro particles. Now in order to be safer, I am creating a sink formation task for all the top level cells (but then they are not necessary active). This could be changed in the future.

I am also sharing the IACT functions between the accretion and the compute_sink_formation tasks and it might not be the most efficient way of doing it. I have added a TODO in order to mention that we could swap the loops (first hydro, then sink -> first sink, then hydro) in order to speedup a bit the code.

Here you can see the new task dependencies: dependency_graph_0

My next steps, in the order, will be to:

  • Implement the star formation from the sink (I will steal the star formation task and adapt the behavior depending on the policies),
  • Start working on the physics,
  • Implement the MPI.

As I am close to the end of my thesis, I am not expecting to finish everything, but I should be able to at least start with the physics.

Edited by Matthieu Schaller

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
  • Mladen Ivkovic
  • Mladen Ivkovic
  • Mladen Ivkovic
  • Mladen Ivkovic
  • Mladen Ivkovic
    • Resolved by Mladen Ivkovic

      Ok, I went through it. Seems mostly fine to me, apart from some questions I have left as comments. However, what's missing is the task types to be added to the python plotting scripts as well as the DefaultSink:CutoffRadius parameter in examples/parameter_examples.yml file

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Cool, good with me. Matthieu's turn now :)

  • Mladen Ivkovic assigned to @matthieu and unassigned @mivkov

    assigned to @matthieu and unassigned @mivkov

  • Matthieu Schaller changed title from Sink accreation to Sink accretion

    changed title from Sink accreation to Sink accretion

  • Matthieu Schaller changed the description

    changed the description

  • Rebase please before review.

  • Loic Hausammann added 36 commits

    added 36 commits

    Compare with previous version

  • Done

  • Matthieu Schaller resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading