]> git.sesse.net Git - nageru/blobdiff - glwidget.cpp
Update the queue length metric after trimming, not before.
[nageru] / glwidget.cpp
index ae4b62f09da9b1d17039c84dcb8f4373d561af08..31018d6b95d5436cddc840ab01738b1b30ded18b 100644 (file)
@@ -20,6 +20,7 @@
 
 #include "audio_mixer.h"
 #include "context.h"
+#include "context_menus.h"
 #include "flags.h"
 #include "mainwindow.h"
 #include "mixer.h"
@@ -31,6 +32,8 @@ class QMouseEvent;
 #include <movit/util.h>
 #include <string>
 
+
+using namespace movit;
 using namespace std;
 using namespace std::placeholders;
 
@@ -39,12 +42,17 @@ GLWidget::GLWidget(QWidget *parent)
 {
 }
 
-void GLWidget::clean_context()
+GLWidget::~GLWidget()
+{
+}
+
+void GLWidget::shutdown()
 {
        if (resource_pool != nullptr) {
                makeCurrent();
                resource_pool->clean_context();
        }
+       global_mixer->remove_frame_ready_callback(output, this);
 }
 
 void GLWidget::initializeGL()
@@ -56,16 +64,13 @@ void GLWidget::initializeGL()
                global_mainwindow->mixer_created(global_mixer);
                global_mixer->start();
        });
