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

[release-0.4, RFC] backports for 0.4.1 #13834

Merged
merged 96 commits into from
Nov 8, 2015
Merged

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Oct 31, 2015

🚂

omus and others added 30 commits October 30, 2015 19:25
When working with Sublime as my default editor on OS X I had set my
editor to:

EDITOR=/Applications/Sublime\ Text.app/Contents/SharedSupport/bin/subl

Since the `ispath` function did not recognize this as a path (because
contained a backslash) this resulted in my editor not being recognized
as "subl" and line number support being unavailable.

The section of `edit` that deals with determining the user's editor has
been abstracted into a new function `editor`. Test cases have been
added for `editor` along with some basic tests for `shell_split`.

Note that `startswith` is used to identify alternative editor names:

- Sublime Text can refer to "subl" or "sublime".
- Emacs can refer to "emacs" or "emacsclient".

(cherry picked from commit 53fcec2)
ref #13032
Moves all the cmdline option error handling code from client.jl to
repl.c.

(cherry picked from commit dff1c90)
ref #13245
(cherry picked from commit 51b6d24)
ref #13315
(cherry picked from commit d5edd11)
(cherry picked from commit b31925c)
ref #13376
This removes the restriction on setindex_shape_check that the index lengths
must all be Int, allowing data structures to return non-Int lengths (e.g.,
`UInt(1):UInt(2)` or `big(1):big(2)`).  Fixes #13250.

(cherry picked from commit e5b3cc6)
ref #13252
ref #922 and 35c5234

(cherry picked from commit 99ee2ff)
ref #13431
(cherry picked from commit fa6c977)
ref #13460
from upgrade to Git 2.x, causing failure of spawn test on appveyor

also remove busybox since Git 2.x makes it unnecessary

(cherry picked from commit c18d271)
ref #13509
fixes #12865

(cherry picked from commit b7697e7)
Docs attached to numbers weren't being found.

Finding docs for specific methods using the syntax

    help?> f(::Int)

wasn't being recognised due to changes in how `gen_call_with_extracted_types`
generated an error when it failed. (Previously the error was thrown
by the function, but is now returned as an `:(error("..."))` expression.)

(cherry picked from commit 446ddaa)
ref #13535
hvcat doc example was not working
I changed it to one that does.
(cherry picked from commit 3d132a9)
ref #13566
Removed test for a non-working example and added clearer doctest example instead.
(cherry picked from commit 0e498d3)
as long as the value is zero

(cherry picked from commit 566acb1)
ref #13587
Docstrings defined before `==` is available for `Symbol` comparisons
causes a build error. Fix by comparing with `===`.

Ref: #13534 (comment)
(cherry picked from commit d8b6a24)
ref #13602
(cherry picked from commit e44938c)
ref #13630
Also added tests/docs for those functions and exported them

(cherry picked from commit 2bf7818)
ref #13637
(cherry picked from commit 658a0a8)
ref #13561
…lication,

e.g. quaternions.

(cherry picked from commit fe7bb29)
ref #13640
This allows the build to proceed for AArch64 (mentioned in #10791) and others.

(cherry picked from commit f244fb6)
ref #13558
@timholy
Copy link
Member

timholy commented Nov 1, 2015

The Images failure looks due to a specific instance of random numbers. I'll have to investigate and likely change that test, but for your purposes you can ignore that particular error.

@timholy
Copy link
Member

timholy commented Nov 1, 2015

For your benefit, justifications for #13803 include: it's necessary for jump-dev/JuMP.jl#604 and would allow operations like

using SIUnits.ShortUnits
A = [1m 2m; -1m 5m]
v = [3m,-1m]
A*v

to become type-stable (currently it returns a non-leaftype as a mechanism to circumvent what would otherwise be an error). That said, I also recognize the risk. As I stated elsewhere, since I'm obviously biased I would prefer to defer to others in making the decision about whether to include it.

The patch list looks delicious, very much looking forward to 0.4.1 even if nothing else gets added.

hayd and others added 2 commits November 3, 2015 06:32
This mirrors html rendering.

Test markdown's terminal output.

(cherry picked from commit 4e3aa06)
ref #13835
…792.

Make fwd/bwdTriSolve! work for triagular views

Add check for triangular matrices in sparse factorize

(cherry picked from commit 81753cc)
ref #13793
@hiccup7
Copy link

hiccup7 commented Nov 3, 2015

Now that OpenBLAS v0.2.15 has been merged to the master branch (#13808), I vote for including it in the backports for 0.4.1. I am assuming that the OpenBLAS API stayed the same. Many bugs and performance issues have been fixed in the 7 months since the last OpenBLAS release, and they include showstoppers for me in using Julia.

@tkelman
Copy link
Contributor Author

tkelman commented Nov 4, 2015

Scroll up, it's already here a3f7c1f

Though when I ran the microbenchmarks it did look like randmatstat got about 15% slower, which might be attributable to the new version of openblas?

@tkelman
Copy link
Contributor Author

tkelman commented Nov 4, 2015

I was wrong to blame openblas, it was actually 1fc2d8e that was responsible for the 15% slowdown in rand_mat_stat.

DataArrays has still not been tagged to work with #13803. I could be convinced to maybe pull #13491 and #13553 in here if perf and PackageEvaluator look okay on them, but unless someone argues for them I'm inclined to leave them until the next point release.

@hiccup7
Copy link

hiccup7 commented Nov 4, 2015

@tkelman - thanks for the clarification.

@jakebolewski
Copy link
Member

Is there a rough ETA on the next tagged bugfix? Some students are asking about it in @alanedelman's class.

@tkelman
Copy link
Contributor Author

tkelman commented Nov 4, 2015

This weekend probably. Asked for input and opinions above and have yet to get any other than from Tim.

On Wed, Nov 4, 2015 at 11:14 AM -0800, "Jake Bolewski" notifications@github.com wrote:
Is there a rough ETA on the next tagged bugfix, some students are asking about it in @alanedelman's class.


Reply to this email directly or view it on GitHub:
#13834 (comment)

amitmurthy and others added 9 commits November 7, 2015 02:31
This would previously have been an infinite loop if
`length(A) == typemax(Int)` so the loop vectorizer couldn't compute a
trip count.

(cherry picked from commit fa89a6e)
ref #13866
This is consistent with eltype() and therefore collect().
Fixes #9261.

(cherry picked from commit f379e08)
ref #13903
(cherry picked from commit bbec291)
When `macroexpand` fails due to an undefined macro in the expression
return the resulting error rather than the more generic and misleading
"invalid doc expression..." error message.

Fixes #13905.

(cherry picked from commit c4c38d1)
ref #13908
@tkelman
Copy link
Contributor Author

tkelman commented Nov 7, 2015

https://gist.github.com/c4ae8dfc79bdc2111f7e

Most of the failures are either intermittent things I've seen before, or odd issues that only happened on the 0.4.0 baseline run where some unrelated package gives something like LoadError: module Graphics not found in current path; removed orphaned cache file /home/vagrant/.julia/lib/v0.4/Graphics.ji.

Will merge and tag shortly unless anyone wants to advocate that anything remaining on the checklist in the first comment is must-do-now.

tkelman added a commit that referenced this pull request Nov 8, 2015
[release-0.4, RFC] backports for 0.4.1
@tkelman tkelman merged commit 3660d47 into release-0.4 Nov 8, 2015
@tkelman tkelman deleted the tk/backports-0.4.1 branch November 8, 2015 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.