crp/OpenMC
1
0
Fork 0
mirror of https://github.com/openmc-dev/openmc.git synced 2026-07-28 06:05:58 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

remove unnecessary comment

Browse source 
Co-authored-by: Paul Romano <paul.k.romano@gmail.com>
This commit is contained in:
Gavin Ridley 2023-04-18 17:24:36 -04:00 • committed by GitHub
parent b52e1f106c
commit 2ff6c0408e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

20
src/state_point.cpp
View file

@ -596,26 +596,6 @@ void write_source_bank(hid_t group_id, gsl::span<SourceSite> source_bank,
hid_t banktype = h5banktype();
// Set total and individual process dataspace sizes for source bank
// TODO: are these correct?
// the old code was
//
// int64_t dims_size = settings::n_particles;
// int64_t count_size = simulation::work_per_rank;
//
// but that doesn't make sense! The count on a rank is not
// necessarily equal to work_per_rank. For the case of the
// fission source bank, that's true, but for surface source
// creation, there may be a different number of particles
// on each rank.
//
// Hence, I have changed count_size to be the number
// on this rank rather than work_per_rank. Similarly,
// dims_size used to be n_particles, but for surface sources,
// we are not guaranteed to create n_particles at the surface.
// As a result, I've changed this to be the total size of
// the bank across all ranks, which maintains the correct
// behavior for fission bank outputs.
int64_t dims_size = bank_index.back();
int64_t count_size = bank_index[mpi::rank + 1] - bank_index[mpi::rank];