-
Notifications
You must be signed in to change notification settings - Fork 122
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
strings.Builder is not available in yunabe/lgo:20190218 #80
Comments
I pushed yunabe/lgo:20190320 with go1.11. One problem of go1.11 is lgo has a performance issue since go1.10 due to a regression in golang and the overhead of code execution is 5x slower with go1.11. With my Macbook Pro Late 13, the overhead is around 3 secs. The overhead can be longer if you don't use SSD. See https://github.com/yunabe/lgo#go110 for details and the route cause of the issue in golang. |
Thanks for that. Yes, I also tried another version. Yesterday I built a go1.10 lgo and tried the code I wanted to use. It was unreasonably slow. It does run though, but fails halfway through with this
I have that here. Related: in order to get lgo to build I needed to remove main packages from the gonum tree. Otherwise I saw errors for those like this
I have asked about this on go-nuts, but have not had an answer yet. Do you know anything about this? I also tried go1.12, but that failed completely (possibly because I have |
I committed a change to ignore main packages (34ba4a2).
Yes, I noticed the compilation of lgo is broken with go1.12. I think it's related to golang/go#30768. But I need further investigation. |
Thank you for working on this. |
BTW I think that the gc linker should not panic no matter what the input, but I have not been able to reproduce it outside |
We have stabilised the graph API in gonum, so I've taken up trying your graphprac-binder with the following URL https://mybinder.org/v2/gh/yunabe/graphprac-binder/master, however it doesn't build. Failing with (briefly because mybinder doesn't allow raw, copiable text logs to be see)
So presumably the docker image has a <go1.10 standard library. Is there a more recent version?
The text was updated successfully, but these errors were encountered: