-
Notifications
You must be signed in to change notification settings - Fork 16
Ability to generate planar metric terms and addition of test_eta.py unit test
#65
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
Changes from 12 commits
1bda968
b18880b
104076f
ba28f33
20ed73d
9cafe2e
9cb8f14
a0c4992
10942b2
712a385
37872f0
c76a486
1082b62
8a236a7
71bcb56
3649691
1102b75
1318b56
49ebd43
a1010dd
069f056
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -297,12 +297,34 @@ def __init__( | |
| self._dy_center = None | ||
| self._area = None | ||
| self._area_c = None | ||
| ( | ||
| self._ks, | ||
| self._ptop, | ||
| self._ak, | ||
| self._bk, | ||
| ) = self._set_hybrid_pressure_coefficients(eta_file, ak, bk) | ||
| if eta_file != "None": | ||
| ( | ||
| self._ks, | ||
| self._ptop, | ||
| self._ak, | ||
| self._bk, | ||
| ) = self._set_hybrid_pressure_coefficients(eta_file, ak, bk) | ||
| else: | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this the behavior we want? I thought we'd rather raise a ValueError or something here?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A ValueError is raised deeper in when the file specified is either non-existent, and even lower if the values do not fit the size passed in the config. My thought was that we could still have hybrid pressure values; zero valued if no file passed, or specified when passed. |
||
| self._ks = self.quantity_factory.zeros( | ||
| [], | ||
| "", | ||
| dtype=Float, | ||
| ) | ||
| self._ptop = self.quantity_factory.zeros( | ||
| [], | ||
| "Pa", | ||
| dtype=Float, | ||
| ) | ||
| self._ak = self.quantity_factory.zeros( | ||
| [Z_INTERFACE_DIM], | ||
| "Pa", | ||
| dtype=Float, | ||
| ) | ||
| self._bk = self.quantity_factory.zeros( | ||
| [Z_INTERFACE_DIM], | ||
| "", | ||
| dtype=Float, | ||
| ) | ||
| self._ec1 = None | ||
| self._ec2 = None | ||
| self._ew1 = None | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.