Skip to content
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

missing cost types #140

Open
drchrislewis opened this issue Apr 16, 2019 · 0 comments
Open

missing cost types #140

drchrislewis opened this issue Apr 16, 2019 · 0 comments

Comments

@drchrislewis
Copy link
Member

Cost_function string2CostType(std::string& cost_type_str)

@d-walsh pointed out that at least one cost type was removed in error. We need to verify that all cost types have been added to this helper function. I believe several cost have been added. Many costs implemented are unnecessary. We should clean it up to make it easier for people to know which cost to use when. For example, nobody I know of has used the library to calibrate the target itself. Therefore only the PK type (point locations known) are used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant