Parser improvement for lp files

JWally / jsLPSolver

Simple OOP javaScript library to solve linear programs, and mixed integer linear programs

The Unlicense

420 stars 69 forks source link

Parser improvement for lp files #35

Closed lvenerosy closed 8 years ago

lvenerosy commented 8 years ago

Slightly modified Reformat.js to be able to load files from lp_solve :

the comment lines containing "int" in them are now ignored
if there is a colon in a constraint line, everything up to it is ignored

JWally commented 8 years ago

Sorry I've been absentee on this. For what it's worth, I built this piece of the library as a quick and dirty add on to build out LPs for LP_Solve to look for options arbitrage opportunities (this is where spy_spy_spy comes from). I probably shouldn't have let it out in the wild without doing a lot more testing on it.

I'd love to have this functionality, but its untested at the moment. Do you think it would make sense to remove it off the master branch until it can get some time? Thoughts?

bchevalier commented 8 years ago

@JWally , what is the functionality you are talking about? to_JSON or from_JSON? With this PR it seems that to_JSON works properly (we haven't done extensive testing though).

JWally commented 8 years ago

That's great! Never mind then :-)

Sent from my iPhone

On Mar 3, 2016, at 8:27 PM, Brice notifications@github.com wrote:

@JWally , what is the functionality you are talking about? to_JSON or from_JSON? With this PR it seems that to_JSON works properly (we haven't done extensive testing though).

— Reply to this email directly or view it on GitHub.