Improve Save as

space.nouspiro.tenmon.metainfo.xml

@@ -59,6 +59,15 @@
     </screenshots>
     <content_rating type="oars-1.1"/>
     <releases>
+    <release version="20251101" date="2025-11-01">
+      <description>
+        <ul>
+          <li>Better image Save as</li>
+          <li>Fix xisf file corruption when platesolving</li>
+          <li>Add selecting language</li>
+        </ul>
+      </description>
+    </release>
     <release version="20250915" date="2025-09-15">
       <description>
         <ul>

src/loadrunable.cpp

@@ -193,7 +193,11 @@ void ConvertRunable::run()
     QFileInfo info(m_outfile);
     info.dir().mkpath(".");
 
-    if(m_params.autostretch)
+    if(m_params.stretch)
+    {
+        rawimage->applySTF(m_params.mtf);
+    }
+    else if(m_params.autostretch)
     {
         rawimage->calcStats();
         MTFParam mtfParam = rawimage->calcMTFParams();

src/loadrunable.h

@@ -6,6 +6,7 @@
 #include <QSemaphore>
 #include <QSize>
 #include "imageinfodata.h"
+#include "mtfparam.h"
 
 class Image;
 
@@ -33,6 +34,8 @@ public:
         QSize resize;
         Qt::AspectRatioMode aspect = Qt::KeepAspectRatio;
         bool autostretch = false;
+        bool stretch = false;
+        MTFParam mtf;
         ConvertParams(){}
         ConvertParams(const QVariantMap &map);
     };

src/mainwindow.cpp

@@ -620,9 +620,13 @@ void MainWindow::reindex()
 void MainWindow::saveAs()
 {
     QString selectedFilter;
+    ImagePtr ptr = m_ringList->currentImage();
+    if(!ptr)return;
+
+    QFileInfo srcFile(ptr->name());
     QString file = QFileDialog::getSaveFileName(this,
                                  tr("Save as"),
-                                 _lastDir,
+                                 _lastDir + "/" + srcFile.baseName(),
                                  _saveFilter,
                                  &selectedFilter);
     auto filterToFormat = [](const QString &file, const QString &filter) -> const QString
@@ -644,25 +648,22 @@ void MainWindow::saveAs()
 
     if(!file.isEmpty())
     {
+        auto button = QMessageBox::question(this, tr("Apply stretch?"), tr("Apply current stretch function to image?"));
+
         QString format = filterToFormat(file, selectedFilter);
 
-        if(format == "fits" || format == "xisf")
+        convert(file, format, button == QMessageBox::Yes);
-        {
-            convert(file, format);
-        }
-        else
-        {
-            QImage img = m_image->renderToImage();
-            if(!img.isNull())
-                img.save(file, filterToFormat(file, selectedFilter).toLatin1().data());
-        }
     }
 }
 
-void MainWindow::convert(const QString &outfile, const QString &format)
+void MainWindow::convert(const QString &outfile, const QString &format, bool stretch)
 {
     QString file = m_ringList->currentImage()->name();
-    QThreadPool::globalInstance()->start(new ConvertRunable(file, outfile, format));
+    ConvertRunable::ConvertParams param;
+    param.stretch = stretch;
+    param.mtf = m_stretchPanel->params();
+
+    QThreadPool::globalInstance()->start(new ConvertRunable(file, outfile, format, param));
 }
 
 void MainWindow::markImage()

src/mainwindow.h

@@ -52,7 +52,7 @@ public slots:
     void indexDir(const QString &dir);
     void reindex();
     void saveAs();
-    void convert(const QString &outfile, const QString &format);
+    void convert(const QString &outfile, const QString &format, bool stretch);
     void markImage();
     void unmarkImage();
     void markAndNext();

src/mtfparam.h

@@ -1,11 +1,41 @@
 #ifndef MTFPARAM_H
 #define MTFPARAM_H
 
