-
Notifications
You must be signed in to change notification settings - Fork 10
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
Generated code: specification of units #82
Comments
You did not mention |
Actually, I don't need any change in these methods in my use case, as I use my own implementation of disciplines. But for sake of consistency, I guess you may declare units in these methods as well. |
|
I override the
Currently, it does not work, that's the point 😁. But once the inputs will have been declared with proper units, OpenMDAO will gracefully do the conversion, and then, it will be great! |
Ok, I will also add units at component level (it was my concern in the first place it gets in the way if I do). And you can test right now as the feature is deployed. |
Many thanks. I will test that very soon! |
Hello,
in generated code for OpenMDAO, I would greatly appreciate if units could be specified in following methods:
set_input_defaults()
add_design_var()
add_objective()
add_constraint()
The text was updated successfully, but these errors were encountered: