-
Notifications
You must be signed in to change notification settings - Fork 4
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
Guatemala May initial setup #444
base: master
Are you sure you want to change the base?
Conversation
ah yes we are still setting that up.
…On Thu, Aug 29, 2024 at 3:04 PM Jeff Turmelle ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In fbfmaproom/fbfmaproom-sample.yaml
<#444 (comment)>
:
> + colormap: precip
+ lower_is_worse: yes
+ enso_state: *enso_3mo
+ bad-years:
+ label: Reported bad years
+ path: guatemala/bad-years-aso-dummy.zarr
+ var_names:
+ value: bad
+ time: T
+ colormap: precip
+ lower_is_worse: no
+ format: bad
+ vhi:
+ label: VHI
+ description: May Vegetation Health Index from NESDIS STAR
+ path: guatemala/vhi-may.zarr
sorry, not this file. it's the next one: guatemala/prcp-may-v1.zarr
—
Reply to this email directly, view it on GitHub
<#444 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQXM72A5LHJZQVRLHAGPYL3ZT5WC3AVCNFSM6AAAAABNLARWIKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDENRZHA3DQNBYHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Max Mauerman
*Please note that I am migrating my email to ***@***.***
Please send any new emails there.*
Senior Staff Associate, Financial Instruments Sector Team
National Center for Disaster Preparedness (NCDP)
Affiliate of the International Research Institute for Climate and Society
Columbia University Climate School
Profile <https://people.climate.columbia.edu/users/profile/max-mauerman>
| Google
Scholar <https://scholar.google.com/citations?user=i4gF_30AAAAJ&hl=en> |
Linkedin <https://www.linkedin.com/in/max-mauerman-76713b9b/> | ORCID
<https://orcid.org/0000-0002-5314-9082>
|
OK then, I should probably wait until you test it before I approve... |
( | ||
# Static (non-updating) snapshot of | ||
# http://iridl.ldeo.columbia.edu/SOURCES/.NOAA/.NESDIS/.STAR/.VHP/.16km/.VHI | ||
# which is doubly broken: the upstram URL has changed, plus they had a |
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.
upstram --> upstream, here and wherever you copied it from.
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.
Hi @aaron-kaplan , do you know what is left or missing to approve this pull request?
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.
It looks like @jturmelle reviewed this while I was away, so I'll pass the question to him.
I will point out, however, that you're replying to a comment where I pointed out a typo, and the typos still hasn't been fixed.
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.
My comment was the files they were pointing to didn't exist, so they couldn't have tested it yet in development mode. I was waiting for them to state the testing was OK before approving and merging.
Initial commit for Guatemala May forecast maproom