Andrew Hearin

Results 71 comments of Andrew Hearin

@andrew-zentner - do you know of a reference for this? @duncandc - have you tested how much of a difference this choice makes for a standard HOD mock, or just...

Great, thanks for weighing in. I'll definitely get rid of the scatter class. I'll take a close look at getting rid of `PrimGalpropModel` class as well. The main feature that...

Give users the option to write supported model components that either do or do not work with Quantities. Try to implement this via a class-level use_quantities attribute. If set to...

Yeah, #234 is also long-standing. Probably all these different variations would best done in a sprint, since they're all so similar. Re: combining things. That'd be fine, like with a...

If it's not being used, then it *should* be deleted, so good catch. So long as the disappearance is clearly marked (make sure the module name and the main function...

I'd love to get rid of the c++ compilation @rainwoodman - The only place where any cpp-specific feature is used is in the pairwise distance engines: `pairwise_distance_3d_engine`, `pairwise_distance_xy_z_engine` and `conditional_pairwise_distances`....

Those three functions are the only ones using libcpp.vector in the entire repo @rainwoodman - if you know of common workarounds for this, I would happily use those instead. These...

@manodeep - are the memory constraints I mentioned here actually a big problem? What do you think? The simplest solution is still to use a multi-dim array, where there is...

An alternative option would be to precompute all pairs once and store all pairs of indices and distances in a sparse matrix. Then, after the sparse matrices are built, in...

I'm exploring an algorithm without any memory increase at all. What I do is add one more inner loop where I randomly select an *index* from each weights array at...