<div dir="ltr">FWIW, I'd prefer nibabel not do automatic type casting, but this is an emotional response, rather than one backed up by "hey this will break my code!"<br><div><br></div><div>But! If we're to do it, I don't like the approach of an as_float parameter. Passing dtype (probably defaulting to np.float64) would be much better, or it might make more sense to add a new method, say, get_data_without_type_cast().</div><div><br></div><div>-n</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 6, 2015 at 11:45 AM Matthew Brett <<a href="mailto:matthew.brett@gmail.com">matthew.brett@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Jul 6, 2015 at 5:32 PM, Bertrand Thirion<br>
<<a href="mailto:bertrand.thirion@inria.fr" target="_blank">bertrand.thirion@inria.fr</a>> wrote:<br>
> +1 we (and more importantly, our students)  should rely as much as possible<br>
> on the standard behavior of numpy arrays and make adequate decisions, rather<br>
> than having to figure out the details of the API of neuroimaging libraries.<br>
> So the defaut should be unchanged.<br>
<br>
Your reasoning implies the opposite.   Numpy tries very hard not to<br>
return arrays of unknown or unpredictable data types, and that is the<br>
situation we have here.   The returned datatype from a nibabel image<br>
is essentially arbitrary, in that very few sources of nifti files<br>
place any weight on whether there are non-default scalefactors or not.<br>
At the moment, we do, depend on this, silently, and that is extremely<br>
confusing, and quite contrary to the standard numpy way,<br>
<br>
Cheers,<br>
<br>
Matthew<br>
_______________________________________________<br>
Neuroimaging mailing list<br>
<a href="mailto:Neuroimaging@python.org" target="_blank">Neuroimaging@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/neuroimaging" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/neuroimaging</a><br>
</blockquote></div>