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

fix #29955, intersection bugs involving triangular constraints #29986

Merged
merged 1 commit into from
Nov 16, 2018

Conversation

JeffBezanson
Copy link
Member

So far I feel this fix is pretty safe, so likely a candidate for backporting. We'll see what the tests say.

@nanosoldier runbenchmarks(ALL, vs = ":master")

@JeffBezanson JeffBezanson added types and dispatch Types, subtyping and method dispatch bugfix This change fixes an existing bug labels Nov 9, 2018
@JeffBezanson
Copy link
Member Author

@nanosoldier runbenchmarks(ALL, vs = ":master")

@nanosoldier
Copy link
Collaborator

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @ararslan

@JeffBezanson
Copy link
Member Author

#29923

@JeffBezanson
Copy link
Member Author

On freebsd ci:

LinearAlgebra/bunchkaufman        (16) |    77.66 |   3.61 |  4.6 |    4972.30 |   326.16
      From worker 15:	Assertion failed: (e && ptls->bt_size != 0), function jl_sig_throw, file /usr/home/julia/julia-fbsd-buildbot/worker/freebsdci/build/src/task.c, line 437.
      From worker 15:	
      From worker 15:	signal (6): Abort trap
      From worker 15:	in expression starting at /usr/home/julia/julia-fbsd-buildbot/worker/freebsdci/build/usr/share/julia/stdlib/v1.1/LinearAlgebra/test/lapack.jl:278
      From worker 15:	_thr_kill at /lib/libc.so.7 (unknown line)
      From worker 15:	__raise at /lib/libc.so.7 (unknown line)
      From worker 15:	abort at /lib/libc.so.7 (unknown line)
      From worker 15:	__assert at /lib/libc.so.7 (unknown line)
      From worker 15:	jl_sig_throw at /usr/home/julia/julia-fbsd-buildbot/worker/freebsdci/build/src/task.c:437
      From worker 15:	unknown function (ip: 0xffffffffffffffff)
      From worker 15:	Allocations: 281299313 (Pool: 281243890; Big: 55423); GC: 572
Worker 15 terminated.

Anybody seen this before?

@ararslan
Copy link
Member

I definitely haven't. @iblis17?

@iblislin
Copy link
Member

🤔 no idea.

@iblislin
Copy link
Member

@mohamed82008
Copy link
Contributor

Is there anything unsafe about this branch, or can I use it for now to solve #29955?

@mohamed82008
Copy link
Contributor

mohamed82008 commented Nov 16, 2018

It seems that tests pass on 64 bits so I will give it a spin.

@JeffBezanson JeffBezanson merged commit 11243da into master Nov 16, 2018
@JeffBezanson JeffBezanson deleted the jb/fix29955 branch November 16, 2018 07:17
blegat pushed a commit to blegat/julia that referenced this pull request Nov 18, 2018
@blegat blegat mentioned this pull request Nov 18, 2018
KristofferC pushed a commit that referenced this pull request Nov 19, 2018
@KristofferC KristofferC mentioned this pull request Nov 19, 2018
61 tasks
tkf pushed a commit to tkf/julia that referenced this pull request Nov 21, 2018
@alyst alyst mentioned this pull request Nov 22, 2018
KristofferC pushed a commit that referenced this pull request Dec 12, 2018
KristofferC pushed a commit that referenced this pull request Feb 11, 2019
KristofferC pushed a commit that referenced this pull request Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug types and dispatch Types, subtyping and method dispatch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants