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

WIP: Backports for Julia 1.5-beta-1 #35846

Merged
merged 17 commits into from
May 27, 2020
Merged

Conversation

KristofferC
Copy link
Member

@KristofferC KristofferC commented May 12, 2020

Backported PRs:

kshyatt and others added 4 commits May 12, 2020 08:02
* Update distributed-computing.md

* Update doc/src/manual/distributed-computing.md

Co-authored-by: Matt Bauman <mbauman@juliacomputing.com>

Co-authored-by: Matt Bauman <mbauman@juliacomputing.com>
(cherry picked from commit 6a9ced3)
@KristofferC KristofferC added needs nanosoldier run This PR should have benchmarks run on it DO NOT MERGE Do not merge this PR! needs pkgeval Tests for all registered packages should be run with this change release Release management and versioning. labels May 12, 2020
@KristofferC KristofferC changed the title WIP: Backports release 1.5 WIP: Backports for Julia 1.5-RC1 May 12, 2020
@KristofferC
Copy link
Member Author

@nanosoldier runtests(ALL, vs = ":release-1.4")

@KristofferC
Copy link
Member Author

@nanosoldier runbenchmarks(ALL, vs = ":release-1.4")

@nanosoldier
Copy link
Collaborator

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

@KristofferC
Copy link
Member Author

KristofferC commented May 12, 2020

Some stuff to look at:

["array", "reductions", "(\"mean\", \"Float64\")"] | 1.08 (5%) ❌ | Inf (1%) ❌
["array", "reductions", "(\"mean\", \"Int64\")"] | 8.12 (5%) ❌ | Inf (1%) ❌

Likely due to the change to mean in Statistics


["io", "array_limit", "(\"display\", \"Array{Float64,2}(10000, 10000)\")"] | 3.13 (5%) ❌ | 0.99 (1%) ✅
["io", "array_limit", "(\"display\", \"Array{Float64,2}(100000000, 1)\")"] | 3.55 (5%) ❌ | 1.05 (1%) ❌

["io", "serialization", "(\"deserialize\", \"Vector{String}\")"] | 2.36 (5%) ❌ | 2.05 (1%) ❌
["io", "serialization", "(\"serialize\", \"Vector{String}\")"] | 3.08 (5%) ❌ | 2.23 (1%) ❌

Likely due to string de-duplication in serialization


["sparse", "index", "(\"spmat\", \"splogical\", 10)"] | 2.90 (30%) ❌ | 0.77 (1%) ✅
["sparse", "index", "(\"spmat\", \"splogical\", 100)"] | 97.14 (30%) ❌ | 0.77 (1%) ✅
["sparse", "index", "(\"spmat\", \"splogical\", 1000)"] | 2.88 (30%) ❌ | 0.87

["sparse", "matmul", "(\"A_mul_B\", \"dense 50x5, sparse 5x50 -> dense 50x50\")"] | 1.64 (30%) ❌ | 1.00 (1%)
["sparse", "matmul", "(\"A_mul_B\", \"dense 50x50, sparse 50x50 -> dense 50x50\")"] | 1.55 (30%) ❌ | 1.00 (1%)
["sparse", "matmul", "(\"A_mul_B\", \"dense 5x5, sparse 5x500 -> dense 5x500\")"] | 1.67 (30%) ❌ | 1.00 (1%)
["sparse", "matmul", "(\"A_mul_B\", \"dense 5x50, sparse 50x500 -> dense 5x500\")"] | 1.70 (30%) ❌ | 1.00 (1%)
["sparse", "matmul", "(\"A_mul_B\", \"dense 5x500, sparse 500x50 -> dense 5x50\")"] | 1.53 (30%) ❌ | 1.00 (1%)
["sparse", "matmul", "(\"A_mul_B\", \"dense 5x500, sparse 500x500 -> dense 5x500\")"] | 1.64 (30%) ❌ | 1.00 (1%)
["sparse", "matmul", "(\"A_mul_B\", \"sparse 50x500, dense 500x5 -> dense 50x5\")"] | 1.36 (30%) ❌ | 1.00 (1%)

