divilian / specstar

Combines SPECscape and SPECnet into one project
1 stars 1 forks source link

Remove R `ineq` library dependency #34

Open divilian opened 5 years ago

divilian commented 5 years ago

Duplicate, for the combined specstar repo, the change from issue #11 of the specscape repo.

venkatachalapathy commented 5 years ago

Just curious. param_sweep.jl uses ineq. And I am not sure if we have time to code statistical comparison tests (see #37 ). Is there a reason to do this?

divilian commented 5 years ago

My personal motivation is to make the two sims work in the same way whenever possible. Scape used R's ineq, so we did too. Now Scape is changing to no longer use this library, so we should swtich too. (Not a high-urgency item, but we ought to do it.)

A secondard motivation is to reduce dependencies on R libraries for things that are perfectly possible in Julia. I'm okay with a trailer hitch on the back of my car, but if I've sold my bike cart, why not remove the hitch?