Skip to content

Commit

Permalink
making the words better
Browse files Browse the repository at this point in the history
  • Loading branch information
robfalck committed May 17, 2024
1 parent 4c4249d commit 5691a80
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion POEM_097.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ For instance, you might have a model parameter associated with physical units bu
### Why adding a `units` argument to OptionsDictionary declare_options won't work.

Some users have experiemented with passing tuples as options, where the second value contains units.
This isn't ideal since another unit may expect a tuple that doesn't involve units, leading to ambiguity.
This isn't ideal since another option may expect a tuple that doesn't involve units, leading to ambiguity.

We could implement a set_val/get_val with units, as we do with OpenMDAO vectors. The existing `OptionsDictionary.set(**kwargs)` and `OptionsDictionary.temporary(**kwargs)` are not compatible with this, nor is the abiltiy to pass options as keyword arguments to to OpenMDAO Systems.

Expand Down

0 comments on commit 5691a80

Please sign in to comment.