Conversation
|
Hm, interesting. Tests pass in non-debug mode. In gdb on debug mode, I get: Any ideas? |
|
Looks similar to #21710. I'll take a look. |
fix method specificity issue causing problems for pr #21750
|
LGTM. |
|
Just to make sure: @nanosoldier |
|
So the benchmarks apparently make use of partial linear indexing: |
|
Bump. |
This removes the partial linear indexing deprecation and implents the new behavior: Linear indexing now only takes effect when there is exactly one non-cartesian index.
Needed to test performance of JuliaLang/julia#21750.
|
Looking through recent BaseBenchmarkReports, it looks like JuliaCI/BaseBenchmarks.jl#106 will be sufficient to fix up the Nanosoldier with this branch. |
|
Hm, this is ending up with a segfault in one of the codegen tests… I trapped it with gdb but all I can see is that it's ending up trying to access null |
|
@nanosoldier |
|
Hmmmm. @nanosoldier |
|
Yeah, those failures are still expected. I forgot that this depended upon JuliaCI/BaseBenchmarks.jl#106. |
* Remove dependence on partial linear indexing Needed to test performance of JuliaLang/julia#21750. * Bump Compat dependency to 0.31.0
|
@nanosoldier |
|
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @ararslan |
|
Thanks so much for shepherding those changes into Nanosoldier, @ararslan! This is looking pretty good now. The FreeBSD failure is in a random fuzzing block within test/ranges.jl — have we seen that before? I managed to reproduce it locally on a different branch by increasing the number of iterations in that loop. I'll try to narrow it down and open an issue. I'd like to merge this soon. |
This removes the partial linear indexing deprecation and implents the new behavior: Linear indexing now only takes effect when there is exactly one non-cartesian index.
I know we've typically held off on removing deprecation until the mid-end of the release cycle, but this is a more complicated removal than typical, and several features that I'd like to see in this release depend on or are much simpler with this deprecation gone.