-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
in lookup_ref: new not defined #577
Comments
I can't reproduce this on OS X. |
Hmm, I see the same error on Ubuntu. I'll try to debug. |
I'm on 32-bit ubuntu 11.10 and I don't see this...strange. |
Something super strange just happened. I had to do something else for a minute on a different computer (well, actually a different VM) and when I came back it suddenly worked again ... I can't explain to myself how that could happen. @stefan-k could you try to do a complete rebuild of Julia and all the dependencies? |
You can't see me type
|
Nevermind my comment. I accidentally used an old build that I had lying around. Still broken :(. But at least it's reproducably still broken. |
I can confirm @pao's analysis of which commit broke it (which would also explain why I didn't see this in my normal fork). I'll have a look at what's going wrong here. EDIT: Also, the terminal issue comes from the fact that the terminal isn't deprepped when Julia encounters an exception. SHould be easily fixed by allowing libradline to deprep at the very end of the program |
Weird, everything works fine for me. |
what OS are you on? I have no idea what causes the problem (that is why the problem exists, I know what line causes it (see cfcc147)). Could this have something to do with 32 vs 64 bit? |
Ubuntu 11.10 (i686) & Fedora 14 (x86_64), so it's not a 32/64-bit problem, at least for me. |
Linux lubuntu-vm 3.0.0-16-generic #28-Ubuntu SMP Fri Jan 27 17:44:39 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux (Ubuntu 11.10) & not working (see above). |
I mentioned in the pull request that the problematic line was
For compleness, I should note that the same line works just fine in my fork (which doesn't have process.jl however). I am truly stomped here. |
I think the key is the definition of print(). If I add a zero-argument print() fallback, all is well. So this may be something to do with varargs? |
Temporary workaround for #577
There's something really messed up here. Maybe a weird memory bug. Going to have to bust out the Valgrind on this. |
I traced this to a possible bug in femtolisp; basically one of the lowering functions in julia-syntax.scm was effectively returning its argument instead of its result. But, it is very sensitive and any perturbation of the input code or lowering code puts the heap in a slightly different state that avoids the problem. I think it would take too long to fully track down, and we should try to ignore it, and eventually replace the front-end with one written in julia anyway. |
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 1 to 3. - [Release notes](https://github.com/codecov/codecov-action/releases) - [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md) - [Commits](codecov/codecov-action@v1...v3) --- updated-dependencies: - dependency-name: codecov/codecov-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
8c84b8c (#56431) Stdlib: SparseArrays URL: https://github.com/JuliaSparse/SparseArrays.jl.git Stdlib branch: release-1.10 Julia branch: backports-release-1.10 Old commit: 279b363 New commit: 8c84b8c Julia version: 1.10.6 SparseArrays version: 1.10.0(Does not match) Bump invoked by: @IanButterworth Powered by: [BumpStdlibs.jl](https://github.com/JuliaLang/BumpStdlibs.jl) Diff: JuliaSparse/SparseArrays.jl@279b363...8c84b8c ``` $ git log --oneline 279b363..8c84b8c 8c84b8c Merge pull request #572 from JuliaSparse/backports-release-1.10 ec38631 Update ci.yml with more architectures 46c8f7e Merge branch 'release-1.10' into backports-release-1.10 2d762b3 Manual commit for PR #550 to backport to 1.10 (#577) 5c37298 Add versions to include arch b539588 Update CI fa49620 Disable nested dissection d2a80a6 Change default QR tolerance to match SPQR (#557) 9b8cd14 SparseMatrixCSC constructor with a Tuple of Integers (#523) 546be18 Fix docs conflict when building as part of full Julia docs (#430) 30fbfc6 Test suite: activate a temp project if we need to install Aqua.jl during the test suite (#425) 91b0aa5 doc: move solvers doc to `src\solvers.md` (#576) 5d3724a Inline sparse-times-dense in-place multiplication ``` Co-authored-by: Dilum Aluthge <[email protected]>
8c84b8c (#56431) Stdlib: SparseArrays URL: https://github.com/JuliaSparse/SparseArrays.jl.git Stdlib branch: release-1.10 Julia branch: backports-release-1.10 Old commit: 279b363 New commit: 8c84b8c Julia version: 1.10.6 SparseArrays version: 1.10.0(Does not match) Bump invoked by: @IanButterworth Powered by: [BumpStdlibs.jl](https://github.com/JuliaLang/BumpStdlibs.jl) Diff: JuliaSparse/SparseArrays.jl@279b363...8c84b8c ``` $ git log --oneline 279b363..8c84b8c 8c84b8c Merge pull request #572 from JuliaSparse/backports-release-1.10 ec38631 Update ci.yml with more architectures 46c8f7e Merge branch 'release-1.10' into backports-release-1.10 2d762b3 Manual commit for PR #550 to backport to 1.10 (#577) 5c37298 Add versions to include arch b539588 Update CI fa49620 Disable nested dissection d2a80a6 Change default QR tolerance to match SPQR (#557) 9b8cd14 SparseMatrixCSC constructor with a Tuple of Integers (#523) 546be18 Fix docs conflict when building as part of full Julia docs (#430) 30fbfc6 Test suite: activate a temp project if we need to install Aqua.jl during the test suite (#425) 91b0aa5 doc: move solvers doc to `src\solvers.md` (#576) 5d3724a Inline sparse-times-dense in-place multiplication ``` Co-authored-by: Dilum Aluthge <[email protected]>
Just pulled fd51447, did
make clean && make
and now I'm getting this error right after starting Julia:julia> in lookup_ref: new not defined
in lookup_ref at /home/stefan/coding/juliafork/jl/multi.jl:340
in take_ref at /home/stefan/coding/juliafork/jl/multi.jl:663
in take at /home/stefan/coding/juliafork/jl/multi.jl:676
in run_repl at /home/stefan/coding/juliafork/jl/client.jl:90
in _start at /home/stefan/coding/juliafork/jl/client.jl:213
Any ideas? I'm running Arch Linux.
The text was updated successfully, but these errors were encountered: