search

nu-radio / radiotools

A tool package for cosmic-ray and neutrino radio detectors
GNU General Public License v3.0
8 stars 5 forks source link

Imports xrange from past to make code work for py3 #22

Closed christophwelling closed 5 years ago

cg-laser commented 5 years ago

I think it is better to just replace xrange with range. In p3 range is clever enough to return an interator when looping over it.

fschlueter commented 5 years ago

right but this is not true for the range in p2 right? So it might slow done p2?

cg-laser commented 5 years ago

yes in principle, but I think this is negligible. And we anyway want to focus on p3, so a little less performance in p2 is ok I'd say. And then we don't need the extra import.