Fix #52 Fix #54

Author: Jean-Romain

Changelog.md

@@ -1,3 +1,8 @@
+## Tabliato 1.3.5
+
+- Bug: #52 ajout d'une option pour les machines très lentes.
+- Amélioration: tabliato n'est plus bloqué quand on lance la compilation. La fenêtre reste réactive.
+
 ## Tabliato 1.3.4
 
 - New: Affichage d’une infobulle au survol de certains mots-clés du code pour aider l’utilisateur.

debian/control

@@ -1,5 +1,5 @@
 Package: tabliato
-Version: 1.3.4
+Version: 1.3.5
 Section: sound
 Maintainer: Jean-Romain Roussel <jromain.roussel@gmail.com>
 Homepage: https://jean-romain.github.io/Tabliato/

header/mainwindow.h

@@ -38,8 +38,8 @@ class MainWindow : public QMainWindow
         //void systemcallFinished(int exit);
 
         void compile();
+        void scoreCompilerFinished(int);
         void midi2audio();
-        void midi2audioReadyRead();
         void midi2audioFinished(int);
 
         void save();
@@ -73,7 +73,8 @@ class MainWindow : public QMainWindow
         Ui::MainWindow *ui;
         Highlighter *highlighterTab;
         QMediaPlayer *music;
-        QProcess *midi2audioCall;
+        QProcess *score_compiler;
+        QProcess *midi_converter;
         PdfViewer *pdf;
         QVector<qreal> scaleFactors;
         QTimer *timer;

src/main.cpp

@@ -14,6 +14,10 @@
 #include "filedownloader.h"
 #include "lilypond.h"
 
