-
Notifications
You must be signed in to change notification settings - Fork 37
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
Diurnal [WIP] #510
Diurnal [WIP] #510
Conversation
…ut.py Added capability to append to json files
generate sftlf if not found works
correct output names
@durack1 @gleckler1 this is ready to be approved and go in |
@doutriaux1 should we wait for a green light with the tests? Or do we still have the precision issue causing the red crosses? |
It's green now. |
@@ -1,44 +1,44 @@ | |||
import cdutil | |||
|
|||
regions_specs = { | |||
'NHEX': {'domain': cdutil.region.domain(latitude=(30., 90, 'ccb'))}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@doutriaux1 why was this truncated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea. Let me put it back in. Nice catch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@durack1 "ccb"
is not used by cdutil.domain
it can only break things and confuse newer users.
src/python/diurnal/common.py
Outdated
"--append", | ||
default=False, | ||
action="store_true", | ||
help="append in json file in json exist (e.g. adding a model to file)") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"append in json file in json exist (e.g. adding a model to file)" -> "append in json file if json exists (e.g. adding a model to an existing file)"
src/python/diurnal/fourierFFT.py
Outdated
Do NOT detrend the time series first, in order to retain the "sawtooth" frequency implied by the input length of the | ||
time series (e.g. the 24-hour period from a composite-diurnal cycle). | ||
|
||
On input: x[i, j] = values at each gridpoint (i) for N times (j), e.g. N = 8 for a 3-hr composite-diurnal cycle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra spaces in doc string
Do NOT detrend the time series first, in order to retain the "sawtooth" frequency implied by the input length of the | ||
time series (e.g. the 24-hour period from a composite-diurnal cycle). | ||
|
||
On input: x[k,i,j] = values at each gridpoint (i,j) for N times (k), e.g. N = 8 for a 3-hr composite-diurnal cycle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here - spaces
Still need to add test suite