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.

Sign up for GitHub

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

Jump to bottom

README update? #66

Open
cscherrer opened this issue Apr 22, 2021 · 4 comments
Open

README update? #66

cscherrer opened this issue Apr 22, 2021 · 4 comments

Comments

@cscherrer
Copy link
Collaborator

Hi,

The README says,

GeneralizedGenerated.jl has issues about latency and extensive memory consumptions, and is sometimes likely to trigger segfault bugs when generated functions get enormous(#45, #59). This suggests that you should avoid your expressions from being too large.

In terms of use cases where no closure is needed, you'd better use RuntimeGeneratedFunctions.jl, which has better scalability than GeneralizedGenerated.jl.

Is it correct that this no longer applies?
@cscherrer
Copy link
Collaborator Author

pinging @thautwarm :)

@thautwarm
Copy link
Member

Some still applies. I think RuntimeGeneratedFunctions.jl still consumes less, but it is now more acceptable for users to use GG..

@thautwarm
Copy link
Member

I will update the README after I made more progress in reducing memory assumption..

@cscherrer
Copy link
Collaborator Author

Sounds good. Thank you for all of your great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cscherrer @thautwarm