sbromberger / LightGraphs.jl

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

[PkgEval] LightGraphs may have a testing issue on Julia 0.4 (2015-06-30) #74

Closed IainNZ closed 9 years ago

IainNZ commented 9 years ago

PackageEvaluator.jl is a script that runs nightly. It attempts to load all Julia packages and run their tests (if available) on both the stable version of Julia (0.3) and the nightly build of the unstable version (0.4). The results of this script are used to generate a package listing enhanced with testing results.

On Julia 0.4

This issue was filed because your testing status became worse. No additional issues will be filed if your package remains in this state, and no issue will be filed if it improves. If you'd like to opt-out of these status-change messages, reply to this message saying you'd like to and @IainNZ will add an exception. If you'd like to discuss PackageEvaluator.jl please file an issue at the repository. For example, your package may be untestable on the test machine due to a dependency - an exception can be added.

Test log:

>>> 'Pkg.add("LightGraphs")' log
INFO: Cloning cache of LightGraphs from git://github.com/JuliaGraphs/LightGraphs.jl.git
INFO: Installing DataStructures v0.3.10
INFO: Installing Distributions v0.7.3
INFO: Installing LightGraphs v0.1.14
INFO: Installing LightXML v0.1.11
INFO: Installing PDMats v0.3.3
INFO: Building LightXML
INFO: Package database updated

>>> 'Pkg.test("LightGraphs")' log
Julia Version 0.4.0-dev+5700
Commit 147fa0b* (2015-06-29 20:31 UTC)
Platform Info:
  System: Linux (x86_64-unknown-linux-gnu)
  CPU: Intel(R) Xeon(R) CPU E5-2650 0 @ 2.00GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Nehalem)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3
INFO: Testing LightGraphs
Warning: could not import Base.@math_const into Distributions
ERROR: LoadError: LoadError: LoadError: LoadError: UndefVarError: @math_const not defined
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:133
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:133
 in reload_path at ./loading.jl:157
 in _require at ./loading.jl:69
 in require at ./loading.jl:55
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:133
 in reload_path at ./loading.jl:157
 in _require at ./loading.jl:69
 in require at ./loading.jl:52
 in include at ./boot.jl:254
 in include_from_node1 at loading.jl:133
 in process_options at ./client.jl:305
 in _start at ./client.jl:405
while loading /home/vagrant/.julia/v0.4/Distributions/src/constants.jl, in expression starting on line 6
while loading /home/vagrant/.julia/v0.4/Distributions/src/Distributions.jl, in expression starting on line 252
while loading /home/vagrant/.julia/v0.4/LightGraphs/src/LightGraphs.jl, in expression starting on line 5
while loading /home/vagrant/.julia/v0.4/LightGraphs/test/runtests.jl, in expression starting on line 2
=============================[ ERROR: LightGraphs ]=============================

failed process: Process(`/home/vagrant/julia/bin/julia --check-bounds=yes --code-coverage=none --color=no /home/vagrant/.julia/v0.4/LightGraphs/test/runtests.jl`, ProcessExited(1)) [1]

================================================================================
INFO: No packages to install, update or remove
ERROR: LightGraphs had test errors
 in error at ./error.jl:21
 in test at pkg/entry.jl:746
 in anonymous at pkg/dir.jl:31
 in cd at file.jl:22
 in cd at pkg/dir.jl:31
 in test at pkg.jl:71
 in process_options at ./client.jl:281
 in _start at ./client.jl:405

>>> End of log
IainNZ commented 9 years ago

https://github.com/JuliaStats/Distributions.jl/issues/388

sbromberger commented 9 years ago

See also https://groups.google.com/d/msg/julia-dev/vkq7sW0JrSU/4sd36dqav2MJ

Why are some packages passing tests and some failing?

(Ah, is it a Distributions.jl thing?)

IainNZ commented 9 years ago

Yeah its Distributions, MathConst became Irrational in Base.

sbromberger commented 9 years ago

Gah. The only reason I was using Distributions was for sample() which is in StatsBase, so I guess I can switch. I wish there were a performant sampler built into Base, though.

IainNZ commented 9 years ago

Living on the edge is tough, thats why I don't use 0.4!

sbromberger commented 9 years ago

Yeah, looks like round() changed somewhere as well. Travis is spewing

ERROR: LoadError: LoadError: test error in expression: round((pagerank(g5))[3],3) == 0.318
DimensionMismatch("")
sbromberger commented 9 years ago

All fixed, I think. Appreciate your help.