Conversation
…) is too large; no documentation yet (can add some to )
|
looks fine by me. |
|
Seems fine to me! Is there anything to consider with the indexing of particles that will change because the arrays are now broken up? Also maybe |
|
I agree with the n_jump change. @lgarrison What kind of issues do you see with the particle array breaking up? I cant immediately think of any failure modes. The particles are only used to populate satellites I am fairly sure. So as long as we believe we are not dividing a single halo's particles between different chunks I think we are fine? |
|
@SandyYuan If the halos are using a start and count to index into the particle arrays, then the start will change if the array is split up. This is how we index the @boryanah Did you want to make the |
|
I was just about to say that I thought I made the |
…) is too large; no documentation yet (can add some)
@SandyYuan Could you take a look at this? I will add documentation to
abacus_hod.pyif it seems fine.