sbromberger / LightGraphs.jl

An optimized graphs package for the Julia programming language
Other
672 stars 185 forks source link

File Missing in Benchmark Suite? #1526

Closed olynch closed 3 years ago

olynch commented 3 years ago

In the benchmark suite, there are these lines of code that open "test/testdata/graph-5k-50k.jgz":

testdatadir = joinpath(dirname(@__FILE__), "..", "test", "testdata")
benchdatadir = joinpath(dirname(@__FILE__), "data")
paramsfile = joinpath(benchdatadir, "params.jld")

println("testdatadir = $testdatadir")
println("paramsfile = $paramsfile")

dg1fn = joinpath(testdatadir, "graph-5k-50k.jgz")

DIGRAPHS = Dict{String,DiGraph}(
    "complete100"   => complete_digraph(100),
    "5000-50000"    => LightGraphs.load(dg1fn)["graph-5000-50000"],
    "path500"       => path_digraph(500)
)

The problem is, there is no such file. What is going on here?

sbromberger commented 3 years ago

We don't really use the benchmark suite for anything (especially these days; it was left over from a pre-1.0 effort). What are you trying to do, exactly?

olynch commented 3 years ago

Well, I was copying off of it for benchmarks for Catlab; there isn't really a need for that specific file, but I thought you all might want to know that it was missing.

sbromberger commented 3 years ago

Thanks. We should probably just remove the benchmarking directory. @jpfairbanks - thoughts?

jpfairbanks commented 3 years ago

I'm pro-benchmarking in general and I'd rather see the benchmarks get integrated into CI. But that requires effort.

sbromberger commented 3 years ago

I am too, and I would too, but I don't have the cycles right now. If we can't find a taker, I think it'd be better to remove these from the master branch.

sbromberger commented 3 years ago

Closing since it appears we have movement on this! #1529