-
Notifications
You must be signed in to change notification settings - Fork 270
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
Composite Z bugs (issue and fix) #1559
Comments
I have the code into the codebase locally, but need to run. I will try to get this pushed up next week. |
Do we know if the |
I feel like dropping ALL tilts that don't match the lowest Nyquist is going to be pretty hit or miss and affect vertical extent. For instance, this is for KFCX 2024-04-15 20:32Z (tilt num, angle, Nyquist):
|
<!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Aptos;
panose-1:2 11 0 4 2 2 2 2 2 4;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:12.0pt;
font-family:"Aptos",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
code
{mso-style-priority:99;
font-family:"Courier New";}
pre
{mso-style-priority:99;
mso-style-link:"HTML Preformatted Char";
margin:0in;
font-size:10.0pt;
font-family:"Courier New";}
span.HTMLPreformattedChar
{mso-style-name:"HTML Preformatted Char";
mso-style-priority:99;
mso-style-link:"HTML Preformatted";
font-family:"Consolas",serif;}
span.EmailStyle22
{mso-style-type:personal-reply;
font-family:"Aptos",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
mso-ligatures:none;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
-->This is why xradar will really help Py-ART deal with all the issues we face with non-“rectangle” radar files. From: Ryan May ***@***.***>Date: Monday, April 15, 2024 at 15:43To: ARM-DOE/pyart ***@***.***>Cc: Subscribed ***@***.***>Subject: Re: [ARM-DOE/pyart] Composite Z bugs (issue and fix) (Issue #1559)I feel like dropping ALL tilts that don't match the lowest Nyquist is going to be pretty hit or miss and affect vertical extent. For instance, this is for KFCX 2024-04-15 20:32Z (tilt num, angle, Nyquist):VCP: 2121 0.53 8.512 0.53 31.33 0.93 8.514 0.92 31.35 1.38 9.256 1.36 31.37 0.52 8.518 0.53 31.39 1.82 31.310 2.43 31.311 3.10 31.312 3.99 31.313 5.10 31.314 6.41 31.3—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
@dopplershift I agree, its a bit strong. But clearly from this case, things are getting weird in that doppler scan. Should the default have the Or should we apply some default gatefilter that cleans up the ugly data? |
I'm curious to look at the sweeps and see just what's going on there... |
Ok, so it looks like the SZ-2 phase coding for the range ambiguity resolution on the velocity cuts results in some kind of power bias. I'm rusty on my phase coding, but I don't remember that being a problem, but, it's clearly there, whatever. The problem with your approach is that, for instance on this volume, it's only going to use the 0.5 and 0.9 cuts, nothing else:
To me that's not a great composite Z product. There's also a general "issue" that needs to be dealt with (not sure if it is here) where SAILS can put another 0.5 cut in the volume--not sure whether that should be included. Options I can think of:
|
@dopplerchase - if the nyquist velocity is in the instrument parameters, it will work! The method is implemented at the Line 609 in db95df7
|
In my original composite reflectivity code, I neglected that different sweeps can have different PRFs, which change the sensitivity and range. This created some odd artifacts like here:
To get around this, I have adjusted the composite reflectivity code to include a quick np.where call to find all the sweeps with the same nyquist as the lowest sweep. This gets rid of the hard artifacts:
I have added a bool flag to turn this on/off (not sure why someone would want to turn this off, but its there if people do).
I would guess in the future i should build it so you can select the specific nyquist, but for now this fix works.
The text was updated successfully, but these errors were encountered: