Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

I/O timing issues #38

Open
rmandelb opened this issue Sep 17, 2014 · 4 comments
Open

I/O timing issues #38

rmandelb opened this issue Sep 17, 2014 · 4 comments

Comments

@rmandelb
Copy link
Collaborator

I was just noticing as I looked at the TreeCorr Readme file that it uses a package called pandas instead of numpy for fast text file i/o:

This package significantly speeds up the reading of ASCII input catalogs over the numpy functions loadtxt or genfromtxt.

I see that we use genfromtxt. Do you have any sense for how much the file I/O is hitting us for typical uses of Stile? If it's a problem we might consider adding this dependency.

@msimet
Copy link
Owner

msimet commented Sep 17, 2014

I don't know, but we can look into it. I don't think it would be hard to
make this an optional dependency either...
On Sep 17, 2014 12:29 PM, "Rachel Mandelbaum" notifications@github.com
wrote:

I was just noticing as I looked at the TreeCorr Readme file that it uses a
package called pandas instead of numpy for fast text file i/o:

This package significantly speeds up the reading of ASCII input catalogs over the numpy functions loadtxt or genfromtxt.

I see that we use genfromtxt. Do you have any sense for how much the file
I/O is hitting us for typical uses of Stile? If it's a problem we might
consider adding this dependency.


Reply to this email directly or view it on GitHub
#38.

@rmandelb
Copy link
Collaborator Author

True. Try/except statements are our friends.

@rmandelb
Copy link
Collaborator Author

Looking at TreeCorr usage of pandas, it looks slightly more complex than genfromtxt:

https://github.com/rmjarvis/TreeCorr/blob/releases/3.1/treecorr/catalog.py

But not too bad. Now that I managed to install TreeCorr I can do it.

@msimet
Copy link
Owner

msimet commented Sep 18, 2014

Great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants