-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
lambda -> discount #6
Comments
Well, I think the people in the community who use Hybrid A* tend to use lambda instead of discount. That's why I chose lambda as the variable name to make it easy to map/understand. |
They use lambda as a symbol for the discount factor, but do they still call it the discount factor, or do they call it something else? Which of the following is a more direct conceptual mapping?
If I have never read a paper that uses lambda for the discount factor, I will have to go look it up. Using mathematical symbols allows for short unambiguous reference within a paper, but if you have to spell out (To be clear, this is an opinion, so you as the maintainer of this package can choose to ignore it) |
Really, all of the arguments should be named consistently. For example,
|
Isn't
discount
a better name for the discount argument thanlambda
?The text was updated successfully, but these errors were encountered: