[Met_help] [rt.rap.ucar.edu #97978] History for StatAnalysis when using MakePlots
George McCabe via RT
met_help at ucar.edu
Mon Dec 21 15:14:40 MST 2020
----------------------------------------------------------------
Initial Request
----------------------------------------------------------------
Hi,
I have been working on updating my METplus conf files to METplus v3.1. I'm
trying to work with running StatAnalysis followed by MakePlots. Everything
looks to be running fine for StatAnalysis, but when the code goes back
through to process the settings and pass to MakePlots the code exits out
and prints:
12/21 19:29:24.735 metplus.StatAnalysis (stat_analysis_wrapper.py:1700)
DEBUG: Preparing settings to pass to MakePlots wrapper
12/21 19:29:25.306 metplus.StatAnalysis (command_builder.py:122) ERROR:
(stat_analysis_wrapper.py:1651) No value found for lookin dir
12/21 19:29:25Z run-METplus-metplus.StatAnalysis: ERROR:
(stat_analysis_wrapper.py:1651) No value found for lookin dir
12/21 19:29:25.306 metplus.StatAnalysis (met_util.py:210) ERROR:
StatAnalysis had 1 error.
MakePlots doesn't need -lookin setting to run like StatAnalysis needs. I
know this portion of the code has got some rework done since I last worked
on it, so maybe I am missing something!
--
Mallory Row
IMSG at NOAA/NWS/NCEP/EMC
----------------------------------------------------------------
Complete Ticket History
----------------------------------------------------------------
Subject: StatAnalysis when using MakePlots
From: George McCabe
Time: Mon Dec 21 12:54:03 2020
Hi Mallory,
Hmmm, it looks like the way it is written, it will always fail when
trying
to set up for MakePlots. I thought these changes had been tested using
MakePlots, so I am confused as to how it worked then.
It looks like this line:
https://github.com/dtcenter/METplus/blob/114ec93f92284ef0b50029efa64320ab8720b49a/metplus/wrappers/stat_analysis_wrapper.py#L1650
Should also check for self.forMakePlots and not report an error if it
is
True.
You could try changing that line to:
if not self.forMakePlots and not runtime_settings_dict['LOOKIN_DIR']:
To see if that fixes it. Let me know what you think.
- George
On Mon, Dec 21, 2020 at 12:40 PM Mallory Row - NOAA Affiliate via RT <
met_help at ucar.edu> wrote:
>
> Mon Dec 21 12:40:38 2020: Request 97978 was acted upon.
> Transaction: Ticket created by mallory.row at noaa.gov
> Queue: met_help
> Subject: StatAnalysis when using MakePlots
> Owner: Nobody
> Requestors: mallory.row at noaa.gov
> Status: new
> Ticket <URL:
https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
>
>
> Hi,
>
> I have been working on updating my METplus conf files to METplus
v3.1. I'm
> trying to work with running StatAnalysis followed by MakePlots.
Everything
> looks to be running fine for StatAnalysis, but when the code goes
back
> through to process the settings and pass to MakePlots the code exits
out
> and prints:
>
> 12/21 19:29:24.735 metplus.StatAnalysis
(stat_analysis_wrapper.py:1700)
> DEBUG: Preparing settings to pass to MakePlots wrapper
>
> 12/21 19:29:25.306 metplus.StatAnalysis (command_builder.py:122)
ERROR:
> (stat_analysis_wrapper.py:1651) No value found for lookin dir
>
> 12/21 19:29:25Z run-METplus-metplus.StatAnalysis: ERROR:
> (stat_analysis_wrapper.py:1651) No value found for lookin dir
>
> 12/21 19:29:25.306 metplus.StatAnalysis (met_util.py:210) ERROR:
> StatAnalysis had 1 error.
>
> MakePlots doesn't need -lookin setting to run like StatAnalysis
needs. I
> know this portion of the code has got some rework done since I last
worked
> on it, so maybe I am missing something!
>
> --
>
> Mallory Row
>
> IMSG at NOAA/NWS/NCEP/EMC
>
>
--
George McCabe - Software Engineer III
National Center for Atmospheric Research
Research Applications Laboratory
303-497-2768
---
My working day may not be your working day. Please do not feel obliged
to
reply to this email outside of your normal working hours.
------------------------------------------------
Subject: StatAnalysis when using MakePlots
From: Mallory Row - NOAA Affiliate
Time: Mon Dec 21 13:04:53 2020
Hi George,
Yup, that fixed it and got it to run through MakePlots. I have
MakePlots
pointing to some old scripts, so they didn't run, but the fix did get
the
code to proceed into MakePlots and not exit out at that line anymore.
On Mon, Dec 21, 2020 at 2:54 PM George McCabe via RT
<met_help at ucar.edu>
wrote:
> Hi Mallory,
>
> Hmmm, it looks like the way it is written, it will always fail when
trying
> to set up for MakePlots. I thought these changes had been tested
using
> MakePlots, so I am confused as to how it worked then.
>
> It looks like this line:
>
>
>
https://github.com/dtcenter/METplus/blob/114ec93f92284ef0b50029efa64320ab8720b49a/metplus/wrappers/stat_analysis_wrapper.py#L1650
>
> Should also check for self.forMakePlots and not report an error if
it is
> True.
>
> You could try changing that line to:
>
> if not self.forMakePlots and not
runtime_settings_dict['LOOKIN_DIR']:
>
> To see if that fixes it. Let me know what you think.
>
> - George
>
> On Mon, Dec 21, 2020 at 12:40 PM Mallory Row - NOAA Affiliate via RT
<
> met_help at ucar.edu> wrote:
>
> >
> > Mon Dec 21 12:40:38 2020: Request 97978 was acted upon.
> > Transaction: Ticket created by mallory.row at noaa.gov
> > Queue: met_help
> > Subject: StatAnalysis when using MakePlots
> > Owner: Nobody
> > Requestors: mallory.row at noaa.gov
> > Status: new
> > Ticket <URL:
https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
> >
> >
> > Hi,
> >
> > I have been working on updating my METplus conf files to METplus
v3.1.
> I'm
> > trying to work with running StatAnalysis followed by MakePlots.
> Everything
> > looks to be running fine for StatAnalysis, but when the code goes
back
> > through to process the settings and pass to MakePlots the code
exits out
> > and prints:
> >
> > 12/21 19:29:24.735 metplus.StatAnalysis
(stat_analysis_wrapper.py:1700)
> > DEBUG: Preparing settings to pass to MakePlots wrapper
> >
> > 12/21 19:29:25.306 metplus.StatAnalysis (command_builder.py:122)
ERROR:
> > (stat_analysis_wrapper.py:1651) No value found for lookin dir
> >
> > 12/21 19:29:25Z run-METplus-metplus.StatAnalysis: ERROR:
> > (stat_analysis_wrapper.py:1651) No value found for lookin dir
> >
> > 12/21 19:29:25.306 metplus.StatAnalysis (met_util.py:210) ERROR:
> > StatAnalysis had 1 error.
> >
> > MakePlots doesn't need -lookin setting to run like StatAnalysis
needs. I
> > know this portion of the code has got some rework done since I
last
> worked
> > on it, so maybe I am missing something!
> >
> > --
> >
> > Mallory Row
> >
> > IMSG at NOAA/NWS/NCEP/EMC
> >
> >
>
> --
> George McCabe - Software Engineer III
> National Center for Atmospheric Research
> Research Applications Laboratory
> 303-497-2768
> ---
> My working day may not be your working day. Please do not feel
obliged to
> reply to this email outside of your normal working hours.
>
>
--
Mallory Row
IMSG at NOAA/NWS/NCEP/EMC
------------------------------------------------
Subject: StatAnalysis when using MakePlots
From: George McCabe
Time: Mon Dec 21 15:14:33 2020
Great! I just added the fix to develop so it will be available in
future versions. I'm closing the ticket but let me know if you notice
anything else wrong with the wrapper.
Thanks,
George
On Mon Dec 21 13:04:53 2020, mallory.row at noaa.gov wrote:
> Hi George,
>
> Yup, that fixed it and got it to run through MakePlots. I have
> MakePlots
> pointing to some old scripts, so they didn't run, but the fix did
get
> the
> code to proceed into MakePlots and not exit out at that line
anymore.
>
> On Mon, Dec 21, 2020 at 2:54 PM George McCabe via RT
> <met_help at ucar.edu>
> wrote:
>
> > Hi Mallory,
> >
> > Hmmm, it looks like the way it is written, it will always fail
when
> > trying
> > to set up for MakePlots. I thought these changes had been tested
> > using
> > MakePlots, so I am confused as to how it worked then.
> >
> > It looks like this line:
> >
> >
> >
https://github.com/dtcenter/METplus/blob/114ec93f92284ef0b50029efa64320ab8720b49a/metplus/wrappers/stat_analysis_wrapper.py#L1650
> >
> > Should also check for self.forMakePlots and not report an error if
it
> > is
> > True.
> >
> > You could try changing that line to:
> >
> > if not self.forMakePlots and not
runtime_settings_dict['LOOKIN_DIR']:
> >
> > To see if that fixes it. Let me know what you think.
> >
> > - George
> >
> > On Mon, Dec 21, 2020 at 12:40 PM Mallory Row - NOAA Affiliate via
RT
> > <
> > met_help at ucar.edu> wrote:
> >
> > >
> > > Mon Dec 21 12:40:38 2020: Request 97978 was acted upon.
> > > Transaction: Ticket created by mallory.row at noaa.gov
> > > Queue: met_help
> > > Subject: StatAnalysis when using MakePlots
> > > Owner: Nobody
> > > Requestors: mallory.row at noaa.gov
> > > Status: new
> > > Ticket <URL:
> > > https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
> > >
> > >
> > > Hi,
> > >
> > > I have been working on updating my METplus conf files to METplus
> > > v3.1.
> > I'm
> > > trying to work with running StatAnalysis followed by MakePlots.
> > Everything
> > > looks to be running fine for StatAnalysis, but when the code
goes
> > > back
> > > through to process the settings and pass to MakePlots the code
> > > exits out
> > > and prints:
> > >
> > > 12/21 19:29:24.735 metplus.StatAnalysis
> > > (stat_analysis_wrapper.py:1700)
> > > DEBUG: Preparing settings to pass to MakePlots wrapper
> > >
> > > 12/21 19:29:25.306 metplus.StatAnalysis (command_builder.py:122)
> > > ERROR:
> > > (stat_analysis_wrapper.py:1651) No value found for lookin dir
> > >
> > > 12/21 19:29:25Z run-METplus-metplus.StatAnalysis: ERROR:
> > > (stat_analysis_wrapper.py:1651) No value found for lookin dir
> > >
> > > 12/21 19:29:25.306 metplus.StatAnalysis (met_util.py:210) ERROR:
> > > StatAnalysis had 1 error.
> > >
> > > MakePlots doesn't need -lookin setting to run like StatAnalysis
> > > needs. I
> > > know this portion of the code has got some rework done since I
last
> > worked
> > > on it, so maybe I am missing something!
> > >
> > > --
> > >
> > > Mallory Row
> > >
> > > IMSG at NOAA/NWS/NCEP/EMC
> > >
> > >
> >
> > --
> > George McCabe - Software Engineer III
> > National Center for Atmospheric Research
> > Research Applications Laboratory
> > 303-497-2768
> > ---
> > My working day may not be your working day. Please do not feel
> > obliged to
> > reply to this email outside of your normal working hours.
> >
> >
------------------------------------------------
More information about the Met_help
mailing list