#3950 Add the minimal --summaryplot commandline option.

Using shell globbing for filenames, and only one additional option implemented.
Avoid creating default plot when using --summaryplot

ApplicationCode/Application/RiaApplication.cpp

@@ -289,7 +289,7 @@ bool RiaApplication::openFile(const QString& fileName)
     }
     else if (fileType & RiaDefines::ANY_ECLIPSE_FILE)
     {
-        loadingSucceded = RicImportGeneralDataFeature::openEclipseFilesFromFileNames(QStringList{fileName});
+        loadingSucceded = RicImportGeneralDataFeature::openEclipseFilesFromFileNames(QStringList{fileName}, true);
         lastUsedDialogTag = RiaDefines::defaultDirectoryLabel(fileType);
     }
 

ApplicationCode/Application/RiaConsoleApplication.cpp

@@ -259,7 +259,7 @@ RiaApplication::ApplicationStatus RiaConsoleApplication::handleArguments(cvf::Pr
     {
         QStringList fileNames = RicImportGeneralDataFeature::fileNamesFromCaseNames(cvfqt::Utils::toQStringList(o.values()));
         RicImportGeneralDataFeature::OpenCaseResults results =
-            RicImportGeneralDataFeature::openEclipseFilesFromFileNames(fileNames);
+            RicImportGeneralDataFeature::openEclipseFilesFromFileNames(fileNames, true);
     }
 
     if (cvf::Option o = progOpt->option("commandFile"))

ApplicationCode/Application/RiaGuiApplication.cpp

@@ -133,6 +133,7 @@
 #ifdef USE_UNIT_TESTS
 #include "gtest/gtest.h"
 #endif // USE_UNIT_TESTS
+#include "SummaryPlotCommands/RicSummaryPlotFeatureImpl.h"
 
 namespace caf
 {
@@ -657,6 +658,11 @@ RiaApplication::ApplicationStatus RiaGuiApplication::handleArguments(cvf::Progra
         }
     }
 
+    if (cvf::Option o = progOpt->option("summaryplot"))
+    {
+        RicSummaryPlotFeatureImpl::createSummaryPlotFromArgumentLine( cvfqt::Utils::toQStringList(o.values()));
+    }
+
     QString projectFileName;
 
     if (progOpt->hasOption("last"))
@@ -777,7 +783,7 @@ RiaApplication::ApplicationStatus RiaGuiApplication::handleArguments(cvf::Progra
         QStringList fileNames = RicImportGeneralDataFeature::fileNamesFromCaseNames(cvfqt::Utils::toQStringList(o.values()));
 
         RicImportGeneralDataFeature::OpenCaseResults results =
-            RicImportGeneralDataFeature::openEclipseFilesFromFileNames(fileNames);
+            RicImportGeneralDataFeature::openEclipseFilesFromFileNames(fileNames, true);
         if (results && !results.eclipseSummaryFiles.empty())
         {
             getOrCreateAndShowMainPlotWindow();

ApplicationCode/Application/Tools/RiaArgumentParser.cpp

@@ -72,6 +72,13 @@ bool RiaArgumentParser::parseArguments(cvf::ProgramOptions* progOpt)
     progOpt->registerOption("server", "[<portnumber>]", "Launch as a GRPC server. Default port is 50051", cvf::ProgramOptions::SINGLE_VALUE);
     progOpt->registerOption("startdir", "<folder>", "Set startup directory.\n", cvf::ProgramOptions::SINGLE_VALUE);
 
+    progOpt->registerOption("summaryplot", 
+                            "[<plotOptions>] <eclipsesummaryvectors> [<eclipsedatafiles>]",
+                            "Creates a summary plot using all the <eclipsedatafiles>," 
+                            "and all the summary vectors defined in <eclipsesummaryvectors>."
+                            "Use --summaryplot -help to show a more detailed help text.\n",
+                            cvf::ProgramOptions::OPTIONAL_MULTI_VALUE);
+
     progOpt->registerOption("commandFile", "<commandfile>", "Execute the command file.", cvf::ProgramOptions::SINGLE_VALUE);
     progOpt->registerOption("commandFileReplaceCases",
                             "[<caseId>] <caseListFile>",

ApplicationCode/Commands/RicImportGeneralDataFeature.cpp

@@ -42,7 +42,8 @@ CAF_CMD_SOURCE_INIT(RicImportGeneralDataFeature, "RicImportGeneralDataFeature");
 ///
 //--------------------------------------------------------------------------------------------------
 RicImportGeneralDataFeature::OpenCaseResults
-    RicImportGeneralDataFeature::openEclipseFilesFromFileNames(const QStringList& fileNames)
+    RicImportGeneralDataFeature::openEclipseFilesFromFileNames(const QStringList& fileNames, 
+                                                               bool doCreateDefaultPlot)
 {
     CVF_ASSERT(!fileNames.empty());
 
@@ -89,7 +90,7 @@ RicImportGeneralDataFeature::OpenCaseResults
     }
     if (!eclipseSummaryFiles.empty())
     {
-        if (!openSummaryCaseFromFileNames(eclipseSummaryFiles))
+        if (!openSummaryCaseFromFileNames(eclipseSummaryFiles, doCreateDefaultPlot))
         {
             return OpenCaseResults();
         }
@@ -200,7 +201,7 @@ void RicImportGeneralDataFeature::openFileDialog(ImportFileType fileTypes)
         RiaApplication::instance()->setLastUsedDialogDirectory(defaultDirectoryLabel(ANY_ECLIPSE_FILE), fileNames.front());
     }
 
-    if (!openEclipseFilesFromFileNames(fileNames))
+    if (!openEclipseFilesFromFileNames(fileNames, true))
     {
         RiaLogging::error(QString("Failed to open file names: %1").arg(fileNames.join(", ")));
     }
@@ -240,10 +241,11 @@ bool RicImportGeneralDataFeature::openInputEclipseCaseFromFileNames(const QStrin
 //--------------------------------------------------------------------------------------------------
 ///
 //--------------------------------------------------------------------------------------------------
-bool RicImportGeneralDataFeature::openSummaryCaseFromFileNames(const QStringList& fileNames)
+bool RicImportGeneralDataFeature::openSummaryCaseFromFileNames(const QStringList& fileNames, 
+                                                               bool doCreateDefaultPlot)
 {
     std::vector<RimSummaryCase*> newCases;
-    if (RicImportSummaryCasesFeature::createAndAddSummaryCasesFromFiles(fileNames, &newCases))
+    if (RicImportSummaryCasesFeature::createAndAddSummaryCasesFromFiles(fileNames, doCreateDefaultPlot, &newCases))
     {
         RicImportSummaryCasesFeature::addCasesToGroupIfRelevant(newCases);
         for (const RimSummaryCase* newCase : newCases)

ApplicationCode/Commands/RicImportGeneralDataFeature.h

@@ -46,7 +46,8 @@ public:
         }
     };
 
-    static OpenCaseResults openEclipseFilesFromFileNames(const QStringList& fileNames);
+    static OpenCaseResults openEclipseFilesFromFileNames(const QStringList& fileNames, 
+                                                         bool doCreateDefaultPlot);
     static QStringList fileNamesFromCaseNames(const QStringList& caseNames);
 
 protected:
@@ -60,6 +61,7 @@ protected:
 
     static bool openEclipseCaseFromFileNames(const QStringList& fileNames);
     static bool openInputEclipseCaseFromFileNames(const QStringList& fileNames);
-    static bool openSummaryCaseFromFileNames(const QStringList& fileNames);
+    static bool openSummaryCaseFromFileNames(const QStringList& fileNames, 
+                                             bool doCreateDefaultPlot = true);
 
 };

ApplicationCode/Commands/RicImportSummaryCasesFeature.cpp

@@ -122,6 +122,7 @@ void RicImportSummaryCasesFeature::setupActionLook(QAction* actionToSetup)
 ///
 //--------------------------------------------------------------------------------------------------
 bool RicImportSummaryCasesFeature::createAndAddSummaryCasesFromFiles(const QStringList&            fileNames,
+                                                                     bool doCreateDefaultPlot,
                                                                      std::vector<RimSummaryCase*>* newCases)
 {
     RiaGuiApplication* app = RiaGuiApplication::instance();
@@ -131,7 +132,7 @@ bool RicImportSummaryCasesFeature::createAndAddSummaryCasesFromFiles(const QStri
     if (createSummaryCasesFromFiles(fileNames, cases))
     {
         addSummaryCases(*cases);
-        if (!cases->empty())
+        if (!cases->empty() && doCreateDefaultPlot)
         {
             RicSummaryPlotFeatureImpl::createDefaultSummaryPlot(cases->back());
         }

ApplicationCode/Commands/RicImportSummaryCasesFeature.h

@@ -38,7 +38,9 @@ class RicImportSummaryCasesFeature : public caf::CmdFeature
 public:
     RicImportSummaryCasesFeature() { }
 
-    static bool createAndAddSummaryCasesFromFiles(const QStringList& fileName, std::vector<RimSummaryCase*>* newCases = nullptr);
+    static bool createAndAddSummaryCasesFromFiles(const QStringList& fileName, 
+                                                  bool doCreateDefaultPlot,
+                                                  std::vector<RimSummaryCase*>* newCases = nullptr);
     static bool createSummaryCasesFromFiles(const QStringList& fileName, std::vector<RimSummaryCase*>* newCases, bool ensembleOrGroup = false);
     static void addSummaryCases(const std::vector<RimSummaryCase*> cases);
     static void addCasesToGroupIfRelevant(const std::vector<RimSummaryCase*> cases);

ApplicationCode/Commands/SummaryPlotCommands/RicSummaryPlotFeatureImpl.cpp

@@ -23,14 +23,20 @@
 #include "RimSummaryPlotCollection.h"
 #include "RimProject.h"
 #include "RimMainPlotCollection.h"
+#include "RimSummaryCase.h"
 
 #include "RiuPlotMainWindowTools.h"
 
 #include "RiaApplication.h"
 #include "RiaColorTables.h"
 #include "RiaPreferences.h"
+#include "RiaEclipseFileNameTools.h"
+
 #include "RifSummaryReaderInterface.h"
-#include "RimSummaryCase.h"
+
+#include "RicImportGeneralDataFeature.h"
+
+#include <QStringList>
 
 //--------------------------------------------------------------------------------------------------
 ///
@@ -91,33 +97,7 @@ std::vector<RimSummaryCurve*> RicSummaryPlotFeatureImpl::addDefaultCurvesToPlot(
     QString curvesTextFilter = RiaApplication::instance()->preferences()->defaultSummaryCurvesTextFilter;
     QStringList curveFilters = curvesTextFilter.split(";", QString::SkipEmptyParts);
 
-    const std::set<RifEclipseSummaryAddress>&  addrs = summaryCase->summaryReader()->allResultAddresses();
+    return addCurvesFromAddressFiltersToPlot(curveFilters,  plot, summaryCase);
-    std::vector<RifEclipseSummaryAddress> curveAddressesToUse;
-
-    for (const auto & addr : addrs)
-    {
-        for (const QString& filter: curveFilters)
-        {
-            if ( addr.isUiTextMatchingFilterText(filter) )
-            {
-                curveAddressesToUse.push_back(addr); 
-            }
-        }
-    }
-
-    for (const auto & addr : curveAddressesToUse)
-    {
-        RimSummaryCurve* newCurve = new RimSummaryCurve();
-        plot->addCurveNoUpdate(newCurve);
-        if (summaryCase)
-        {
-            newCurve->setSummaryCaseY(summaryCase);
-        }
-        newCurve->setSummaryAddressYAndApplyInterpolation(addr);
-        defaultCurves.push_back(newCurve);
-    }
-
-    return defaultCurves;
 }
 
 //--------------------------------------------------------------------------------------------------
@@ -141,7 +121,7 @@ void RicSummaryPlotFeatureImpl::createDefaultSummaryPlot( RimSummaryCase* summar
 {
     RimSummaryPlotCollection* summaryPlotCollection = RiaApplication::instance()->project()->mainPlotCollection->summaryPlotCollection();
 
-    if (summaryPlotCollection && summaryCase)
+    if (summaryPlotCollection && summaryCase && !RiaApplication::instance()->preferences()->defaultSummaryCurvesTextFilter().isEmpty())
     {
         auto plot = summaryPlotCollection->createSummaryPlotWithAutoTitle();
 
@@ -160,4 +140,145 @@ void RicSummaryPlotFeatureImpl::createDefaultSummaryPlot( RimSummaryCase* summar
     }
 }
 
+//--------------------------------------------------------------------------------------------------
+/// 
+//--------------------------------------------------------------------------------------------------
+RimSummaryPlot* RicSummaryPlotFeatureImpl::createSummaryPlotFromCommandLine(const QStringList & arguments)
+{
+    RimSummaryPlot* plot = RicSummaryPlotFeatureImpl::createSummaryPlotFromArgumentLine(arguments);
+    return plot;
+}
 
+//--------------------------------------------------------------------------------------------------
+/// 
+//--------------------------------------------------------------------------------------------------
+RimSummaryPlot* RicSummaryPlotFeatureImpl::createSummaryPlotFromArgumentLine(const QStringList & arguments)
+{
+    // Split arguments in options, vectors and filenames
+
+    QStringList options;
+    QStringList summaryAddressFilters;
+    QStringList summaryFiles;
+
+
+    for (int optionIdx = 0; optionIdx < arguments.size(); ++optionIdx)
+    {
+        if (arguments[optionIdx].startsWith("-"))
+        {
+            if (arguments[optionIdx] == "-help")
+            {
+                RiaApplication::instance()->showFormattedTextInMessageBoxOrConsole(
+                    "The --summaryplot option has the following syntax:\n"
+                    "\n"
+                    "[<plotOptions>] <eclipsesummaryvectors> [<eclipsedatafiles>]\n"
+                    "\n"
+                    "It Creates a summary plot using all the <eclipsedatafiles>, and all the summary vectors defined in <eclipsesummaryvectors>.\n"
+                    "The <eclipsesummaryvectors> has the syntax <vectorname>[:<item>[:<subitem>[:i,j,k]]] and can be repeated.\n"
+                    "Wildcards can also be used, eg. \"WOPT:*\" to select the total oil production from all the wells.\n"
+                    "The <eclipsedatafiles> can be written with or without extension. Only the corresponding SMSPEC file will be opened for each case.\n"
+                    "\n"
+                    "The summary plot options are: \n"
+                    "  -help\t Show this help text and ignore the rest of the options.\n"
+                    "  -nl Omit legend in plot."
+                );
+                return nullptr;
+            }
+            options.push_back(arguments[optionIdx]);
+        }
+        else 
+        {
+            RiaEclipseFileNameTools nameTool(arguments[optionIdx]);
+            QString smSpecFileName = nameTool.findRelatedSummarySpecFile();
+            if (smSpecFileName != "")
+            {
+                summaryFiles.push_back(smSpecFileName);
+            }
+            else
+            {
+                summaryAddressFilters.push_back(arguments[optionIdx]) ;
+            }
+        }
+    }
+
+    if ( summaryAddressFilters.empty() )
+    {
+        RiaLogging::error("Needs at least one summary vector to create a plot.");
+    }
+    
+    if ( summaryFiles.size() )
+    {
+        RicImportGeneralDataFeature::OpenCaseResults results =
+            RicImportGeneralDataFeature::openEclipseFilesFromFileNames(summaryFiles, false);
+    }
+
+    std::vector<RimSummaryCase*> summaryCasesToUse = RiaApplication::instance()->project()->allSummaryCases();
+
+    if ( summaryCasesToUse.size() )
+    {
+        RimSummaryPlotCollection* sumPlotColl = RiaApplication::instance()->project()->mainPlotCollection()->summaryPlotCollection();
+        RimSummaryPlot* newPlot = sumPlotColl->createSummaryPlotWithAutoTitle();
+
+        for (RimSummaryCase* sumCase : summaryCasesToUse)
+        {
+            RicSummaryPlotFeatureImpl::addCurvesFromAddressFiltersToPlot(summaryAddressFilters, newPlot, sumCase);
+        }
+
+        if (options.contains("-nl"))
+        {
+            newPlot->showLegend(false);
+        }
+
+        newPlot->applyDefaultCurveAppearances();
+        newPlot->loadDataAndUpdate();
+
+        sumPlotColl->updateConnectedEditors();
+
+        RiuPlotMainWindowTools::setExpanded(newPlot);
+        RiuPlotMainWindowTools::selectAsCurrentItem(newPlot);
+    }
+    else
+    {
+        RiaLogging::error("Needs at least one summary case to create a plot.");
+    }
+
+
+    return nullptr;
+}
+
+//--------------------------------------------------------------------------------------------------
+/// 
+//--------------------------------------------------------------------------------------------------
+std::vector<RimSummaryCurve*> RicSummaryPlotFeatureImpl::addCurvesFromAddressFiltersToPlot(const QStringList& curveFilters, 
+                                                                                           RimSummaryPlot* plot, 
+                                                                                           RimSummaryCase* summaryCase)
+{
+    std::vector<RimSummaryCurve*> createdCurves;
+
+    const std::set<RifEclipseSummaryAddress>&  addrs = summaryCase->summaryReader()->allResultAddresses();
+    std::vector<RifEclipseSummaryAddress> curveAddressesToUse;
+
+    for (const auto & addr : addrs)
+    {
+        for (const QString& filter: curveFilters)
+        {
+            if ( addr.isUiTextMatchingFilterText(filter) )
+            {
+                curveAddressesToUse.push_back(addr); 
+            }
+        }
+    }
+
+    for (const auto & addr : curveAddressesToUse)
+    {
+        RimSummaryCurve* newCurve = new RimSummaryCurve();
+        plot->addCurveNoUpdate(newCurve);
+        if (summaryCase)
+        {
+            newCurve->setSummaryCaseY(summaryCase);
+        }
+        newCurve->setSummaryAddressYAndApplyInterpolation(addr);
+        createdCurves.push_back(newCurve);
+    }
+
+    return createdCurves;
+}

ApplicationCode/Commands/SummaryPlotCommands/RicSummaryPlotFeatureImpl.h

@@ -23,15 +23,21 @@ class RimSummaryCase;
 class RimSummaryPlotCollection;
 
 #include <vector>
+class QStringList;
+
 
 class RicSummaryPlotFeatureImpl
 {
 public:
     static RimSummaryCurve* addDefaultCurveToPlot(RimSummaryPlot* plot, RimSummaryCase* summaryCase);
+    static std::vector<RimSummaryCurve*> addCurvesFromAddressFiltersToPlot(const QStringList& curveFilters, 
+                                                                           RimSummaryPlot* plot, 
+                                                                           RimSummaryCase* summaryCase);
     static std::vector<RimSummaryCurve*> addDefaultCurvesToPlot(RimSummaryPlot* plot, RimSummaryCase* summaryCase);
     static void ensureAtLeastOnePlot(RimSummaryPlotCollection* summaryPlotCollection, RimSummaryCase* summaryCase);
     static void createDefaultSummaryPlot(RimSummaryCase* summaryCase);
-
+    
-
+    static RimSummaryPlot* createSummaryPlotFromCommandLine(const QStringList & arguments);
+    static RimSummaryPlot* createSummaryPlotFromArgumentLine(const QStringList & arguments);
 };
 

ApplicationCode/ProjectDataModel/Summary/RimSummaryPlot.cpp

@@ -1794,6 +1794,14 @@ caf::PdmObject* RimSummaryPlot::findRimPlotObjectFromQwtCurve(const QwtPlotCurve
     return nullptr;
 }
 
+//--------------------------------------------------------------------------------------------------
+/// 
+//--------------------------------------------------------------------------------------------------
+void RimSummaryPlot::showLegend(bool enable)
+{
+    m_showLegend = enable;
+}
+
 //--------------------------------------------------------------------------------------------------
 /// 
 //--------------------------------------------------------------------------------------------------

ApplicationCode/ProjectDataModel/Summary/RimSummaryPlot.h

@@ -148,6 +148,7 @@ public:
     void selectAxisInPropertyEditor(int axis) override;
     void setAutoZoomForAllAxes(bool enableAutoZoom) override;
     caf::PdmObject* findRimPlotObjectFromQwtCurve(const QwtPlotCurve* curve) const override;
+    void showLegend(bool enable);
 
 public:
     // RimViewWindow overrides