-       global_mixer->set_frame_ready_callback(output, [this]{
+       global_mixer->add_frame_ready_callback(output, this, [this]{
                QMetaObject::invokeMethod(this, "update", Qt::AutoConnection);
        });
        if (output == Mixer::OUTPUT_LIVE) {
                global_mixer->set_transition_names_updated_callback(output, [this](const vector<string> &names){
                        emit transition_names_updated(names);
                });
-               setContextMenuPolicy(Qt::CustomContextMenu);
-               connect(this, &QWidget::customContextMenuRequested,
-                       bind(&GLWidget::show_live_context_menu, this, _1));
        }
        if (output >= Mixer::OUTPUT_INPUT0) {
                global_mixer->set_name_updated_callback(output, [this](const string &name){
@@ -74,14 +79,9 @@ void GLWidget::initializeGL()
                global_mixer->set_color_updated_callback(output, [this](const string &color){
                        emit color_updated(output, color);
                });
-
-               int signal_num = global_mixer->get_channel_signal(output);
-               if (signal_num != -1) {
-                       setContextMenuPolicy(Qt::CustomContextMenu);
-                       connect(this, &QWidget::customContextMenuRequested,
-                               bind(&GLWidget::show_preview_context_menu, this, signal_num, _1));
-               }
        }
+       setContextMenuPolicy(Qt::CustomContextMenu);
+       connect(this, &QWidget::customContextMenuRequested, bind(&GLWidget::show_context_menu, this, _1));
 
        glDisable(GL_BLEND);
        glDisable(GL_DEPTH_TEST);
@@ -109,6 +109,8 @@ void GLWidget::paintGL()
        check_error();
        frame.setup_chain();
        check_error();
+       glDisable(GL_FRAMEBUFFER_SRGB);
+       check_error();
        frame.chain->render_to_screen();
        check_error();
 
@@ -124,6 +126,17 @@ void GLWidget::mousePressEvent(QMouseEvent *event)
        emit clicked();
 }
 
+void GLWidget::show_context_menu(const QPoint &pos)
+{
+       if (output == Mixer::OUTPUT_LIVE) {
+               show_live_context_menu(pos);
+       }
+       if (output >= Mixer::OUTPUT_INPUT0) {
+               int signal_num = global_mixer->get_channel_signal(output);
+               show_preview_context_menu(signal_num, pos);
+       }
+}
+
 void GLWidget::show_live_context_menu(const QPoint &pos)
 {
        QPoint global_pos = mapToGlobal(pos);
@@ -132,76 +145,20 @@ void GLWidget::show_live_context_menu(const QPoint &pos)
 
        // Add a submenu for selecting output card, with an action for each card.
        QMenu card_submenu;
-       QActionGroup card_group(&card_submenu);
-
-       int current_card = global_mixer->get_output_card_index();
-
-       QAction *none_action = new QAction("None", &card_group);
-       none_action->setCheckable(true);
-       if (current_card == -1) {
-               none_action->setChecked(true);
-       }
-       none_action->setData(QList<QVariant>{"output_card", -1});
-       card_submenu.addAction(none_action);
-
-       unsigned num_cards = global_mixer->get_num_cards();
-       for (unsigned card_index = 0; card_index < num_cards; ++card_index) {
-               if (!global_mixer->card_can_be_used_as_output(card_index)) {
-                       continue;
-               }
-
-               QString description(QString::fromStdString(global_mixer->get_output_card_description(card_index)));
-               QAction *action = new QAction(description, &card_group);
-               action->setCheckable(true);
-               if (current_card == int(card_index)) {
-                       action->setChecked(true);
-               }
-               action->setData(QList<QVariant>{"output_card", card_index});
-               card_submenu.addAction(action);
-       }
-
-       card_submenu.setTitle("HDMI/SDI output");
+       fill_hdmi_sdi_output_device_menu(&card_submenu);
+       card_submenu.setTitle("HDMI/SDI output device");
        menu.addMenu(&card_submenu);
 
        // Add a submenu for choosing the output resolution. Since this is
        // card-dependent, it is disabled if we haven't chosen a card
        // (but it's still there so that the user will know it exists).
        QMenu resolution_submenu;
-       QActionGroup resolution_group(&resolution_submenu);
-       if (current_card == -1) {
-               resolution_submenu.setEnabled(false);
-       } else {
-               uint32_t current_mode = global_mixer->get_output_video_mode();
-               map<uint32_t, bmusb::VideoMode> video_modes = global_mixer->get_available_output_video_modes();
-               for (const auto &mode : video_modes) {
-                       QString description(QString::fromStdString(mode.second.name));
-                       QAction *action = new QAction(description, &resolution_group);
-                       action->setCheckable(true);
-                       if (current_mode == mode.first) {
-                               action->setChecked(true);
-                       }
-                       action->setData(QList<QVariant>{"video_mode", mode.first});
-                       resolution_submenu.addAction(action);
-               }
-       }
-
+       fill_hdmi_sdi_output_resolution_menu(&resolution_submenu);
        resolution_submenu.setTitle("HDMI/SDI output resolution");
        menu.addMenu(&resolution_submenu);
 
-       // Show the menu and look at the result.
-       QAction *selected_item = menu.exec(global_pos);
-       if (selected_item != nullptr) {
-               QList<QVariant> selected = selected_item->data().toList();
-               if (selected[0].toString() == "output_card") {
-                       unsigned output_card = selected[1].toUInt(nullptr);
-                       global_mixer->set_output_card(output_card);
-               } else if (selected[0].toString() == "video_mode") {
-                       uint32_t mode = selected[1].toUInt(nullptr);
-                       global_mixer->set_output_video_mode(mode);
-               } else {
-                       assert(false);
-               }
-       }
+       // Show the menu; if there's an action selected, it will deal with it itself.
+       menu.exec(global_pos);
 }
 
 void GLWidget::show_preview_context_menu(unsigned signal_num, const QPoint &pos)
@@ -230,6 +187,48 @@ void GLWidget::show_preview_context_menu(unsigned signal_num, const QPoint &pos)
        card_submenu.setTitle("Input source");
        menu.addMenu(&card_submenu);
 
+       // Note that this setting depends on which card is active.
+       // TODO: Consider hiding this for BGRA sources.
+
+       QMenu interpretation_submenu;
+       QActionGroup interpretation_group(&interpretation_submenu);
+
+       YCbCrInterpretation current_interpretation = global_mixer->get_input_ycbcr_interpretation(current_card);
+       {
+               QAction *action = new QAction("Auto", &interpretation_group);
+               action->setCheckable(true);
+               if (current_interpretation.ycbcr_coefficients_auto) {
+                       action->setChecked(true);
+               }
+               action->setData(QList<QVariant>{"interpretation", true, YCBCR_REC_709, false});
+               interpretation_submenu.addAction(action);
+       }
+       for (YCbCrLumaCoefficients ycbcr_coefficients : { YCBCR_REC_709, YCBCR_REC_601 }) {
+               for (bool full_range : { false, true }) {
+                       std::string description;
+                       if (ycbcr_coefficients == YCBCR_REC_709) {
+                               description = "Rec. 709 (HD)";
+                       } else {
+                               description = "Rec. 601 (SD)";
+                       }
+                       if (full_range) {
+                               description += ", full range (nonstandard)";
+                       }
+                       QAction *action = new QAction(QString::fromStdString(description), &interpretation_group);
+                       action->setCheckable(true);
+                       if (!current_interpretation.ycbcr_coefficients_auto &&
+                           ycbcr_coefficients == current_interpretation.ycbcr_coefficients &&
+                           full_range == current_interpretation.full_range) {
+                               action->setChecked(true);
+                       }
+                       action->setData(QList<QVariant>{"interpretation", false, ycbcr_coefficients, full_range});
+                       interpretation_submenu.addAction(action);
+               }
+       }
+
+       interpretation_submenu.setTitle("Input interpretation");
+       menu.addMenu(&interpretation_submenu);
+
        // --- The choices in the next few options depend a lot on which card is active ---
 
        // Add a submenu for selecting video input, with an action for each input.
@@ -353,6 +352,12 @@ void GLWidget::show_preview_context_menu(unsigned signal_num, const QPoint &pos)
                } else if (selected[0].toString() == "card") {
                        unsigned card_index = selected[1].toUInt(nullptr);
                        global_mixer->set_signal_mapping(signal_num, card_index);
+               } else if (selected[0].toString() == "interpretation") {
+                       YCbCrInterpretation interpretation;
+                       interpretation.ycbcr_coefficients_auto = selected[1].toBool();
+                       interpretation.ycbcr_coefficients = YCbCrLumaCoefficients(selected[2].toUInt(nullptr));
+                       interpretation.full_range = selected[3].toBool();
+                       global_mixer->set_input_ycbcr_interpretation(current_card, interpretation);
                } else {
                        assert(false);
                }