Code Review TODO

[kbanker]

• Add units to function definitions and other variables (arguments specifically)
• Top level directory should be cleaner (move physics to inner dir)
• Fix read_yaml and load_vars (load_vars initialized multiple times)
• Clean up unused functions in notebook + modules
• Standardize get vs make (in get_dm_halo vs make_sz_cluster): leaning towards renaming to make
• Make the start/end of the different parts of the notebook clearer (Making distribution vs Running sims using them)
• Update reading in massdist from csv to h5
• Rename M200_to_R200 function to reflect use
• Create universal Eq11 from Battaglia2012, then move the param calculations into their own functions
• make x-axis log scale for Compton profile 1-d plot
• Make separate toolbox module for noise/beam convolution/fSZ calculation
• Look into CMB generation assumed geometry from pixell
• Halo ID for each halo
• Run ID for the sim run
• Tool/filter module for aperture photometry
• Move noise method from run_sims to a noise module

[evavagiakis]

Mostly completed, moving into two discussions: #153 for packaging and #154 for physics to dos