["sparse", "sparse solves", "square system (lu), vector rhs"] | 1.87 (5%) ❌ | 1.71 (1%) ❌
["sparse", "transpose", "(\"adjoint!\", (20000, 10000))"] | 1.00 (30%) | 4.00 (1%) ❌
["sparse", "transpose", "(\"adjoint!\", (20000, 20000))"] | 1.00 (30%) | 4.00 (1%) ❌
["sparse", "transpose", "(\"adjoint!\", (600, 400))"] | 1.01 (30%) | 4.00 (1%) ❌
["sparse", "transpose", "(\"adjoint!\", (600, 600))"] | 1.00 (30%) | 4.00 (1%) ❌

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here. cc @maleadt

@KristofferC KristofferC changed the base branch from master to release-1.5 May 13, 2020 14:26
@KristofferC
Copy link
Member Author

I ran PkgEval with the wrong base branch so results here are not 100% accurate (although it should be pretty close because not much have happened on master since branching).

@maleadt
Copy link
Member

maleadt commented May 13, 2020

Let's rerun to be sure (and stress-test some PkgEval changes):

@nanosoldier runtests(ALL, vs = ":release-1.4")

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here. cc @maleadt

* Note that views no longer allocate in NEWS

Co-authored-by: Viral B. Shah <ViralBShah@users.noreply.github.com>
@KristofferC
Copy link
Member Author

