pytorch_mppi icon indicating copy to clipboard operation
pytorch_mppi copied to clipboard

Update Readme: terminal_state_cost() parameter includes action

Open 28shambhavi opened this issue 1 year ago • 0 comments

The Readme says terminal_state_cost is only a function of the full state: "terminal state cost- function(state (K x T x nx)) -> cost (K x 1)"

But in file src/mppi.py, line 276: "c = self.terminal_state_cost(states, actions)" gives two inputs to the function

28shambhavi avatar Nov 26 '24 03:11 28shambhavi