+#include <QVariantMap>
+
 struct MTFParam
 {
     float blackPoint[3] = {0.0f, 0.0f, 0.0f};
     float midPoint[3] = {0.5f, 0.5f, 0.5f};
     float whitePoint[3] = {1.0f, 1.0f, 1.0f};
+    QVariantMap toVariantMap() const
+    {
+        QVariantMap ret;
+        ret["black"] = QList<QVariant>({blackPoint[0], blackPoint[1], blackPoint[2]});
+        ret["mid"] = QList<QVariant>({midPoint[0], midPoint[1], midPoint[2]});
+        ret["white"] = QList<QVariant>({whitePoint[0], whitePoint[1], whitePoint[2]});
+        return ret;
+    }
+    static bool fromVariantMap(const QVariantMap &map, MTFParam &mtf)
+    {
+        if(!map.contains("black") || !map.contains("mid") || !map.contains("white"))
+            return false;
+
+        QVariantList black = map["black"].toList();
+        QVariantList mid = map["mid"].toList();
+        QVariantList white = map["white"].toList();
+        if(black.size() < 3 || mid.size() < 3 || white.size() < 3)
+            return false;
+
+        for(int i = 0; i < 3; i++)
+        {
+            mtf.blackPoint[i] = black[i].toFloat();
+            mtf.midPoint[i] = mid[i].toFloat();
+            mtf.whitePoint[i] = white[i].toFloat();
+        }
+
+        return true;
+    }
 };
 
 #endif // MTFPARAM_H

src/rawimage.cpp

@@ -1195,12 +1195,28 @@ void RawImage::applySTF(const MTFParam &mtfParams)
         for(size_t i = 0; i < len; i++)
         {
             float x;
-            if constexpr(std::numeric_limits<std::remove_reference_t<decltype(*src)>>::is_integer)x = src[i] * iscale;
+            if(m_ch == 4)
-            else x = src[i] * unit.first + unit.second;
+            {
-            x = (x - mtfParams.blackPoint[0]) / (mtfParams.whitePoint[0] - mtfParams.blackPoint[0]);
+                size_t c = i & 0x3;
-            x = std::clamp(x, 0.0f, 1.0f);
+                if(c < 3)
-            x = ((mtfParams.midPoint[0] - 1.0f) * x) / ((2.0f * mtfParams.midPoint[0] - 1.0f) * x - mtfParams.midPoint[0]);
+                {
-            src[i] = x * s;
+                    if constexpr(std::numeric_limits<std::remove_reference_t<decltype(*src)>>::is_integer)x = src[i] * iscale;
+                    else x = src[i] * unit.first + unit.second;
+                    x = (x - mtfParams.blackPoint[c]) / (mtfParams.whitePoint[c] - mtfParams.blackPoint[c]);
+                    x = std::clamp(x, 0.0f, 1.0f);
+                    x = ((mtfParams.midPoint[c] - 1.0f) * x) / ((2.0f * mtfParams.midPoint[c] - 1.0f) * x - mtfParams.midPoint[c]);
+                    src[i] = x * s;
+                }
+            }
+            else
+            {
+                if constexpr(std::numeric_limits<std::remove_reference_t<decltype(*src)>>::is_integer)x = src[i] * iscale;
+                else x = src[i] * unit.first + unit.second;
+                x = (x - mtfParams.blackPoint[0]) / (mtfParams.whitePoint[0] - mtfParams.blackPoint[0]);
+                x = std::clamp(x, 0.0f, 1.0f);
+                x = ((mtfParams.midPoint[0] - 1.0f) * x) / ((2.0f * mtfParams.midPoint[0] - 1.0f) * x - mtfParams.midPoint[0]);
+                src[i] = x * s;
+            }
         }
     };
 

src/stretchtoolbar.cpp

@@ -104,6 +104,11 @@ StretchToolbar::~StretchToolbar()
     settings.setValue("stretchtoolbar/autostretch", m_autoStretchOnLoad->isChecked());
 }
 
+const MTFParam &StretchToolbar::params() const
+{
+    return m_mtfParam;
+}
+
 void StretchToolbar::stretchImage(Image *img)
 {
     if(img && img->rawImage())

src/stretchtoolbar.h

@@ -22,6 +22,7 @@ class StretchToolbar : public QToolBar
 public:
     explicit StretchToolbar(QWidget *parent = nullptr);
     ~StretchToolbar();
+    const MTFParam& params() const;
 public slots:
     void stretchImage(Image *img);
     void resetMTF();