On Thu, May 29, 2014 at 12:36 PM, Blake Griffith <notifications@github.com>
wrote:

> In distarray/dist/maps.py:
>
> > @@ -415,7 +417,10 @@ def from_dim_data_per_rank(cls, context, dim_data_per_rank, targets=None):
> >
> >          coords = [tuple(d['proc_grid_rank'] for d in dd) for dd in
> >                    dim_data_per_rank]
> > -        self.rank_from_coords = {c: r for (r, c) in enumerate(coords)}
> > +
> > +        self.rank_from_coords = np.empty(self.grid_shape, dtype=np.int32)
>
> Why np.int32 and not the system int or float which is usually the default
> right?
>

It definitely needs to be an integer, so `float` is out.

I know of no supercomputer that has anywhere close to 2.1 billion processes
available for use, so we're comfortably safe with using int32's.  Switching
this to np.int is fine by me, but can be done in another PR.


> —
> Reply to this email directly or view it on GitHub
> <https://github.com/enthought/distarray/pull/378/files#r13193236>.
>



-- 
Kurt W. Smith, Ph.D. <ksmith@enthought.com>
Enthought, Inc. <http://www.enthought.com>  | 512.536.1057