Closed funnell closed 4 years ago
Thanks for bringing this up @funnell !! Great spot. As mentioned in #11 due to my lack of GitHub savvy-ness I couldn't figure out how to add to this PR so I just made another one with your changes and a few other bits like updating tests and creating an Enum structure for the memory units.
Thanks for your contribution!
My cluster uses GB as the default units. Explicitly specifying the memory units as MB fixes this and is probably a good idea anyways!