Include trimming.
[dcpomatic.git] / src / wx / colour_conversion_editor.cc
index d82239b79f07e8477c9ceff6af85cac9c9be680c..aab05d71bd3836dd4b7be9b73bfacc549a4fe889 100644 (file)
 #include <dcp/modified_gamma_transfer_function.h>
 #include <wx/spinctrl.h>
 #include <wx/gbsizer.h>
-#include <boost/lexical_cast.hpp>
+#include <iostream>
 
 using std::string;
 using std::cout;
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
-using boost::lexical_cast;
 
-ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
+ColourConversionEditor::ColourConversionEditor (wxWindow* parent, bool yuv)
        : wxPanel (parent, wxID_ANY)
+       , _ignore_chromaticity_changed (false)
 {
        wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
        SetSizer (overall_sizer);
@@ -51,12 +51,12 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        table->Add (_input_gamma_linearised, wxGBPosition (r, 0), wxGBSpan (1, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("Input gamma"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, _("Input gamma"), true, wxGBPosition (r, 0));
        _input_gamma = new wxSpinCtrlDouble (this);
        table->Add (_input_gamma, wxGBPosition (r, 1));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("Input power"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, _("Input power"), true, wxGBPosition (r, 0));
        {
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _input_power = new wxSpinCtrlDouble (this);
@@ -91,46 +91,52 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
 
        /* YUV to RGB conversion */
 
-       subhead (table, this, _("YUV to RGB conversion"), r);
+       wxStaticText* yuv_heading = subhead (table, this, _("YUV to RGB conversion"), r);
 
-       add_label_to_grid_bag_sizer (table, this, _("YUV to RGB matrix"), true, wxGBPosition (r, 0));
+       wxStaticText* yuv_label = add_label_to_sizer (table, this, _("YUV to RGB matrix"), true, wxGBPosition (r, 0));
        _yuv_to_rgb = new wxChoice (this, wxID_ANY);
        _yuv_to_rgb->Append (_("Rec. 601"));
        _yuv_to_rgb->Append (_("Rec. 709"));
        table->Add (_yuv_to_rgb, wxGBPosition (r, 1));
        ++r;
 
+       if (!yuv) {
+               yuv_heading->Enable (false);
+               yuv_label->Enable (false);
+               _yuv_to_rgb->Enable (false);
+       }
+
        /* RGB to XYZ conversion */
 
        subhead (table, this, _("RGB to XYZ conversion"), r);
 
-       add_label_to_grid_bag_sizer (table, this, _("x"), false, wxGBPosition (r, 1));
-       add_label_to_grid_bag_sizer (table, this, _("y"), false, wxGBPosition (r, 2));
-       add_label_to_grid_bag_sizer (table, this, _("Matrix"), false, wxGBPosition (r, 3));
+       add_label_to_sizer (table, this, _("x"), false, wxGBPosition (r, 1));
+       add_label_to_sizer (table, this, _("y"), false, wxGBPosition (r, 2));
+       add_label_to_sizer (table, this, _("Matrix"), false, wxGBPosition (r, 3));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("Red chromaticity"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, _("Red chromaticity"), true, wxGBPosition (r, 0));
        _red_x = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_red_x, wxGBPosition (r, 1));
        _red_y = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_red_y, wxGBPosition (r, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("Green chromaticity"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, _("Green chromaticity"), true, wxGBPosition (r, 0));
        _green_x = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_green_x, wxGBPosition (r, 1));
        _green_y = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_green_y, wxGBPosition (r, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("Blue chromaticity"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, _("Blue chromaticity"), true, wxGBPosition (r, 0));
        _blue_x = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_blue_x, wxGBPosition (r, 1));
        _blue_y = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_blue_y, wxGBPosition (r, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("White point"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, _("White point"), true, wxGBPosition (r, 0));
        _white_x = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
        table->Add (_white_x, wxGBPosition (r, 1));
        _white_y = new wxTextCtrl (this, wxID_ANY, wxT (""), wxDefaultPosition, size, 0, validator);
@@ -164,7 +170,7 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        table->Add (_adjusted_white_y, wxGBPosition (r, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, wxT (""), false, wxGBPosition (r, 0));
+       add_label_to_sizer (table, this, wxT (""), false, wxGBPosition (r, 0));
        ++r;
 
         size = dc.GetTextExtent (wxT ("0.12345678"));
@@ -179,27 +185,12 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        }
        table->Add (bradford_sizer, wxGBPosition (r - 2, 3), wxGBSpan (2, 1));
 
-       subhead (table, this, _("Output gamma correction"), r);
-
-       add_label_to_grid_bag_sizer (table, this, _("Output gamma"), true, wxGBPosition (r, 0));
-       wxBoxSizer* output_sizer = new wxBoxSizer (wxHORIZONTAL);
-       /// TRANSLATORS: this means the mathematical reciprocal operation, i.e. we are dividing 1 by the control that
-       /// comes after it.
-       add_label_to_sizer (output_sizer, this, _("1 / "), false);
-       _output_gamma = new wxSpinCtrlDouble (this);
-       output_sizer->Add (_output_gamma);
-       table->Add (output_sizer, wxGBPosition (r, 1), wxGBSpan (1, 2));
-       ++r;
-
        _input_gamma->SetRange (0.1, 4.0);
        _input_gamma->SetDigits (2);
        _input_gamma->SetIncrement (0.1);
        _input_power->SetRange (0.1, 4.0);
        _input_power->SetDigits (6);
        _input_power->SetIncrement (0.1);
-       _output_gamma->SetRange (0.1, 4.0);
-       _output_gamma->SetDigits (2);
-       _output_gamma->SetIncrement (0.1);
 
        _input_gamma->Bind (wxEVT_COMMAND_SPINCTRLDOUBLE_UPDATED, boost::bind (&ColourConversionEditor::changed, this, _input_gamma));
        _input_gamma_linearised->Bind (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&ColourConversionEditor::changed, this));
@@ -219,16 +210,18 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        _adjusted_white_x->Bind (wxEVT_COMMAND_TEXT_UPDATED, boost::bind (&ColourConversionEditor::adjusted_white_changed, this));
        _adjusted_white_y->Bind (wxEVT_COMMAND_TEXT_UPDATED, boost::bind (&ColourConversionEditor::adjusted_white_changed, this));
        _yuv_to_rgb->Bind (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&ColourConversionEditor::changed, this));
-       _output_gamma->Bind (wxEVT_COMMAND_SPINCTRLDOUBLE_UPDATED, boost::bind (&ColourConversionEditor::changed, this, _output_gamma));
 }
 
-void
+wxStaticText *
 ColourConversionEditor::subhead (wxGridBagSizer* sizer, wxWindow* parent, wxString text, int& row) const
 {
-       wxStaticText* m = new wxStaticText (parent, wxID_ANY, wxT (""));
-       m->SetLabelMarkup ("<b>" + text + "</b>");
+       wxStaticText* m = new wxStaticText (parent, wxID_ANY, text);
+       wxFont font (*wxNORMAL_FONT);
+       font.SetWeight (wxFONTWEIGHT_BOLD);
+       m->SetFont (font);
        sizer->Add (m, wxGBPosition (row, 0), wxGBSpan (1, 3), wxALIGN_CENTER_VERTICAL | wxTOP, 12);
        ++row;
+       return m;
 }
 
 void
@@ -251,6 +244,8 @@ ColourConversionEditor::set (ColourConversion conversion)
 
        _yuv_to_rgb->SetSelection (conversion.yuv_to_rgb ());
 
+       _ignore_chromaticity_changed = true;
+
        SafeStringStream s;
        s.setf (std::ios::fixed, std::ios::floatfield);
        s.precision (6);
@@ -286,6 +281,8 @@ ColourConversionEditor::set (ColourConversion conversion)
        s << conversion.white().y;
        _white_y->SetValue (std_to_wx (s.str ()));
 
+       _ignore_chromaticity_changed = false;
+
        if (conversion.adjusted_white ()) {
                _adjust_white->SetValue (true);
                s.str ("");
@@ -300,8 +297,7 @@ ColourConversionEditor::set (ColourConversion conversion)
 
        update_rgb_to_xyz ();
        update_bradford ();
-
-       set_spin_ctrl (_output_gamma, dynamic_pointer_cast<const dcp::GammaTransferFunction> (conversion.out ())->gamma ());
+       changed ();
 }
 
 ColourConversion
@@ -352,7 +348,7 @@ ColourConversionEditor::get () const
                conversion.unset_adjusted_white ();
        }
 
-       conversion.set_out (shared_ptr<dcp::GammaTransferFunction> (new dcp::GammaTransferFunction (_output_gamma->GetValue ())));
+       conversion.set_out (shared_ptr<dcp::GammaTransferFunction> (new dcp::GammaTransferFunction (2.6)));
 
        return conversion;
 }
@@ -373,6 +369,10 @@ ColourConversionEditor::changed ()
 void
 ColourConversionEditor::chromaticity_changed ()
 {
+       if (_ignore_chromaticity_changed) {
+               return;
+       }
+
        update_rgb_to_xyz ();
        changed ();
 }