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

Optimize CREFL for memory #10

Closed
mraspaud opened this issue Sep 20, 2016 · 5 comments
Closed

Optimize CREFL for memory #10

mraspaud opened this issue Sep 20, 2016 · 5 comments
Labels
enhancement code enhancements, features, improvements

Comments

@mraspaud
Copy link
Member

No description provided.

@djhoese
Copy link
Member

djhoese commented Apr 27, 2017

@mraspaud iirc you did this with the python version. It still requires quite a bit of memory but should we consider this closable? Or maybe we should leave it open as incentive to rewrite it in cython/C? That way we can benefit from element-wise for-loops like the original C code.

@mraspaud
Copy link
Member Author

Yes I used the python version, and I would like to keep it open for the reason you mention

@djhoese djhoese added the enhancement code enhancements, features, improvements label Dec 4, 2017
@mraspaud
Copy link
Member Author

I'm closing this as dask made this much better

@djhoese
Copy link
Member

djhoese commented May 22, 2018

Do we know if crefl actually works right now?

@mraspaud
Copy link
Member Author

mraspaud commented May 22, 2018 via email

@mraspaud mraspaud moved this from In Progress to Done in SatPy 1.0 Release May 22, 2018
mraspaud pushed a commit that referenced this issue Jun 20, 2018
mraspaud pushed a commit that referenced this issue Jun 10, 2019
Bring my fork up to date
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement code enhancements, features, improvements
Projects
Development

No branches or pull requests

2 participants