search

mlubin / ReverseDiffSparse.jl

Reverse-mode automatic differentiation for sparse Hessians
Other
24 stars 12 forks source link

support exporting to flat expressions #3

Closed mlubin closed 10 years ago

mlubin commented 10 years ago

Prepping for MPB expression output from JuMP. @IainNZ @joehuchette, would this be useful for printing, or did we want to keep the original structure of the expression?

coveralls commented 10 years ago

Coverage Status

Coverage decreased (-0.11%) when pulling 6fa982148f5db4469f5b837bb181589ae10170de on flatexpr into c49758716776fe86f8739d4b1d41e90b60da327a on master.

coveralls commented 10 years ago

Coverage Status

Coverage decreased (-0.11%) when pulling 6fa982148f5db4469f5b837bb181589ae10170de on flatexpr into c49758716776fe86f8739d4b1d41e90b60da327a on master.

joehuchette commented 10 years ago

It definitely looks useful for printing, but it's not clear to me why the variable names are getting stripped out and replaced by x.

mlubin commented 10 years ago

ReverseDiffSparse doesn't know about variable names, it just sees linear indices.

mlubin commented 10 years ago

I guess we'd need to translate these back to JuMP variable names.

IainNZ commented 10 years ago

Could be useful, but this strips the sum{} right?

mlubin commented 10 years ago

Yes