[Met_help] [rt.rap.ucar.edu #97978] History for StatAnalysis when using MakePlots
George McCabe via RT
met_help at ucar.edu
Tue Dec 22 12:36:11 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.
> >
> >
------------------------------------------------
Subject: StatAnalysis when using MakePlots
From: Mallory Row - NOAA Affiliate
Time: Tue Dec 22 05:51:17 2020
Would it be possible to get it in a big fix version, so we can have
NCO
install this version?
On Mon, Dec 21, 2020 at 5:14 PM George McCabe via RT
<met_help at ucar.edu>
wrote:
> 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.
> > >
> > >
>
>
>
>
--
Mallory Row
IMSG at NOAA/NWS/NCEP/EMC
------------------------------------------------
Subject: StatAnalysis when using MakePlots
From: George McCabe
Time: Tue Dec 22 06:15:16 2020
Great timing! I was actually planning on cutting a bugfix release for
3.1.
I'll make sure to get this change into the release and let you know
when it
is ready.
- George
On Tue, Dec 22, 2020 at 5:51 AM Mallory Row - NOAA Affiliate via RT <
met_help at ucar.edu> wrote:
>
> <URL: https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
>
> Would it be possible to get it in a big fix version, so we can have
NCO
> install this version?
>
> On Mon, Dec 21, 2020 at 5:14 PM George McCabe via RT
<met_help at ucar.edu>
> wrote:
>
> > 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.
> > > >
> > > >
> >
> >
> >
> >
>
> --
>
> 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: Tue Dec 22 06:16:21 2020
Great! Thank you so much!
On Tue, Dec 22, 2020 at 8:15 AM George McCabe via RT
<met_help at ucar.edu>
wrote:
> Great timing! I was actually planning on cutting a bugfix release
for 3.1.
> I'll make sure to get this change into the release and let you know
when it
> is ready.
>
> - George
>
> On Tue, Dec 22, 2020 at 5:51 AM Mallory Row - NOAA Affiliate via RT
<
> met_help at ucar.edu> wrote:
>
> >
> > <URL: https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
> >
> > Would it be possible to get it in a big fix version, so we can
have NCO
> > install this version?
> >
> > On Mon, Dec 21, 2020 at 5:14 PM George McCabe via RT
<met_help at ucar.edu>
> > wrote:
> >
> > > 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.
> > > > >
> > > > >
> > >
> > >
> > >
> > >
> >
> > --
> >
> > 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: Tue Dec 22 06:25:29 2020
Could you review this pull request to make sure I didn't make a typo
in the
change?
https://github.com/dtcenter/METplus/pull/744
Thanks,
George
On Tue, Dec 22, 2020 at 6:16 AM Mallory Row - NOAA Affiliate via RT <
met_help at ucar.edu> wrote:
>
> <URL: https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
>
> Great! Thank you so much!
>
> On Tue, Dec 22, 2020 at 8:15 AM George McCabe via RT
<met_help at ucar.edu>
> wrote:
>
> > Great timing! I was actually planning on cutting a bugfix release
for
> 3.1.
> > I'll make sure to get this change into the release and let you
know when
> it
> > is ready.
> >
> > - George
> >
> > On Tue, Dec 22, 2020 at 5:51 AM Mallory Row - NOAA Affiliate via
RT <
> > met_help at ucar.edu> wrote:
> >
> > >
> > > <URL: https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
> > >
> > > Would it be possible to get it in a big fix version, so we can
have NCO
> > > install this version?
> > >
> > > On Mon, Dec 21, 2020 at 5:14 PM George McCabe via RT <
> met_help at ucar.edu>
> > > wrote:
> > >
> > > > 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.
> > > > > >
> > > > > >
> > > >
> > > >
> > > >
> > > >
> > >
> > > --
> > >
> > > 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
>
>
--
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: George McCabe
Time: Tue Dec 22 12:36:03 2020
Hi Mallory,
The 3.1.1 release is now available:
The METplus 3.1.1 bugfix release is now available here:
https://github.com/dtcenter/METplus/releases/tag/v3.1.1
Thanks,
George
On Tue Dec 22 06:25:29 2020, mccabe wrote:
> Could you review this pull request to make sure I didn't make a typo
> in the
> change?
>
> https://github.com/dtcenter/METplus/pull/744
>
> Thanks,
> George
>
>
> On Tue, Dec 22, 2020 at 6:16 AM Mallory Row - NOAA Affiliate via RT
<
> met_help at ucar.edu> wrote:
>
> >
> > <URL: https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978 >
> >
> > Great! Thank you so much!
> >
> > On Tue, Dec 22, 2020 at 8:15 AM George McCabe via RT
> > <met_help at ucar.edu>
> > wrote:
> >
> > > Great timing! I was actually planning on cutting a bugfix
release
> > > for
> > 3.1.
> > > I'll make sure to get this change into the release and let you
know
> > > when
> > it
> > > is ready.
> > >
> > > - George
> > >
> > > On Tue, Dec 22, 2020 at 5:51 AM Mallory Row - NOAA Affiliate via
RT
> > > <
> > > met_help at ucar.edu> wrote:
> > >
> > > >
> > > > <URL: https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=97978
>
> > > >
> > > > Would it be possible to get it in a big fix version, so we can
> > > > have NCO
> > > > install this version?
> > > >
> > > > On Mon, Dec 21, 2020 at 5:14 PM George McCabe via RT <
> > met_help at ucar.edu>
> > > > wrote:
> > > >
> > > > > 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.
> > > > > > >
> > > > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > >
> > > > --
> > > >
> > > > 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
> >
> >
------------------------------------------------
More information about the Met_help
mailing list