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

Precompile the package? #12

Closed
sglyon opened this issue Jan 11, 2018 · 5 comments
Closed

Precompile the package? #12

sglyon opened this issue Jan 11, 2018 · 5 comments

Comments

@sglyon
Copy link
Contributor

sglyon commented Jan 11, 2018

Could the Rsvg module be precompiled?
@lobingera
Copy link
Owner

Yes. But the observable performance gain is limited, as Rsvg.jl is only a very thin layer i.e, function header + ccall to the (compiled) code in the library.

@sglyon
Copy link
Contributor Author

sglyon commented Jan 12, 2018

I see.

This is actually causing problems downstream. I would like to precompile PlotlyJS - which loads Rsvg — but cannot because Rsvg is not precompiled.

@lobingera
Copy link
Owner

PR?

@sglyon
Copy link
Contributor Author

sglyon commented Jan 12, 2018

Yep, happy to open one. Just wanted to chat in an issue to make sure there wasn’t something you knew of that precluded precompilation from working.

@davidanthoff
Copy link
Collaborator

This can also be closed, right?

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
3 participants
@sglyon @davidanthoff @lobingera