Skip to content
This repository has been archived by the owner on May 21, 2022. It is now read-only.

Conflicting exports between StochasticOptimization.Iteration and LearnBase #14

Open
rdeits opened this issue May 4, 2017 · 2 comments

Comments

@rdeits
Copy link
Contributor

rdeits commented May 4, 2017

The Iteration sub-module exports an ObsIterator type that clashes with LearnBase. This results in the unit tests currently failing with:

julia> Pkg.test("StochasticOptimization")
INFO: Computing test dependencies for StochasticOptimization...
INFO: Installing LearningStrategies v0.0.1
INFO: Installing UnicodePlots v0.2.3
INFO: Building Rmath
INFO: Testing StochasticOptimization
WARNING: both Iteration and LearnBase export "ObsIterator"; uses of it in module StochasticOptimization must be qualified
ERROR: LoadError: LoadError: UndefVarError: ObsIterator not defined
 in include_from_node1(::String) at ./loading.jl:488 (repeats 2 times)
 in macro expansion; at ./none:2 [inlined]
 in anonymous at ./<missing>:?
 in eval(::Module, ::Any) at ./boot.jl:234
 in process_options(::Base.JLOptions) at ./client.jl:242
 in _start() at ./client.jl:321
while loading /home/rdeits/.julia/v0.5/StochasticOptimization/src/gradients/gradients.jl, in expression starting on line 24
while loading /home/rdeits/.julia/v0.5/StochasticOptimization/src/StochasticOptimization.jl, in expression starting on line 66
ERROR: LoadError: Failed to precompile StochasticOptimization to /home/rdeits/.julia/lib/v0.5/StochasticOptimization.ji.
 in compilecache(::String) at ./loading.jl:593
 in require(::Symbol) at ./loading.jl:422
 in include_from_node1(::String) at ./loading.jl:488
 in process_options(::Base.JLOptions) at ./client.jl:265
 in _start() at ./client.jl:321
while loading /home/rdeits/.julia/v0.5/StochasticOptimization/test/runtests.jl, in expression starting on line 1
@Evizero
Copy link
Member

Evizero commented May 4, 2017

Oh yea. This is pre-MLDataPattern. Tom prototyped some of his data access pattern suggestions here. What needs to be done to get this running again is to switch from the custom pattern here https://github.com/JuliaML/StochasticOptimization.jl/blob/master/src/iteration.jl to MLDataPattern.

@rdeits
Copy link
Contributor Author

rdeits commented May 5, 2017

Ok, thanks. I've started doing some of that work over at #15 but it's slow going.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants