-
Notifications
You must be signed in to change notification settings - Fork 30
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
Support Level III Super Res #73
Conversation
color table for 256 level reflectivity
@akrherz I didn't realize that table was there. My radar composites are running with an older version of GEMPAK that didn't have that upc_ref2.tbl. If it's identical, that's fine. But you'll have to update the line in gempak/tables/sat/imgtyp.tbl as well, since it's using my file name. |
I think I am done tweaking for now, anybody wish to review this? @mzuranski @pjpokran ? |
I'm following along with these changes, trying to adjust my 7.5.1 install, but I'm still seeing IMCBAR frequency ignored. This image was made with IMCBAR = 31/V/CR/.98;.5/.84;.02/-10 The only other difference I can think of is I'm running Ubuntu but I don't see how that could be related. Can others verify if IMCBAR frequency works for them or not on 7.14? |
I get the same thing, nevermind my comment about frequency. I am looking into this more. |
Ah, |
|
Daryl, this all looks good to me. I haven't tested any of your other changes since I'm not using other angles, but on paper they all look good. Thanks for all the additional work on this! |
So, of concern now is that GEMPAK seems to think N0Q starts at -32 dbz and N0B at -33 dbz. Similar off by one knot with N0U and N0G, I've hacked around that now. |
I'm done tinkering with this PR and will follow up with others as I find things to fix :/ |
Just curious, what command syntax did you use to make that national mosaic? |
Just nex2img |
Gilbert - I'm writing up an email to another explaining how I do the mosiacs - when I'm done I can forward that and the script/etc to you. |
Great, thanks! |
I don't know of further feedback coming on this and can easily followup with issues folks find with any testing that comes. I have some more cosmetic things to tweak overtop this PR, so want to get it in now to keep merge nightmares limited. |
@akrherz Since this is going to get closed out soon, how would you feel if I opened a new issue for imcbar frequency? The 7.5.1 documentation states:
...and that's it. Granted those docs are now obsolete, they're still the best we've got (as far as I'm aware). While I'm not expecting a change to the behavior anytime soon, I would at least like it noted someplace that the frequency value is ignored for certain products as to avoid confusion. I'll also close out #15 as it's old and probably related to this anyway. |
Yes please, I was going to do the same, but it slipped my mind yesterday. |
refs #72
obsoletes #71
Pull request that implements code/table changes necessary to support NEXRAD Level III super res. Thanks to @pjpokran for getting this PR off to an excellent start!