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

OMX #2

Open
bstabler opened this issue Sep 13, 2016 · 4 comments
Open

OMX #2

bstabler opened this issue Sep 13, 2016 · 4 comments

Comments

@bstabler
Copy link

Have you considered the work done under the OMX project?

@PeterKucirek
Copy link
Collaborator

Oh indeed! I tested using OMX as my matrix-caching format however found the implementation to be much too slow for what I needed. However, ideally I would like to include an OMX-to-DataFrame reader in this project, when I have some time.

@bstabler
Copy link
Author

Are you reading and writing to matrix rows/cols/cells from/to the disk during your model run? If so, then, yes, OMX is too slow. It is designed more for entire matrix I/O at the start or end of a model run.

@PeterKucirek
Copy link
Collaborator

PeterKucirek commented Sep 23, 2016

I do each matrix in one shot, but in general, yes I'm doing within a model run.

The other thing to note is that this package converts these formats to a DataFrame which I believe is a different output format from the current OMX library (which IIRC returns a numpy array).

@bstabler
Copy link
Author

so just reading or writing the entire matrix from OMX was too slow? Typically you just need to read, say, skims, at the start of the demand model, so if it takes 5 or 10 minutes, then that is acceptable since the demand model takes hours. Are you doing something different? I'm curious what your use case is and why OMX doesn't work. Thanks.

Yes OMX returns a numpy array not a pandas DataFrame.

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