TulipaEnergy / TulipaIO.jl

Apache License 2.0
1 stars 4 forks source link

fill argument has ill-defined type / code doesn't work if fill is Vector #32

Closed abelsiqueira closed 6 months ago

abelsiqueira commented 6 months ago

https://github.com/TulipaEnergy/TulipaIO.jl/blob/e5c2be1925dc2522f0f83b05521d375c98ed9c9b/src/fmtsql.jl#L50

The Vector::Any becomes Vector, so that is ignored. But most important, the mixed type logic doesn't work. If fill is a Vector, then the code will fail because if fill doesn't work - i.e., Vector is not a Bool.

I think a better alternative would be to have another keyword argument to pass the Vector information.