+#define EXITOK 0
+#define EXITERROR 1
+#define TIMEOUT 2
+
 QCoreApplication* createApplication(int &argc, char *argv[])
 {
     if (argc > 2){
@@ -28,7 +32,7 @@ int main(int argc, char *argv[])
 
     QCoreApplication::setOrganizationName("tabliato");
     QCoreApplication::setApplicationName("tabliato");
-    QCoreApplication::setApplicationVersion("1.3.4");
+    QCoreApplication::setApplicationVersion("1.3.5");
 
     APPDIR = QApplication::applicationDirPath();
     APPPATH = QApplication::applicationFilePath();
@@ -81,6 +85,9 @@ int main(int argc, char *argv[])
     QCommandLineOption opt_sf2("soundfont", "Soundfont to generate audio from midi (implies --ogg)", "file.sf2");
     parser.addOption(opt_sf2);
 
+    QCommandLineOption opt_slow("slow-machine", "Use this option to increase lilypond and timidity timeout");
+    parser.addOption(opt_slow);
+
     QCommandLineOption opt_output(QStringList() << "o" << "output", "Write output in file or folder FILE. File extension is automatically added", "FILE");
     parser.addOption(opt_output);
 
@@ -95,6 +102,17 @@ int main(int argc, char *argv[])
     bool all = parser.isSet(opt_all);
     bool sf2 = parser.isSet(opt_sf2);
     bool out = parser.isSet(opt_output);
+    bool slow = parser.isSet(opt_slow);
+
+    int lilypond_timeout = 5000;
+    int timidity_timeout = 10000;
+
+    // See issue #52
+    if (slow)
+    {
+        lilypond_timeout *= 10;
+        timidity_timeout *= 10;
+    }
 
     const QStringList args = parser.positionalArguments();
 
@@ -143,15 +161,15 @@ int main(int argc, char *argv[])
             if (new_version > current_version)
             {
                 // Si il n'y a pas de fichier c'est que l'utilisateur n'a jamais cliqué sur Non pour l'update
-                // donc il on demande s'il veut mettre à jour
+                // donc on demande s'il veut mettre à jour
                 bool suggest_update = false;
                 if (!skip_update.exists())
                 {
                     suggest_update = true;
                 }
                 // Si le fichier existe c'est que l'utilisateur a dit non pour l'update
-                // mais quand même ca bloque pour l'éternité. Donc si c'est une version encore
-                // plus grande que celle refusée par l'utilisateur on propose quand mêm
+                // mais ca bloque pour l'éternité. Donc si c'est une version encore
+                // plus grande que celle refusée par l'utilisateur on propose quand même
                 else
                 {
                     QVersionNumber skiped_version = QVersionNumber::fromString(File::read(fileName));
@@ -260,6 +278,12 @@ int main(int argc, char *argv[])
 
             File::write(filebasepath + ".ly", tab.lilypond);
 
+            if (!QFile(filebasepath + ".ly").exists())
+            {
+                QTextStream(stderr) << QString::fromUtf8("Erreur inconnue: le fichier lilypond n'a pas été produit.");
+                return 1;
+            }
+
             QStringList arguments;
             if (png) arguments.append("--png");
             if (pdf) arguments.append("--pdf");
@@ -268,26 +292,42 @@ int main(int argc, char *argv[])
 
             QString command = Lilypond::get_lilypond_command();
 
-            if (verbose) qDebug() << command << " " << arguments;
-
             lilypond.start(command, arguments);
-            lilypond.waitForFinished(5000);
-            lilypond.readAllStandardOutput();
 
-            if(!QFile(filebasepath + ".pdf").exists() && !QFile(filebasepath + ".png").exists())
+            if (!lilypond.waitForStarted())
+            {
+                QTextStream(stderr) << QString::fromUtf8("Erreur inconnue: lilypond n'a pas été démarré.") << Qt::endl;
+                return 1;
+            }
+
+            if (!lilypond.waitForFinished(lilypond_timeout))
             {
-                //QTextStream(stderr) << lilypond.readAllStandardError() << endl;
-                QTextStream(stderr) << QString::fromUtf8("Erreur inconnue: le document n'a pas été compilé. Code probablement incorrect.");
+                lilypond.kill();
+                lilypond.waitForFinished();
+                QTextStream(stderr) << QString::fromUtf8("Temps maximum dépassé pour lilypond. La partition n'a pas été produite. Réessayer avec l'option 'Outils' > 'Configuration' > 'CPU faible performance'.") << Qt::endl;
                 return 1;
             }
 
-            if(!ly) QFile::remove(filebasepath + ".ly");
-            if(!midi)
+            // Check output files
+            bool pdfExists = QFile(filebasepath + ".pdf").exists();
+            bool pngExists = QFile(filebasepath + ".png").exists();
+
+            if (!pdfExists && !pngExists)
+            {
+                QTextStream(stderr) << QString::fromUtf8("Erreur inconnue: le document n'a pas été compilé. Code probablement incorrect.") << Qt::endl;
+                return 1;
+            }
+
+            if (!ly)
+            {
+                QFile::remove(filebasepath + ".ly");
+            }
+
+            if (!midi)
             {
                 QFile::remove(filebasepath + ".mid");
                 QFile::remove(filebasepath + ".midi");
             }
-
         }
 
         // ---------------------------------------
@@ -319,7 +359,15 @@ int main(int argc, char *argv[])
         arguments.append("--output-file=" + ofolder + "/" + ofile + AUDIOEXT);
 
         timidity.start(command, arguments);
-        timidity.waitForFinished(10000);
+
+        if (!timidity.waitForFinished(timidity_timeout))
+        {
+            timidity.kill();
+            timidity.waitForFinished();
+            QTextStream(stderr) << QString::fromUtf8("Erreur: temps maximum dépassé pour la conversion midi > audio. Le rendu audio n'a pas été produit. ´Réessayer avec l'option 'Outils' > 'Configuration' > 'CPU faible performance'.") << Qt::endl;
+            return 1;
+        }
+
         timidity.readAllStandardOutput();
 
         return 0;

src/mainwindow.cpp

@@ -28,7 +28,8 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::MainWi
 
     highlighterTab = new Highlighter(ui->melodie_textarea->document());
     music = new QMediaPlayer(this);
-    midi2audioCall = new QProcess(this);
+    score_compiler = new QProcess(this);
+    midi_converter = new QProcess(this);
     pdf = new PdfViewer();
     timer = new QTimer(this);
 
@@ -53,6 +54,7 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::MainWi
     // ==== Trigger rendering ====
     connect(ui->actionCompile, SIGNAL(triggered()), this, SLOT(compile()));
     connect(ui->actionCompile_tool, SIGNAL(triggered()), this, SLOT(compile()));
+    connect(score_compiler,SIGNAL(finished(int)),this,SLOT(scoreCompilerFinished(int)));
 
     // ==== Various exernal opening ====
     connect(ui->actionHelpTabliato,  &QAction::triggered, [=]() { QDesktopServices::openUrl(QUrl("https://jean-romain.github.io/tabliato/doc.html")); });
@@ -104,8 +106,7 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::MainWi
     connect(music, SIGNAL(stateChanged(QMediaPlayer::State)), this, SLOT(updateMusic(QMediaPlayer::State)));
 
     // ==== Music renderer ====
-    connect(midi2audioCall,SIGNAL(readyReadStandardOutput()),this,SLOT(midi2audioReadyRead()));
-    connect(midi2audioCall,SIGNAL(finished(int)),this,SLOT(midi2audioFinished(int)));
+    connect(midi_converter,SIGNAL(finished(int)), this,SLOT(midi2audioFinished(int)));
 
     // ===- Music download soundfonts ====
     connect(ui->actionBallone_Burini,  &QAction::triggered, [=]() { this->download_soundfonts("BalloneBurini.sf2"); });
@@ -177,38 +178,41 @@ MainWindow::~MainWindow()
     delete ui;
     delete highlighterTab;
     delete music;
-    delete midi2audioCall;
+    delete score_compiler;
+    delete midi_converter;
     delete pdf;
 }
 
 void MainWindow::compile()
 {
     if (!currentOpenedFile.isEmpty()) save();
 
-    terminal("Compilation");
-
     if (!Lilypond::is_lilypond_installed())
     {
         QMessageBox::critical(this, "Erreur", "Impossible de trouver le logicel lilypond sur cet machine. Lilypond ne semble pas avoir été installé.");
         return;
     }
 
+    terminal("Compilation...");
+
+
+    ui->actionCompile->setEnabled(false);
+    ui->actionCompile_tool->setEnabled(false);
+
     Tabulature tab = readMusicFromUI();
 
     try
     {
-        QElapsedTimer timer;
-        timer.start();
-
         TabliatoProcessor proc(tab);
         m_timeline = proc.m_timeline;
         //terminal(proc.get_logs());
-        terminal("Tablature parsée en " + QString::number(timer.elapsed()) + " milliseconds");
     }
     catch(const std::exception &e)
     {
         QMessageBox::critical(this, "Erreur", QString(e.what()));
         terminal(e.what());
+        ui->actionCompile->setEnabled(true);
+        ui->actionCompile_tool->setEnabled(true);
         return;
     }
 
@@ -219,28 +223,34 @@ void MainWindow::compile()
     QElapsedTimer timer;
     timer.start();
 
-    QProcess tabliato;
     QStringList arguments;
+
+    if (ui->actionSlowCPU->isChecked()) arguments.append("--slow-machine"); // See #52
+
     arguments.append("--png");
     arguments.append("--pdf");
     arguments.append("--midi");
     arguments.append("--ly");
     arguments.append(dtb);
 
-    tabliato.start(APPPATH, arguments);
-    tabliato.waitForFinished(5000);
+    score_compiler->start(APPPATH, arguments);
+}
+
+
+void MainWindow::scoreCompilerFinished(int exit)
+{
+    ui->actionCompile->setEnabled(true);
+    ui->actionCompile_tool->setEnabled(true);
 
-    if (tabliato.exitCode() != 0)
+    if (exit != 0)
     {
-        QString err = tabliato.readAllStandardError();
-        QMessageBox::critical(this, "Erreur :", err);
+        QString errorDetails = score_compiler->readAllStandardError();
+        if (errorDetails.isEmpty())errorDetails = "La partition n'a pas pu être générée pour une raison inconnue.";
+        QMessageBox::critical(this, "Erreur", errorDetails);
+        terminal(errorDetails);
         return;
     }
 
-    if (!tabliato.readAllStandardOutput().isEmpty())
-        terminal(tabliato.readAllStandardOutput());
-
-    terminal("Gravure terminée en " + QString::number(timer.elapsed()) + " milliseconds");
     updatePreview(OUTPUT + "/output.pdf");
     stopMusic();
     midi2audio();
@@ -252,22 +262,26 @@ void MainWindow::midi2audio()
     ui->play_pushButton->setEnabled(false);
 
     QStringList arguments;
+    if (ui->actionSlowCPU->isChecked()) arguments.append("--slow-machine"); // See #52
     arguments.append("--soundfont=" + SOUNDFONTS + "/" + ui->sf2_combobox->currentText());
     arguments.append(OUTPUT + "/output" + MIDIEXT);
-    midi2audioCall->start(APPPATH, arguments);
-}
-
-void MainWindow::midi2audioReadyRead()
-{
-    terminal(midi2audioCall->readAllStandardOutput());
+    midi_converter->start(APPPATH, arguments);
 }
 
 void MainWindow::midi2audioFinished(int exit)
 {
     exportFiles();
-    if (exit) QMessageBox::critical(this, "Erreur", "La musique n'a pas pu être générée pour une raison inconnue");
+
+    if (exit != 0)
+    {
+        QString errorDetails = midi_converter->readAllStandardError();
+        if (errorDetails.isEmpty())errorDetails = "La musique n'a pas pu être générée pour une raison inconnue.";
+        QMessageBox::critical(this, "Erreur", errorDetails);
+        terminal(errorDetails);
+    }
+
     ui->play_pushButton->setEnabled(true);
-    QUrl audio = QUrl::fromLocalFile(OUTPUT+ "/output" + AUDIOEXT);
+    QUrl audio = QUrl::fromLocalFile(OUTPUT + "/output" + AUDIOEXT);
     music->setMedia(QMediaContent());
     music->setMedia(audio);
     music->setPosition(0);
@@ -771,6 +785,7 @@ void MainWindow::writeSettings()
     settings.setValue("previewWidth", ui->previewDock->width());
     settings.setValue("editorWidth", ui->centralwidget->width());
     settings.setValue("editorFontSize", ui->melodie_textarea->font().pointSize());
+    settings.setValue("slowcpu", ui->actionSlowCPU->isChecked());
     settings.setValue("v110", 0);
 }
 
@@ -800,6 +815,8 @@ void MainWindow::readSettings()
         ui->centralwidget->resize(settings.value("editorWidth").toInt(), ui->centralwidget->height());
         ui->previewDock->resize(settings.value("previewWidth").toInt(), ui->previewDock->height());
 
+        ui->actionSlowCPU->setChecked(settings.value("slowcpu").toBool());
+
         QFont font = ui->melodie_textarea->font();
         font.setPointSize(settings.value("editorFontSize").toInt());
         ui->melodie_textarea->setFont(font);

src/processor.cpp

@@ -475,6 +475,10 @@ void TabliatoProcessor::parseMusic()
                 {
                     m_tab->set("hasFingering", "true");
                 }
+                else if (symbol == "\\tempo")
+                {
+                   // The tempo changed, we need to update the timeline speed.
+                }
                 else if (symbol == "\\relative")
                 {
                     throw std::logic_error("\\relative est une commande lilypond non supportée par tabliato");