odow / SDDP.jl

A JuMP extension for Stochastic Dual Dynamic Programming
https://sddp.dev
Other
304 stars 62 forks source link

documentation fix #616

Closed slwu89 closed 1 year ago

slwu89 commented 1 year ago

I noticed the code block in "Access a first-stage decision in a future stage" of this help page (https://odow.github.io/SDDP.jl/stable/guides/access_previous_variables/) needs the following changes to run when using SDDP.train on the resulting model:

  1. capcity variable name misspelled when defined in @variable macro
  2. capacity variable needs to be constrained to be non-negative
  3. current version of SDDP needs a upper_bound defined when the sense is :Max
odow commented 1 year ago

I should update these so that the code actually gets tested when the documentation builds 😆

See https://github.com/odow/SDDP.jl/pull/617