The PkgEval result is extermely noisy now due to all changes in the RNG streams (#32208, #33721, #35078, #29240).

This are the packages that failed the previous PkgEval but I filtered out all packages that failed in one of the PkgEvals for the RNG changes to at least be able start looking at stuff that are less likely to just be reliance on RNG streams in test.

@nanosoldier runtests(["AMRVW", "AdvancedHMC", "ApproxFun", "ApproxFunFourier", "ApproxFunRational", "ApproxFunSingularities", "ArrayLayouts", "AutoPreallocation", "AxisIndices", "AxisKeys", "BangBang", "BenchmarkCI", "BlockArrays", "Bridge", "BrkgaMpIpr", "CBindingGen", "ChainRulesCore", "Chemfiles", "ClinicalTrialUtilities", "ColorVectorSpace", "ComponentArrays", "ContinuumArrays", "CorticalSpectralTemporalResponses", "CoverageCore", "CoverageTools", "CurveProximityQueries", "DataAssim", "DataStructures", "DependencyWalker", "DiffEqBayes", "DiffEqParamEstim", "DimensionalData", "DistributedArrays", "Diversity", "Documenter", "DynamicAxisWarping", "DynamicGrids", "FIGlet", "FastIOBuffers", "FillArrays", "FunctionOperators", "FunctionalCollections", "FunctionalTables", "GEMPIC", "GeometryTypes", "Gtk", "GtkReactive", "HomotopyContinuation", "IMFData", "ImageInTerminal", "ImageSegmentation", "ImageView", "InfiniteArrays", "IntervalArithmetic", "JLD2", "JSONSchema", "JuliaVariables", "KrigingModel", "LazyArrays", "LazyStack", "Lerche", "LibGEOS", "LikelihoodProfiler", "LinearMapsAA", "LiveServer", "MIToS", "MicroCoverage", "MultiplesOfPi", "Nabla", "NiLangCore", "OceanGrids", "OpenEphysLoader", "OrthogonalPolynomialsQuasi", "OscillatoryIntegrals", "Pajarito", "PassiveTracerFlows", "PkgBenchmark", "PlotUtils", "Plots", "Pluto", "Polymake", "PowerDynSolve", "PredictMD", "PredictMDFull", "QuadraticToBinary", "QuantumOpticsBase", "QuasiArrays", "RecursiveArrayTools", "RiemannHilbert", "RigidBodyDynamics", "Simulate", "SingularIntegralEquations", "SingularSpectrumAnalysis", "SolverTraces", "SparseGaussianProcesses", "StaticArrays", "StaticNumbers", "StrBase", "StrRegex", "TaylorIntegration", "TensorKitManifolds", "ThreadTools", "ThreePhasePowerModels", "TikzQTrees", "Tracker", "Turf", "TypedPolynomials", "VarStructs", "Wrangling", "XLSXasJSON", "ArchGDAL", "AugmentedGaussianProcesses", "CUDAatomics", "CancerSeqSim", "CompScienceMeshes", "Crayons", "DynamicGridsGtk", "DynamicGridsInteract", "ExprRules", "FEMBase", "FEMBasis", "Fatou", "FieldMetadata", "GtkUtilities", "HybridArrays", "Immerse", "InspectDR", "KnetOnnx", "LegibleLambdas", "LocalRegistry", "MCHammer", "MLStyle", "Manifolds", "Manopt", "Maxvol", "Miletus", "NTFk", "NTNk", "NarrativeTest", "OhMyQSIM", "Persa", "PhysicsInformedML", "Poltergeist", "ProfileView", "RevealedPreferences", "RoMEPlotting", "SimpleGF2", "StructuralInheritance", "TrajectoryOptimization", "TuringModels", "TwoFAST", "VerTeX", "VideoIO", "Winston", "BloomFilters", "Hyperopt", "QuartetNetworkGoodnessFit", "ReinforcementLearningZoo", "DRIPs", "DirectGaussianSimulation", "Gridap", "ImageInpainting", "PowerModels", "StratiGraphics", "DataFlow", "EfficientGlobalOptimization", "ExtensibleUnions", "FastJet", "MbedTLS", "OptionalArgChecks"], vs=":release-1.4")

@KristofferC
Copy link
Member Author

For people pushing commits to this branch (@ViralBShah), please make sure the original PR has a backport label, that you use cherry-pick -x and that you update the first post with the PR.

@ViralBShah
Copy link
Member

I merged a PR and only later realized it went onto this branch. It would be fine to revert it and do the right thing.

However what is the right way to do this? Since this updated NEWS on master but here the update was to be made in HISTORY.

@KristofferC
Copy link
Member Author

If it is a commit that wasn't merged into master then just mergint he PR into here (like what was done) makes sense, I didn't notice that was the case. Could perhaps just add a note to the top post so that everything that gets put in is accounted for.

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here. cc @maleadt

sethaxen and others added 5 commits May 19, 2020 09:24
(cherry picked from commit 686052b)
(cherry picked from commit 8ef29e6)
Currently, the behavior of `Dict(pairs...)` and
`Base.ImmutableDict(pairs...)` are different:

    julia> Dict(:a => 1, :a => 2)[:a]
    2

    julia> Base.ImmutableDict(:a => 1, :a => 2)[:a]
    1

This PR fixes the latter to return 2.

(cherry picked from commit fe59346)
@KristofferC
Copy link
Member Author

There are so many PkgEval "regressions" here that I think it would be useful to merge this, and release a 1.5-beta to make it easier for the community in help testing / fixing packages.

JeffBezanson and others added 6 commits May 25, 2020 22:05
- restore some `copyto!` methods that Knet assumed existed
- allow `findfirst` to work on InfiniteArrays again
- allow constructing a LineNumberNode from a String

(cherry picked from commit 853fe04)
…#35979)

* Bump Zlib BB release to `v1.2.11+10` which enables `-O3` optimisation

* Configure Zlib with CMake

(cherry picked from commit 11c942f)
(cherry picked from commit 664e9ba)
@KristofferC
Copy link
Member Author

@nanosoldier runtests(["AMRVW", "AdvancedHMC", "ApproxFun", "ApproxFunFourier", "ApproxFunRational", "ApproxFunSingularities", "ArchGDAL", "ArrayLayouts", "AugmentedGaussianProcesses", "AutoPreallocation", "AxisIndices", "AxisKeys", "BangBang", "BenchmarkCI", "BlockArrays", "BloomFilters", "Bridge", "CBindingGen", "CUDAatomics", "CancerSeqSim", "ChainRulesCore", "Chemfiles", "ClinicalTrialUtilities", "ColorVectorSpace", "CompScienceMeshes", "ComponentArrays", "ContinuumArrays", "CoverageCore", "CoverageTools", "CurveProximityQueries", "DRIPs", "DataAssim", "DataFlow", "DimensionalData", "DirectGaussianSimulation", "DistributedArrays", "Diversity", "Documenter", "DynamicGrids", "DynamicGridsGtk", "DynamicGridsInteract", "EfficientGlobalOptimization", "ExprRules", "ExtensibleUnions", "FEMBase", "FEMBasis", "FIGlet", "FastIOBuffers", "FastJet", "FieldMetadata", "FillArrays", "FunctionalCollections", "FunctionalTables", "GeometryTypes", "Gtk", "GtkReactive", "GtkUtilities", "HomotopyContinuation", "HybridArrays", "ImageInpainting", "ImageView", "Immerse", "InfiniteArrays", "InspectDR", "IntervalArithmetic", "JLD2", "JSONSchema", "JuliaVariables", "KnetOnnx", "KrigingModel", "LazyArrays", "LazyStack", "LegibleLambdas", "Lerche", "LibGEOS", "LinearMapsAA", "LocalRegistry", "MCHammer", "MIToS", "MLStyle", "Manifolds", "Manopt", "Maxvol", "MbedTLS", "MicroCoverage", "Miletus", "MultiplesOfPi", "NTFk", "NTNk", "Nabla", "NarrativeTest", "NiLangCore", "OceanGrids", "OhMyQSIM", "OpenEphysLoader", "OptionalArgChecks", "OrthogonalPolynomialsQuasi", "OscillatoryIntegrals", "Pajarito", "PassiveTracerFlows", "Persa", "PhysicsInformedML", "PkgBenchmark", "PlotUtils", "Plots", "Poltergeist", "Polymake", "PowerDynSolve", "PredictMD", "PredictMDFull", "ProfileView", "QuantumOpticsBase", "QuartetNetworkGoodnessFit", "QuasiArrays", "RecursiveArrayTools", "RevealedPreferences", "RiemannHilbert", "RigidBodyDynamics", "RoMEPlotting", "SimpleGF2", "SingularIntegralEquations", "SingularSpectrumAnalysis", "StaticArrays", "StaticNumbers", "StrBase", "StrRegex", "StratiGraphics", "StructuralInheritance", "TaylorIntegration", "TensorKitManifolds", "ThreadTools", "ThreePhasePowerModels", "TikzQTrees", "TrajectoryOptimization", "Turf", "TuringModels", "TwoFAST", "VarStructs", "VerTeX", "Winston", "Wrangling", "XLSXasJSON"], vs = ":release-1.4")

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here. cc @maleadt

@KristofferC KristofferC merged commit 9f6bed6 into release-1.5 May 27, 2020
@KristofferC KristofferC deleted the backports-release-1.5 branch May 27, 2020 15:34
@KristofferC KristofferC changed the title WIP: Backports for Julia 1.5-RC1 WIP: Backports for Julia 1.5-beta-1 Jun 1, 2020
@DilumAluthge DilumAluthge removed the DO NOT MERGE Do not merge this PR! label Jun 18, 2021
vchuravy pushed a commit to JuliaLang/Distributed.jl that referenced this pull request Oct 6, 2023
…ease-1.5

WIP: Backports for Julia 1.5-RC1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs nanosoldier run This PR should have benchmarks run on it needs pkgeval Tests for all registered packages should be run with this change release Release management and versioning.
Projects
None yet