Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Find FOF hosts of subhaloes more efficiently #31

Open
VictorForouhar opened this issue Apr 25, 2024 · 3 comments
Open

Find FOF hosts of subhaloes more efficiently #31

VictorForouhar opened this issue Apr 25, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@VictorForouhar
Copy link
Collaborator

As mentioned in #12, we now update the HostId value of particles already belonging to a subhalo. This opens up the possibility of doing host FOF decisions locally, rather than doing MPI communication across all tasks to use the particle hashmap to retrieve the updated value. This would simply involve iterating over the Particle vector and doing the same weighted average as we currently do.

An alternative approach is to use John's new method of particle property retrieval to not have to do as many communications (at the moment, the information is requested from all ranks).

@VictorForouhar VictorForouhar added the enhancement New feature or request label Apr 25, 2024
@jchelly
Copy link
Collaborator

jchelly commented Apr 25, 2024

I think it's clearly better to just use the local information that's already there to decide the FoF host. But we also need to know which MPI rank the host is on. Can we determine that locally too?

@VictorForouhar
Copy link
Collaborator Author

I agree about the local-based estimate. About the rank hosting the FOF, we would unfortunately not know that, so we might still require some communication.

@VictorForouhar
Copy link
Collaborator Author

We could assign hosts locally, and retrieve the rank from one of the tracer particles that belongs to the assigned host, e.g. do

HostIdRank.Rank = CandidateHostRank[HostIdRank.Id];
but only with one particle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants