Plugin analysis numerics
[ardour.git] / gtk2_ardour / export_video_dialog.cc
index 7d7e14505f0eb648386f79d6f59ac5d6e2b0f8a6..fdf631db1c253591aaea26bf52130caf1f483131 100644 (file)
 
 #include <sigc++/bind.h>
 
+#include <gtkmm/filechooserdialog.h>
+#include <gtkmm/stock.h>
+#include <gtkmm/table.h>
+
 #include "pbd/gstdio_compat.h"
 
 #include "pbd/error.h"
@@ -55,7 +59,7 @@
 #include "opts.h"
 #include "export_video_dialog.h"
 #include "utils_videotl.h"
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace Gtk;
 using namespace std;
@@ -150,7 +154,7 @@ ExportVideoDialog::ExportVideoDialog ()
        path_hbox->pack_start (*l, false, false, 2);
        vbox->pack_start (*path_hbox, false, false, 2);
 
-       insnd_combo.append_text (string_compose (_("from the %1 session's start to the session's end"), PROGRAM_NAME));
+       insnd_combo.append_text (string_compose (_("from session start marker to session end marker"), PROGRAM_NAME));
        outfn_path_entry.set_width_chars(38);
 
        l = manage (new Label (_("<b>Settings:</b>"), Gtk::ALIGN_LEFT, Gtk::ALIGN_CENTER, false));
@@ -330,14 +334,14 @@ ExportVideoDialog::apply_state (TimeSelection &tme, bool range)
        outfn_path_entry.set_text (_session->session_directory().export_path() + G_DIR_SEPARATOR +"export.avi");
 
        // TODO remember setting for export-range.. somehow, (let explicit range override)
-       frameoffset_t av_offset = ARDOUR_UI::instance()->video_timeline->get_offset();
+       sampleoffset_t av_offset = ARDOUR_UI::instance()->video_timeline->get_offset();
        if (av_offset < 0 ) {
-               insnd_combo.append_text (_("from 00:00:00:00 to the video's end"));
+               insnd_combo.append_text (_("from 00:00:00:00 to the video end"));
        } else {
-               insnd_combo.append_text (_("from the video's start to the video's end"));
+               insnd_combo.append_text (_("from video start to video end"));
        }
        if (!export_range.empty()) {
-               insnd_combo.append_text (_("Selected range"));  // TODO show export_range.start() -> export_range.end_frame()
+               insnd_combo.append_text (_("Selected range"));  // TODO show export_range.start() -> export_range.end_sample()
        }
        if (range) {
                insnd_combo.set_active(2);
@@ -366,25 +370,21 @@ ExportVideoDialog::apply_state (TimeSelection &tme, bool range)
 
        float tcfps = _session->timecode_frames_per_second();
 
-       LocaleGuard lg ();
-
        XMLNode* node = _session->extra_xml (X_("Videotimeline"));
        bool filenameset = false;
        if (node) {
-               if (node->property(X_("OriginalVideoFile"))) {
-                       std::string filename = node->property(X_("OriginalVideoFile"))->value();
+               std::string filename;
+               if (node->get_property(X_("OriginalVideoFile"), filename)) {
                        if (Glib::file_test(filename, Glib::FILE_TEST_EXISTS)) {
                                invid_path_entry.set_text (filename);
                                filenameset = true;
                        }
                }
-               if (!filenameset
-                               && node->property(X_("Filename"))
-                               && node->property(X_("LocalFile"))
-                               && node->property(X_("LocalFile"))->value() == X_("1")
-                  )
-               {
-                       std::string filename = node->property(X_("Filename"))->value();
+
+               bool local_file;
+
+               if (!filenameset && node->get_property (X_("Filename"), filename) &&
+                   node->get_property (X_("LocalFile"), local_file) && local_file) {
                        if (filename.at(0) != G_DIR_SEPARATOR)
                        {
                                filename = Glib::build_filename (_session->session_directory().video_path(), filename);
@@ -402,55 +402,78 @@ ExportVideoDialog::apply_state (TimeSelection &tme, bool range)
 
        node = _session->extra_xml (X_("Videoexport"));
        if (node) {
-               XMLProperty const * prop;
-               prop = node->property (X_("ChangeGeometry"));
-               if (prop) { scale_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("KeepAspect"));
-               if (prop) { scale_aspect.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("ChangeAspect"));
-               if (prop) { aspect_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("NormalizeAudio"));
-               if (prop) { normalize_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("TwoPassEncode"));
-               if (prop) { twopass_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("CodecOptimzations"));
-               if (prop) { optimizations_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("Deinterlace"));
-               if (prop) { deinterlace_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("BFrames"));
-               if (prop) { bframes_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("ChangeFPS"));
-               if (prop) { fps_checkbox.set_active(atoi(prop->value())?true:false); }
-               prop = node->property (X_("Metadata"));
-               if (prop) { meta_checkbox.set_active(atoi(prop->value())?true:false); }
-
-               prop = node->property (X_("Format"));
-               if (prop && !prop->value().empty()) { change_file_extension( "." + prop->value()); }
+               bool yn;
+               if (node->get_property (X_("ChangeGeometry"), yn)) {
+                       scale_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("KeepAspect"), yn)) {
+                       scale_aspect.set_active (yn);
+               }
+               if (node->get_property (X_("ChangeAspect"), yn)) {
+                       aspect_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("NormalizeAudio"), yn)) {
+                       normalize_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("TwoPassEncode"), yn)) {
+                       twopass_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("CodecOptimzations"), yn)) {
+                       optimizations_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("Deinterlace"), yn)) {
+                       deinterlace_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("BSamples"), yn)) {
+                       bframes_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("ChangeFPS"), yn)) {
+                       fps_checkbox.set_active (yn);
+               }
+               if (node->get_property (X_("Metadata"), yn)) {
+                       meta_checkbox.set_active (yn);
+               }
+
+               std::string str;
+               if (node->get_property (X_("Format"), str) && !str.empty()) {
+                       change_file_extension ("." + str);
+               }
 
                _suspend_signals = true;
-               prop = node->property (X_("Width"));
-               if (prop) { width_spinner.set_value(atoi(prop->value())); }
-               prop = node->property (X_("Height"));
-               if (prop) { height_spinner.set_value(atoi(prop->value())); }
+               double val;
+               if (node->get_property (X_("Width"), val)) {
+                       width_spinner.set_value (val);
+               }
+               if (node->get_property (X_("Height"), val)) {
+                       height_spinner.set_value (val);
+               }
                _suspend_signals = false;
 
-               prop = node->property (X_("FPS"));
-               if (prop && fps_checkbox.get_active()) { tcfps = atof(prop->value()); }
-
-               prop = node->property (X_("Preset"));
-               if (prop) { preset_combo.set_active_text(prop->value()); }
-               prop = node->property (X_("VCodec"));
-               if (prop) { video_codec_combo.set_active_text(prop->value()); }
-               prop = node->property (X_("ACodec"));
-               if (prop) { audio_codec_combo.set_active_text(prop->value()); }
-               prop = node->property (X_("VBitrate"));
-               if (prop) { video_bitrate_combo.set_active_text(prop->value()); }
-               prop = node->property (X_("ABitrate"));
-               if (prop) { audio_bitrate_combo.set_active_text(prop->value()); }
-               prop = node->property (X_("AspectRatio"));
-               if (prop) { aspect_combo.set_active_text(prop->value()); }
-               prop = node->property (X_("SampleRate"));
-               if (prop) { audio_samplerate_combo.set_active_text(prop->value()); }
+               if (fps_checkbox.get_active () && node->get_property (X_("FPS"), val)) {
+                       tcfps = val;
+               }
+
+               if (node->get_property (X_("Preset"), str)) {
+                       preset_combo.set_active_text (str);
+               }
+               if (node->get_property (X_("VCodec"), str)) {
+                       video_codec_combo.set_active_text (str);
+               }
+               if (node->get_property (X_("ACodec"), str)) {
+                       audio_codec_combo.set_active_text (str);
+               }
+               if (node->get_property (X_("VBitrate"), str)) {
+                       video_bitrate_combo.set_active_text (str);
+               }
+               if (node->get_property (X_("ABitrate"), str)) {
+                       audio_bitrate_combo.set_active_text (str);
+               }
+               if (node->get_property (X_("AspectRatio"), str)) {
+                       aspect_combo.set_active_text (str);
+               }
+               if (node->get_property (X_("SampleRate"), str)) {
+                       audio_samplerate_combo.set_active_text (str);
+               }
        }
 
        if      (fabs(tcfps - 23.976) < 0.01) { fps_combo.set_active(0); }
@@ -482,32 +505,31 @@ ExportVideoDialog::apply_state (TimeSelection &tme, bool range)
 XMLNode&
 ExportVideoDialog::get_state ()
 {
-       LocaleGuard lg ();
        XMLNode* node = new XMLNode (X_("Videoexport"));
-       node->add_property (X_("ChangeGeometry"), scale_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("KeepAspect"), scale_aspect.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("ChangeAspect"), aspect_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("NormalizeAudio"), normalize_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("TwoPassEncode"), twopass_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("CodecOptimzations"), optimizations_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("Deinterlace"), deinterlace_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("BFrames"), bframes_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("ChangeFPS"), fps_checkbox.get_active() ? X_("1") : X_("0"));
-       node->add_property (X_("Metadata"), meta_checkbox.get_active() ? X_("1") : X_("0"));
-
-       node->add_property (X_("Format"), get_file_extension(outfn_path_entry.get_text()));
-
-       node->add_property (X_("Width"), width_spinner.get_value());
-       node->add_property (X_("Height"), height_spinner.get_value());
-
-       node->add_property (X_("Preset"), preset_combo.get_active_text());
-       node->add_property (X_("VCodec"), video_codec_combo.get_active_text());
-       node->add_property (X_("ACodec"), audio_codec_combo.get_active_text());
-       node->add_property (X_("VBitrate"), video_bitrate_combo.get_active_text());
-       node->add_property (X_("ABitrate"), audio_bitrate_combo.get_active_text());
-       node->add_property (X_("AspectRatio"), aspect_combo.get_active_text());
-       node->add_property (X_("SampleRate"), audio_samplerate_combo.get_active_text());
-       node->add_property (X_("FPS"), fps_combo.get_active_text());
+       node->set_property (X_("ChangeGeometry"), scale_checkbox.get_active());
+       node->set_property (X_("KeepAspect"), scale_aspect.get_active());
+       node->set_property (X_("ChangeAspect"), aspect_checkbox.get_active());
+       node->set_property (X_("NormalizeAudio"), normalize_checkbox.get_active());
+       node->set_property (X_("TwoPassEncode"), twopass_checkbox.get_active());
+       node->set_property (X_("CodecOptimzations"), optimizations_checkbox.get_active());
+       node->set_property (X_("Deinterlace"), deinterlace_checkbox.get_active());
+       node->set_property (X_("BSamples"), bframes_checkbox.get_active());
+       node->set_property (X_("ChangeFPS"), fps_checkbox.get_active());
+       node->set_property (X_("Metadata"), meta_checkbox.get_active());
+
+       node->set_property (X_("Format"), get_file_extension(outfn_path_entry.get_text()));
+
+       node->set_property (X_("Width"), width_spinner.get_value());
+       node->set_property (X_("Height"), height_spinner.get_value());
+
+       node->set_property (X_("Preset"), preset_combo.get_active_text());
+       node->set_property (X_("VCodec"), video_codec_combo.get_active_text());
+       node->set_property (X_("ACodec"), audio_codec_combo.get_active_text());
+       node->set_property (X_("VBitrate"), video_bitrate_combo.get_active_text());
+       node->set_property (X_("ABitrate"), audio_bitrate_combo.get_active_text());
+       node->set_property (X_("AspectRatio"), aspect_combo.get_active_text());
+       node->set_property (X_("SampleRate"), audio_samplerate_combo.get_active_text());
+       node->set_property (X_("FPS"), fps_combo.get_active_text());
 
        return *node;
 }
@@ -533,7 +555,7 @@ ExportVideoDialog::abort_clicked ()
 }
 
 void
-ExportVideoDialog::update_progress (framecnt_t c, framecnt_t a)
+ExportVideoDialog::update_progress (samplecnt_t c, samplecnt_t a)
 {
        if (a == 0 || c > a) {
                pbar.set_pulse_step(.1);
@@ -559,12 +581,12 @@ ExportVideoDialog::audio_progress_display ()
        switch (status->active_job) {
                case ExportStatus::Normalizing:
                        pbar.set_text (_("Normalizing audio"));
-                       progress = ((float) status->current_normalize_cycle) / status->total_normalize_cycles;
+                       progress = ((float) status->current_postprocessing_cycle) / status->total_postprocessing_cycles;
                        progress = progress / (_twopass ? 4.0 : 3.0) + (_twopass ? .25 : 1.0 / 3.0);
                        break;
                case ExportStatus::Exporting:
                        pbar.set_text (_("Exporting audio"));
-                       progress = ((float) status->processed_frames_current_timespan) / status->total_frames_current_timespan;
+                       progress = ((float) status->processed_samples_current_timespan) / status->total_samples_current_timespan;
                        progress = progress / ((_twopass ? 2.0 : 1.0) + (_normalize ? 2.0 : 1.0));
                        break;
                default:
@@ -679,19 +701,19 @@ ExportVideoDialog::launch_export ()
        boost::shared_ptr<ExportFormatSpecification> fmp = _session->get_export_handler()->add_format(*tree.root());
 
        /* set up range */
-       framepos_t start, end;
+       samplepos_t start, end;
        start = end = 0;
        if (insnd_combo.get_active_row_number() == 1) {
                _transcoder = new TranscodeFfmpeg(invid_path_entry.get_text());
                if (_transcoder->probe_ok() && _transcoder->get_fps() > 0) {
-                       end = _transcoder->get_duration() * _session->nominal_frame_rate() / _transcoder->get_fps();
+                       end = _transcoder->get_duration() * _session->nominal_sample_rate() / _transcoder->get_fps();
                } else {
                        warning << _("Export Video: Cannot query duration of video-file, using duration from timeline instead.") << endmsg;
                        end = ARDOUR_UI::instance()->video_timeline->get_duration();
                }
                if (_transcoder) {delete _transcoder; _transcoder = 0;}
 
-               frameoffset_t av_offset = ARDOUR_UI::instance()->video_timeline->get_offset();
+               sampleoffset_t av_offset = ARDOUR_UI::instance()->video_timeline->get_offset();
 #if 0 /* DEBUG */
                printf("audio-range -- AV offset: %lld\n", av_offset);
 #endif
@@ -701,19 +723,19 @@ ExportVideoDialog::launch_export ()
                end += av_offset;
        }
        else if (insnd_combo.get_active_row_number() == 2) {
-               start = ARDOUR_UI::instance()->video_timeline->quantify_frames_to_apv(export_range.start());
-               end   = ARDOUR_UI::instance()->video_timeline->quantify_frames_to_apv(export_range.end_frame());
+               start = ARDOUR_UI::instance()->video_timeline->quantify_samples_to_apv(export_range.start());
+               end   = ARDOUR_UI::instance()->video_timeline->quantify_samples_to_apv(export_range.end_sample());
        }
        if (end <= 0) {
-               start = _session->current_start_frame();
-               end   = _session->current_end_frame();
+               start = _session->current_start_sample();
+               end   = _session->current_end_sample();
        }
 #if 0 /* DEBUG */
        printf("audio export-range %lld -> %lld\n", start, end);
 #endif
 
-       const frameoffset_t vstart = ARDOUR_UI::instance()->video_timeline->get_offset();
-       const frameoffset_t vend   = vstart + ARDOUR_UI::instance()->video_timeline->get_duration();
+       const sampleoffset_t vstart = ARDOUR_UI::instance()->video_timeline->get_offset();
+       const sampleoffset_t vend   = vstart + ARDOUR_UI::instance()->video_timeline->get_duration();
 
        if ( (start >= end) || (end < vstart) || (start > vend)) {
                warning << _("Export Video: export-range does not include video.") << endmsg;
@@ -749,6 +771,7 @@ ExportVideoDialog::launch_export ()
 
        /* do sound export */
        fmp->set_soundcloud_upload(false);
+       _session->get_export_handler()->reset ();
        _session->get_export_handler()->add_export_config (tsp, ccp, fmp, fnp, b);
        _session->get_export_handler()->do_export();
        status = _session->get_export_status ();
@@ -914,23 +937,23 @@ ExportVideoDialog::encode_pass (int pass)
                ffs["-passlogfile"] =  Glib::path_get_dirname (outfn) + G_DIR_SEPARATOR + "ffmpeg2pass";
        }
 
-       frameoffset_t av_offset = ARDOUR_UI::instance()->video_timeline->get_offset();
+       sampleoffset_t av_offset = ARDOUR_UI::instance()->video_timeline->get_offset();
        double duration_s  = 0;
 
        if (insnd_combo.get_active_row_number() == 0) {
                /* session start to session end */
-               framecnt_t duration_f = _session->current_end_frame() - _session->current_start_frame();
-               duration_s = (double)duration_f / (double)_session->nominal_frame_rate();
+               samplecnt_t duration_f = _session->current_end_sample() - _session->current_start_sample();
+               duration_s = (double)duration_f / (double)_session->nominal_sample_rate();
        } else if (insnd_combo.get_active_row_number() == 2) {
                /* selected range */
-               duration_s = export_range.length() / (double)_session->nominal_frame_rate();
+               duration_s = export_range.length() / (double)_session->nominal_sample_rate();
        } else {
                /* video start to end */
-               framecnt_t duration_f = ARDOUR_UI::instance()->video_timeline->get_duration();
+               samplecnt_t duration_f = ARDOUR_UI::instance()->video_timeline->get_duration();
                if (av_offset < 0 ) {
                        duration_f += av_offset;
                }
-               duration_s = (double)duration_f / (double)_session->nominal_frame_rate();
+               duration_s = (double)duration_f / (double)_session->nominal_sample_rate();
        }
 
        std::ostringstream osstream; osstream << duration_s;
@@ -938,14 +961,14 @@ ExportVideoDialog::encode_pass (int pass)
        _transcoder->set_duration(duration_s * _transcoder->get_fps());
 
        if (insnd_combo.get_active_row_number() == 0 || insnd_combo.get_active_row_number() == 2) {
-               framepos_t start, snend;
-               const frameoffset_t vid_duration = ARDOUR_UI::instance()->video_timeline->get_duration();
+               samplepos_t start, snend;
+               const sampleoffset_t vid_duration = ARDOUR_UI::instance()->video_timeline->get_duration();
                if (insnd_combo.get_active_row_number() == 0) {
-                       start = _session->current_start_frame();
-                       snend = _session->current_end_frame();
+                       start = _session->current_start_sample();
+                       snend = _session->current_end_sample();
                } else {
                        start = export_range.start();
-                       snend = export_range.end_frame();
+                       snend = export_range.end_sample();
                }
 
 #if 0 /* DEBUG */
@@ -954,27 +977,27 @@ ExportVideoDialog::encode_pass (int pass)
 #endif
 
                if (av_offset > start && av_offset + vid_duration < snend) {
-                       _transcoder->set_leadinout((av_offset - start) / (double)_session->nominal_frame_rate(),
-                               (snend - (av_offset + vid_duration)) / (double)_session->nominal_frame_rate());
+                       _transcoder->set_leadinout((av_offset - start) / (double)_session->nominal_sample_rate(),
+                               (snend - (av_offset + vid_duration)) / (double)_session->nominal_sample_rate());
                } else if (av_offset > start) {
-                       _transcoder->set_leadinout((av_offset - start) / (double)_session->nominal_frame_rate(), 0);
+                       _transcoder->set_leadinout((av_offset - start) / (double)_session->nominal_sample_rate(), 0);
                } else if (av_offset + vid_duration < snend) {
-                       _transcoder->set_leadinout(0, (snend - (av_offset + vid_duration)) / (double)_session->nominal_frame_rate());
-                       _transcoder->set_avoffset((av_offset - start) / (double)_session->nominal_frame_rate());
+                       _transcoder->set_leadinout(0, (snend - (av_offset + vid_duration)) / (double)_session->nominal_sample_rate());
+                       _transcoder->set_avoffset((av_offset - start) / (double)_session->nominal_sample_rate());
                }
 #if 0
                else if (start > av_offset) {
-                       std::ostringstream osstream; osstream << ((start - av_offset) / (double)_session->nominal_frame_rate());
+                       std::ostringstream osstream; osstream << ((start - av_offset) / (double)_session->nominal_sample_rate());
                        ffs["-ss"] = osstream.str();
                }
 #endif
                else {
-                       _transcoder->set_avoffset((av_offset - start) / (double)_session->nominal_frame_rate());
+                       _transcoder->set_avoffset((av_offset - start) / (double)_session->nominal_sample_rate());
                }
 
        } else if (av_offset < 0) {
                /* from 00:00:00:00 to video-end */
-               _transcoder->set_avoffset(av_offset / (double)_session->nominal_frame_rate());
+               _transcoder->set_avoffset(av_offset / (double)_session->nominal_sample_rate());
        }
 
        TranscodeFfmpeg::FFSettings meta = _transcoder->default_meta_data();
@@ -1135,7 +1158,7 @@ ExportVideoDialog::preset_combo_changed ()
                video_codec_combo.set_active(6);
                audio_bitrate_combo.set_active(2);
                video_bitrate_combo.set_active(4);
-               if (_session->nominal_frame_rate() == 48000 || _session->nominal_frame_rate() == 96000) {
+               if (_session->nominal_sample_rate() == 48000 || _session->nominal_sample_rate() == 96000) {
                        audio_samplerate_combo.set_active(2);
                } else {
                        audio_samplerate_combo.set_active(1);
@@ -1147,7 +1170,7 @@ ExportVideoDialog::preset_combo_changed ()
                video_codec_combo.set_active(2);
                audio_bitrate_combo.set_active(3);
                video_bitrate_combo.set_active(4);
-               if (_session->nominal_frame_rate() == 48000 || _session->nominal_frame_rate() == 96000) {
+               if (_session->nominal_sample_rate() == 48000 || _session->nominal_sample_rate() == 96000) {
                        audio_samplerate_combo.set_active(2);
                } else {
                        audio_samplerate_combo.set_active(1);
@@ -1159,7 +1182,7 @@ ExportVideoDialog::preset_combo_changed ()
                video_codec_combo.set_active(7);
                audio_bitrate_combo.set_active(3);
                video_bitrate_combo.set_active(4);
-               if (_session->nominal_frame_rate() == 48000 || _session->nominal_frame_rate() == 96000) {
+               if (_session->nominal_sample_rate() == 48000 || _session->nominal_sample_rate() == 96000) {
                        audio_samplerate_combo.set_active(2);
                } else {
                        audio_samplerate_combo.set_active(1);
@@ -1190,7 +1213,7 @@ ExportVideoDialog::preset_combo_changed ()
                video_codec_combo.set_active(5);
                audio_bitrate_combo.set_active(4);
                video_bitrate_combo.set_active(5);
-               if (_session->nominal_frame_rate() == 48000 || _session->nominal_frame_rate() == 96000) {
+               if (_session->nominal_sample_rate() == 48000 || _session->nominal_sample_rate() == 96000) {
                        audio_samplerate_combo.set_active(2);
                } else {
                        audio_samplerate_combo.set_active(1);
@@ -1202,7 +1225,7 @@ ExportVideoDialog::preset_combo_changed ()
                video_codec_combo.set_active(6);
                audio_bitrate_combo.set_active(0);
                video_bitrate_combo.set_active(0);
-               if (_session->nominal_frame_rate() == 48000 || _session->nominal_frame_rate() == 96000) {
+               if (_session->nominal_sample_rate() == 48000 || _session->nominal_sample_rate() == 96000) {
                        audio_samplerate_combo.set_active(2);
                } else {
                        audio_samplerate_combo.set_active(1);