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

Validate sign of inputs to T Model functions. #295

Open
davidorme opened this issue Sep 24, 2024 · 0 comments
Open

Validate sign of inputs to T Model functions. #295

davidorme opened this issue Sep 24, 2024 · 0 comments

Comments

@davidorme
Copy link
Collaborator

At the moment, there's nothing stopping people giving negative heights, dbh etc to the T Model functions. We probably should add this within the validation of the inputs to the functions of that module.

I'm not quite sure what level of user error we want to guard against though. Stuff that blows up the code, sure, but there is a runtime and code bloat cost to guarding against everything.

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

No branches or pull requests

1 participant