Skip to content

Conversation

N5N3
Copy link
Member

@N5N3 N5N3 commented Apr 17, 2025

and avoid eager UnionAll unwrapping to hit more fast path.
close #58129 (test passed locally)
close #56350 (MWE returns Tuple{Any, Any, Vararg} now.)

@N5N3 N5N3 added types and dispatch Types, subtyping and method dispatch backport 1.10 Change should be backported to the 1.10 release backport 1.11 Change should be backported to release-1.11 backport 1.12 Change should be backported to release-1.12 labels Apr 17, 2025
@vtjnash vtjnash added the needs pkgeval Tests for all registered packages should be run with this change label Apr 17, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
@N5N3
Copy link
Member Author

N5N3 commented Apr 17, 2025

@nanosoldier runtests()

@nanosoldier
Copy link
Collaborator

The package evaluation job you requested has completed - possible new issues were detected.
The full report is available.

Report summary

❗ Packages that crashed

1 packages crashed only on the current version.

  • A segmentation fault happened: 1 packages

6 packages crashed on the previous version too.

✖ Packages that failed

44 packages failed only on the current version.

  • Package has test failures: 3 packages
  • Package tests unexpectedly errored: 2 packages
  • Tests became inactive: 4 packages
  • Test duration exceeded the time limit: 35 packages

1248 packages failed on the previous version too.

✔ Packages that passed tests

12 packages passed tests only on the current version.

  • Other: 12 packages

5316 packages passed tests on the previous version too.

~ Packages that at least loaded

4 packages successfully loaded only on the current version.

  • Other: 4 packages

2937 packages successfully loaded on the previous version too.

➖ Packages that were skipped altogether

918 packages were skipped on the previous version too.

@N5N3
Copy link
Member Author

N5N3 commented Apr 19, 2025

@nanosoldier runtests(["MbedTLS", "ParaReal", "BlockArrays", "CharacteristicInvFourier", "GraphViz", "Manopt", "QuantumPropagators", "AstroForceModels", "Miter", "SDDP", "LSODA", "OrdinaryDiffEqHighOrderRK", "OrdinaryDiffEqStabilizedRK", "OrdinaryDiffEqNordsieck", "DECAES", "DrugInteractions", "OptimizationNLopt", "MCMCTempering", "MLJ", "PotentialFlow", "ComradeGalactic", "ODEProblemLibrary", "JumpProblemLibrary", "StochasticDelayDiffEq", "Makie", "PeriodicMatrices", "CRRao", "RealTimeAudioDiffEq", "FrequencySweep", "SurfaceCoverage", "PlugFlowReactor", "FunctionalModels", "ModiaLang", "DPFEHM", "Modia", "ClimaCalibrate", "EcologicalNetworksDynamics", "BoxCox", "DataDrivenEnzymeRateEqs", "GeoIO", "EarthSciData", "IntegratedOscillatorModel", "BloqadeNoisy", "QuantumAnnealingAnalytics", "Test"])

@nanosoldier
Copy link
Collaborator

The package evaluation job you requested has completed - no new issues were detected.
The full report is available.

Report summary

✖ Packages that failed

11 packages failed on the previous version too.

✔ Packages that passed tests

24 packages passed tests on the previous version too.

~ Packages that at least loaded

1 packages successfully loaded only on the current version.

  • Other: 1 packages

9 packages successfully loaded on the previous version too.

@N5N3 N5N3 removed the needs pkgeval Tests for all registered packages should be run with this change label Apr 20, 2025
@N5N3 N5N3 merged commit 334c316 into JuliaLang:master Apr 20, 2025
8 of 9 checks passed
@N5N3 N5N3 deleted the subtype-fix branch April 20, 2025 10:48
KristofferC pushed a commit that referenced this pull request Apr 22, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
close #58129 (test passed locally)
close #56350 (MWE returns `Tuple{Any, Any, Vararg}` now.)

(cherry picked from commit 334c316)
@KristofferC KristofferC mentioned this pull request Apr 22, 2025
51 tasks
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Apr 25, 2025
KristofferC pushed a commit that referenced this pull request Apr 25, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
close #58129 (test passed locally)
close #56350 (MWE returns `Tuple{Any, Any, Vararg}` now.)

(cherry picked from commit 334c316)
@KristofferC KristofferC mentioned this pull request Apr 25, 2025
71 tasks
LebedevRI pushed a commit to LebedevRI/julia that referenced this pull request May 2, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
close JuliaLang#58129 (test passed locally)
close JuliaLang#56350 (MWE returns `Tuple{Any, Any, Vararg}` now.)
anchal-physics added a commit to ProjectTorreyPines/IMASggd.jl that referenced this pull request Jun 2, 2025
project_prop_on_subset! function  has been changed so that it does not
restrict the input type anymore. The input type restriction was causing
too much overhead in compilation (potentially a julia issue) but we aren't
wining anything right now by fixing the type.

The overhead comes when during compile time the script does not know the
type of the input. This was happening in the second method which takes
a string path from the ggd array to the property. Since this string path
can be anything, the compiler goes into endless computation in pre-compile
step and gets stuck without giving any warning or errors. I suspect another
overflow issue similar to JuliaLang/julia#58129
fixed in JuliaLang/julia#58159 .
anchal-physics added a commit to ProjectTorreyPines/IMASggd.jl that referenced this pull request Jun 2, 2025
Fixed projection function on ggd array

project_prop_on_subset! function has been changed so that it does not restrict the input type anymore. The input type restriction was causing too much overhead in compilation (potentially a julia issue), but we aren't winning anything right now by fixing the type.

The overhead comes when, during compile time, the script does not know the type of the input. This was happening in the second method, which takes a string path from the ggd array to the property. Since this string path can be anything, the compiler goes into endless computation inthe  pre-compile step and gets stuck without giving any warning or errors. I suspect another overflow issue similar to JuliaLang/julia#58129 fixed in JuliaLang/julia#58159 .
KristofferC pushed a commit that referenced this pull request Jun 4, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
close #58129 (test passed locally)
close #56350 (MWE returns `Tuple{Any, Any, Vararg}` now.)

(cherry picked from commit 334c316)
@KristofferC KristofferC mentioned this pull request Jun 4, 2025
75 tasks
KristofferC pushed a commit that referenced this pull request Jun 5, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
close #58129 (test passed locally)
close #56350 (MWE returns `Tuple{Any, Any, Vararg}` now.)

(cherry picked from commit 334c316)
KristofferC pushed a commit that referenced this pull request Jul 3, 2025
and avoid eager UnionAll unwrapping to hit more fast path.
close #58129 (test passed locally)
close #56350 (MWE returns `Tuple{Any, Any, Vararg}` now.)

(cherry picked from commit 334c316)
@KristofferC KristofferC removed the backport 1.11 Change should be backported to release-1.11 label Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.10 Change should be backported to the 1.10 release types and dispatch Types, subtyping and method dispatch
Projects
None yet
4 participants