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

Add tooling updates to NEWS.md #17582

Merged
merged 2 commits into from
Jul 27, 2016
Merged

Add tooling updates to NEWS.md #17582

merged 2 commits into from
Jul 27, 2016

Conversation

ViralBShah
Copy link
Member

Mention debugger, Juno+Debugger integration, C++ FFI and multi-threading. I thought these are all big enough to be mentioned right upfront as the major highlights. Perhaps we should also have a few other major highlights such as jb/functions and broadcasting?


* A convenient C++ FFI in
[Cxx.jl](https://github.com/Keno/Cxx.jl). Using it requires
building Julia with some additional build flags that will build clang along with llvm.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is the case any more. Still a bit in flux.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Cxx.jl page still says so. @Keno - suggestions?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Readme last modified a month ago, but see https://github.com/Keno/Cxx.jl/commits/master and the deps/build.jl script

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tkelman is correct, even though there is still a small bug right now that makes it necessary. Once that's resolved Pkg.add will just work though.

@ViralBShah ViralBShah added the docs This change adds or pertains to documentation label Jul 23, 2016
@tkelman
Copy link
Contributor

tkelman commented Jul 23, 2016

Startup time on Windows will be very noticeably better, and the upgraded LLVM version is worth noting.

@ViralBShah
Copy link
Member Author

Is there an issue or PR for faster startup on windows, or something I can refer to? Not necessary, but good to link if available.

@tkelman
Copy link
Contributor

tkelman commented Jul 23, 2016

#16953, though a lot of the discussion happened in a few older issues that were mostly about backtraces

@ViralBShah ViralBShah changed the title Update NEWS.md Add highlights to NEWS.md Jul 23, 2016
@tkelman
Copy link
Contributor

tkelman commented Jul 23, 2016

Many, many new deprecations are also not mentioned in https://github.com/JuliaLang/julia/blob/master/NEWS.md#deprecated-or-removed

@ViralBShah
Copy link
Member Author

I guess the deprecations list should either be complete, or removed altogether. Personally, I don't find the list of all deprecations in NEWS.md useful.

@tkelman
Copy link
Contributor

tkelman commented Jul 23, 2016

The list in NEWS can have more elaboration on subtleties and backwards-compatibility concerns for porting code, since naively following the warning has been frequently seen to be the wrong way to update packages if they don't want to drop support for old versions of Julia yet.

@ViralBShah
Copy link
Member Author

The current one looks not much better than the deprecation warnings. If you can list out the different kinds of subtleties and backwards compatibility, everyone can then put things in the right place.

My guess is that many people will just move on and support 0.5 as the min version within a couple of months of its release, unless we have very clear instructions.

@timholy
Copy link
Member

timholy commented Jul 23, 2016

I like the idea of highlights, but I wonder if it's a concern that 3/4 of them are about packages rather than improvements in core julia. Of course core changes enable that functionality, but a casual reader won't know that and might think that julia proper has hardly changed since 0.4. ("Julia development slowing down, they rely on advertising community packages just to have enough NEWS!")

I also wonder a bit about picking and choosing among the package ecosystem (should Plots and GLVisualize be in that list? won't they be more widely used than Cxx?), but of course the ones you picked out are indeed tremendously exciting highlights.

@ViralBShah
Copy link
Member Author

I don't think the highlights should be restricted to the things I listed, and there are probably other things worth calling out that are of general interest. I am not sure about GLVisualize being of direct wide interest (even though it is fantastic), but Plots.jl certainly deserves a mention. Even the language and compiler improvements could perhaps use more description.

I think the fact that many of these are packages is just the way they are organized, but the debugger and Cxx FFI could equally be in the julia repo.

@JeffBezanson
Copy link
Member

Let's just mention multithreading under new language features, and then we can label this new section something like "tooling improvements", since that's basically what it's about.

-1 to a "highlights" section. I don't want to go through the file and debate/decide which items count as highlights.

@StefanKarpinski
Copy link
Member

I think that we should have a blog post that goes along with the release that highlights the most significant changes with cool examples of what you can do with them. I can work on something.

@ViralBShah
Copy link
Member Author

Yes, we can have highlights as a blog post, and change this to tools in the NEWS.

@ViralBShah
Copy link
Member Author

I have renamed the section to tools improvements. I am perhaps missing this, but the jb/functions seems to be missing a mention. If it is, the performance impact is not clearly explained.

Do we need a performance section?

Julia provides an API for writing generic algorithms for arbitrary
indexing schemes ([#16260]).

New architectures
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're duplicating this

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is duplicated?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you have this section twice now

@tkelman
Copy link
Contributor

tkelman commented Jul 26, 2016

Do we need a performance section?

A short one would be a good idea. Explaining the consequences of jb/functions making anonymous functions now fast, better startup time on Windows, and better SIMD from newer LLVM (we're paying for it in compile time, but maybe that's a known bug rather than something to advertise...)?

@ViralBShah ViralBShah changed the title Add highlights to NEWS.md Add tooling updates to NEWS.md Jul 26, 2016
@ViralBShah
Copy link
Member Author

I am unlikely to be able to update this for the next couple of days as I will be flying. If someone wants to clean this up and move the breaking changes back to the original place, I'll be grateful. Otherwise, I'll do it once I am back home.

@StefanKarpinski
Copy link
Member

Having language changes first seems like the only sane thing to me – you go from most central to least. Starting at tooling seems like going from the outside in, Viral. Any intuitive reason for that?

@ViralBShah
Copy link
Member Author

My thinking was having things that more users are likely to find value in as they read this. Perhaps as you said, maybe the blog post is the right place to do this. Please feel free to move things around as necessary.

Can someone check if @JeffBezanson's jb/functions stuff is mentioned and appropriately highlighted. Would be nice to have it in a performance section.

@JeffBezanson
Copy link
Member

Updated.

fixes #16982, mention `writemime` deprecation

[ci skip]
@JeffBezanson JeffBezanson merged commit 557e483 into master Jul 27, 2016
@tkelman tkelman deleted the vs/news branch July 27, 2016 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants