From db6f6e87dc4ff620f399597913f14a3b4eda277f Mon Sep 17 00:00:00 2001 From: David Robillard Date: Mon, 2 Apr 2007 01:52:53 +0000 Subject: Edge label toggling. Made state properties dialog actually work. Other things... git-svn-id: http://svn.drobilla.net/lad/machina@390 a436a847-0d15-0410-975c-d299462d15a1 --- src/engine/Engine.cpp | 24 +++++++++++++++++++++++- src/engine/JackDriver.cpp | 13 ++++++++++++- src/engine/Loader.cpp | 7 +++++++ src/engine/MachineBuilder.cpp | 24 +++++++++++++++--------- src/engine/machina/Driver.hpp | 3 +++ src/engine/machina/Engine.hpp | 1 + src/engine/machina/JackDriver.hpp | 2 ++ src/gui/EdgeView.cpp | 21 ++++++++++++++++++--- src/gui/EdgeView.hpp | 3 ++- src/gui/MachinaGUI.cpp | 24 ++++++++++++++++++++++-- src/gui/MachinaGUI.hpp | 4 +++- src/gui/NodePropertiesWindow.cpp | 19 ++++++++++++++----- src/gui/NodePropertiesWindow.hpp | 4 +++- src/gui/machina.glade | 33 ++++++++++++++++++++++++++------- src/gui/main.cpp | 5 +++++ 15 files changed, 156 insertions(+), 31 deletions(-) (limited to 'src') diff --git a/src/engine/Engine.cpp b/src/engine/Engine.cpp index 5da3289..69402a3 100644 --- a/src/engine/Engine.cpp +++ b/src/engine/Engine.cpp @@ -36,6 +36,7 @@ Engine::load_machine(const Glib::ustring& uri) if (m) { m->activate(); _driver->set_machine(m); + //_driver->machine()->nodes().append(m->nodes()); } // .. and drop it in this thread (to prevent deallocation in the RT thread) @@ -44,6 +45,26 @@ Engine::load_machine(const Glib::ustring& uri) } +/** Load the machine at @a uri, and insert it into the current machine.. + * Safe to call while engine is processing. + */ +SharedPtr +Engine::import_machine(const Glib::ustring& uri) +{ + SharedPtr old_machine = _driver->machine(); // Hold a reference to current machine.. + + SharedPtr m = Loader().load(uri); + if (m) { + m->activate(); + _driver->machine()->nodes().append(m->nodes()); + } + + // .. and drop it in this thread (to prevent deallocation in the RT thread) + + return _driver->machine(); +} + + /** Learn the SMF (MIDI) file at @a uri, and run the resulting machine * (replacing current machine). * Safe to call while engine is processing. @@ -56,7 +77,8 @@ Engine::learn_midi(const Glib::ustring& uri) SharedPtr file_driver(new SMFDriver()); SharedPtr m = file_driver->learn(uri, 32.0); // FIXME: hardcoded m->activate(); - _driver->set_machine(m); + //_driver->set_machine(m); + _driver->machine()->nodes().append(m->nodes()); // .. and drop it in this thread (to prevent deallocation in the RT thread) diff --git a/src/engine/JackDriver.cpp b/src/engine/JackDriver.cpp index 7df3a4b..3493105 100644 --- a/src/engine/JackDriver.cpp +++ b/src/engine/JackDriver.cpp @@ -294,6 +294,16 @@ JackDriver::on_process(jack_nframes_t nframes) } +void +JackDriver::reset() +{ + // FIXME: Flag audio thread and end active notes, etc + _machine->deactivate(); + _machine->reset(); + _cycle_time.set_start(0); +} + + void JackDriver::start_record() { @@ -314,7 +324,8 @@ JackDriver::finish_record() std::cout << "Learned machine! " << machine->nodes().size() << " nodes." << std::endl; _recorder.reset(); machine->activate(); - set_machine(machine); + //set_machine(machine); + _machine->nodes().append(machine->nodes()); } diff --git a/src/engine/Loader.cpp b/src/engine/Loader.cpp index 539c86f..b3e4c81 100644 --- a/src/engine/Loader.cpp +++ b/src/engine/Loader.cpp @@ -137,6 +137,11 @@ Loader::load(const Glib::ustring& uri) } + for (Created::iterator n = created.begin(); n != created.end(); ++n) { + cout << "NODE: " << n->first << endl; + } + + /* Get note actions */ query = Raul::RDFQuery(*_namespaces, Glib::ustring( @@ -152,6 +157,8 @@ Loader::load(const Glib::ustring& uri) const Glib::ustring& node_id = (*i)["node"]; const Glib::ustring& note = (*i)["note"]; + cerr << "NOTE: " << node_id << " = " << note << endl; + Created::iterator node_i = created.find(node_id); if (node_i != created.end()) { SharedPtr node = node_i->second; diff --git a/src/engine/MachineBuilder.cpp b/src/engine/MachineBuilder.cpp index 515f1a0..c8ad273 100644 --- a/src/engine/MachineBuilder.cpp +++ b/src/engine/MachineBuilder.cpp @@ -73,22 +73,22 @@ MachineBuilder::connect_nodes(SharedPtr m, SharedPtr delay_node; - cerr << "******" << endl; + /*cerr << "******" << endl; cerr << "Connect nodes durations: " << tail->duration() << " .. " << head->duration() << endl; - cerr << "Connect nodes times: " << tail_end_time << " .. " << head_start_time << endl; + cerr << "Connect nodes times: " << tail_end_time << " .. " << head_start_time << endl;*/ if (is_delay_node(tail) && tail->outgoing_edges().size() == 0) { // Tail is a delay node, just accumulate the time difference into it - cerr << "Accumulating delay " << tail_end_time << " .. " << head_start_time << endl; + //cerr << "Accumulating delay " << tail_end_time << " .. " << head_start_time << endl; tail->set_duration(tail->duration() + head_start_time - tail_end_time); tail->add_outgoing_edge(SharedPtr(new Edge(tail, head))); } else if (head_start_time == tail_end_time) { // Connect directly - cerr << "Connnecting directly " << tail_end_time << " .. " << head_start_time << endl; + //cerr << "Connnecting directly " << tail_end_time << " .. " << head_start_time << endl; tail->add_outgoing_edge(SharedPtr(new Edge(tail, head))); } else { // Need to actually create a delay node - cerr << "Adding delay node for " << tail_end_time << " .. " << head_start_time << endl; + //cerr << "Adding delay node for " << tail_end_time << " .. " << head_start_time << endl; delay_node = SharedPtr(new Node()); delay_node->set_duration(head_start_time - tail_end_time); tail->add_outgoing_edge(SharedPtr(new Edge(tail, delay_node))); @@ -96,7 +96,7 @@ MachineBuilder::connect_nodes(SharedPtr m, m->add_node(delay_node); } - cerr << "******" << endl << endl; + /*cerr << "******" << endl << endl;*/ return delay_node; } @@ -109,7 +109,7 @@ MachineBuilder::event(Raul::BeatTime time_offset, { Raul::BeatTime t = _time + time_offset; - cerr << "t = " << t << endl; + //cerr << "t = " << t << endl; if (ev_size > 0) { if ((buf[0] & 0xF0) == MIDI_CMD_NOTE_ON) { @@ -150,6 +150,8 @@ MachineBuilder::event(Raul::BeatTime time_offset, resolved->set_duration(t - resolved->enter_time()); if (_active_nodes.size() == 1) { + + //cerr << "{ RESOLVING, t= " << t << "\n"; _connect_node_end_time = t; @@ -176,7 +178,7 @@ MachineBuilder::event(Raul::BeatTime time_offset, if (is_delay_node(_connect_node) && _connect_node->duration() == 0 && _connect_node->outgoing_edges().size() == 1 && (*_connect_node->outgoing_edges().begin())->head() == resolved) { - cerr << "TRIMMING\n"; + //cerr << "TRIMMING\n"; _connect_node->outgoing_edges().clear(); assert(_connect_node->outgoing_edges().empty()); _connect_node->set_enter_action(resolved->enter_action()); @@ -188,12 +190,16 @@ MachineBuilder::event(Raul::BeatTime time_offset, if (_machine->nodes().find(_connect_node) == _machine->nodes().end()) _machine->add_node(_connect_node); } else { + //cerr << "RESOLVED\n"; _connect_node = resolved; _machine->add_node(resolved); } } + //cerr << "}"; + } else { + //cerr << "ADDING POLY\n"; _poly_nodes.push_back(make_pair(resolved->enter_time(), resolved)); } @@ -217,7 +223,7 @@ MachineBuilder::resolve() { if ( ! _active_nodes.empty()) { for (list >::iterator i = _active_nodes.begin(); i != _active_nodes.end(); ++i) { - cerr << "WARNING: Resolving stuck note from MIDI file." << endl; + cerr << "WARNING: Resolving stuck note." << endl; SharedPtr action = PtrCast((*i)->enter_action()); if (!action) continue; diff --git a/src/engine/machina/Driver.hpp b/src/engine/machina/Driver.hpp index bf4d2be..acc4278 100644 --- a/src/engine/machina/Driver.hpp +++ b/src/engine/machina/Driver.hpp @@ -1,4 +1,5 @@ /* This file is part of Machina. + * _engine->driver()->reset_time(); * Copyright (C) 2007 Dave Robillard * * Machina is free software; you can redistribute it and/or modify it under the @@ -39,6 +40,8 @@ public: virtual void activate() {} virtual void deactivate() {} + virtual void reset() {} + virtual bool recording() { return false; } virtual void start_record() {} virtual void finish_record() {} diff --git a/src/engine/machina/Engine.hpp b/src/engine/machina/Engine.hpp index 82c21ba..dc47b2a 100644 --- a/src/engine/machina/Engine.hpp +++ b/src/engine/machina/Engine.hpp @@ -37,6 +37,7 @@ public: SharedPtr machine() { return _driver->machine(); } SharedPtr load_machine(const Glib::ustring& uri); + SharedPtr import_machine(const Glib::ustring& uri); SharedPtr learn_midi(const Glib::ustring& uri); void set_bpm(double bpm); diff --git a/src/engine/machina/JackDriver.hpp b/src/engine/machina/JackDriver.hpp index 47b718d..1ab6643 100644 --- a/src/engine/machina/JackDriver.hpp +++ b/src/engine/machina/JackDriver.hpp @@ -62,6 +62,8 @@ public: void set_bpm(double bpm) { _bpm.set(bpm); } void set_quantization(double quantization) { _quantization.set(quantization); } + void reset(); + bool recording() { return _recording.get(); } void start_record(); void finish_record(); diff --git a/src/gui/EdgeView.cpp b/src/gui/EdgeView.cpp index fccf8d5..d8e11a9 100644 --- a/src/gui/EdgeView.cpp +++ b/src/gui/EdgeView.cpp @@ -41,12 +41,27 @@ EdgeView::length_hint() const } +void +EdgeView::show_label(bool show) +{ + if (show) { + char label[4]; + snprintf(label, 4, "%3f", _edge->probability()); + set_label(label); + } else { + set_label(""); + } +} + + void EdgeView::update_label() { - char label[4]; - snprintf(label, 4, "%3f", _edge->probability()); - set_label(label); + if (_label) { + char label[4]; + snprintf(label, 4, "%3f", _edge->probability()); + set_label(label); + } } diff --git a/src/gui/EdgeView.hpp b/src/gui/EdgeView.hpp index c394d49..61de52f 100644 --- a/src/gui/EdgeView.hpp +++ b/src/gui/EdgeView.hpp @@ -33,11 +33,12 @@ public: SharedPtr edge() { return _edge; } - void update_label(); + void show_label(bool show); virtual double length_hint() const; private: + void update_label(); bool on_event(GdkEvent* ev); SharedPtr _edge; diff --git a/src/gui/MachinaGUI.cpp b/src/gui/MachinaGUI.cpp index 3ddb13f..4098ff8 100644 --- a/src/gui/MachinaGUI.cpp +++ b/src/gui/MachinaGUI.cpp @@ -29,6 +29,7 @@ #include "MachinaGUI.hpp" #include "MachinaCanvas.hpp" #include "NodeView.hpp" +#include "EdgeView.hpp" MachinaGUI::MachinaGUI(SharedPtr engine) @@ -58,6 +59,7 @@ MachinaGUI::MachinaGUI(SharedPtr engine) xml->get_widget("export_midi_menuitem", _menu_export_midi); xml->get_widget("export_graphviz_menuitem", _menu_export_graphviz); xml->get_widget("view_toolbar_menuitem", _menu_view_toolbar); + xml->get_widget("view_labels_menuitem", _menu_view_labels); //xml->get_widget("view_refresh_menuitem", _menu_view_refresh); //xml->get_widget("view_messages_menuitem", _menu_view_messages); xml->get_widget("help_about_menuitem", _menu_help_about); @@ -115,6 +117,8 @@ MachinaGUI::MachinaGUI(SharedPtr engine) // sigc::mem_fun(this, &MachinaGUI::menu_view_refresh)); _menu_view_toolbar->signal_toggled().connect( sigc::mem_fun(this, &MachinaGUI::show_toolbar_toggled)); + _menu_view_labels->signal_toggled().connect( + sigc::mem_fun(this, &MachinaGUI::show_labels_toggled)); //_menu_view_messages->signal_toggled().connect( // sigc::mem_fun(this, &MachinaGUI::show_messages_toggled)); _menu_help_about->signal_activate().connect( @@ -346,7 +350,8 @@ MachinaGUI::menu_file_open() const int result = dialog.run(); if (result == Gtk::RESPONSE_OK) { - SharedPtr new_machine = _engine->load_machine(dialog.get_uri()); + //SharedPtr new_machine = _engine->load_machine(dialog.get_uri()); + SharedPtr new_machine = _engine->import_machine(dialog.get_uri()); if (new_machine) { _canvas->destroy(); _canvas->build(new_machine); @@ -596,6 +601,21 @@ MachinaGUI::show_toolbar_toggled() _toolbar->hide(); } + +void +MachinaGUI::show_labels_toggled() +{ + const bool show = _menu_view_labels->get_active(); + + for (ConnectionList::iterator c = _canvas->connections().begin(); + c != _canvas->connections().end(); ++c) { + const SharedPtr ev = PtrCast(*c); + if (ev) + ev->show_label(show); + } +} + + /* void MachinaGUI::menu_view_refresh() @@ -643,7 +663,7 @@ MachinaGUI::stop_clicked() _engine->driver()->finish_record(); } else { _engine->machine()->deactivate(); - _engine->machine()->reset(); + _engine->driver()->reset(); } update_toolbar(); diff --git a/src/gui/MachinaGUI.hpp b/src/gui/MachinaGUI.hpp index 96fb772..eba8a93 100644 --- a/src/gui/MachinaGUI.hpp +++ b/src/gui/MachinaGUI.hpp @@ -64,6 +64,7 @@ protected: void menu_export_graphviz(); //void show_messages_toggled(); void show_toolbar_toggled(); + void show_labels_toggled(); //void menu_view_refresh(); void menu_help_about(); void menu_help_help(); @@ -110,9 +111,10 @@ protected: Gtk::MenuItem* _menu_export_midi; Gtk::MenuItem* _menu_export_graphviz; Gtk::MenuItem* _menu_help_about; + Gtk::CheckMenuItem* _menu_view_labels; Gtk::CheckMenuItem* _menu_view_toolbar; //Gtk::CheckMenuItem* _menu_view_messages; - Gtk::MenuItem* _menu_view_refresh; + //Gtk::MenuItem* _menu_view_refresh; Gtk::MenuItem* _menu_help_help; Gtk::ScrolledWindow* _canvas_scrolledwindow; Gtk::TextView* _status_text; diff --git a/src/gui/NodePropertiesWindow.cpp b/src/gui/NodePropertiesWindow.cpp index 151c602..6ba3e47 100644 --- a/src/gui/NodePropertiesWindow.cpp +++ b/src/gui/NodePropertiesWindow.cpp @@ -34,11 +34,13 @@ NodePropertiesWindow::NodePropertiesWindow(BaseObjectType* cobject, const Glib:: xml->get_widget("node_properties_note_spinbutton", _note_spinbutton); xml->get_widget("node_properties_duration_spinbutton", _duration_spinbutton); + xml->get_widget("node_properties_apply_button", _apply_button); xml->get_widget("node_properties_cancel_button", _cancel_button); xml->get_widget("node_properties_ok_button", _ok_button); - _ok_button->signal_clicked().connect(sigc::mem_fun(this, &NodePropertiesWindow::ok_clicked)); + _apply_button->signal_clicked().connect(sigc::mem_fun(this, &NodePropertiesWindow::apply_clicked)); _cancel_button->signal_clicked().connect(sigc::mem_fun(this, &NodePropertiesWindow::cancel_clicked)); + _ok_button->signal_clicked().connect(sigc::mem_fun(this, &NodePropertiesWindow::ok_clicked)); } @@ -48,11 +50,10 @@ NodePropertiesWindow::~NodePropertiesWindow() void -NodePropertiesWindow::ok_clicked() +NodePropertiesWindow::apply_clicked() { - assert(this == _instance); - delete _instance; - _instance = NULL; + double duration = _duration_spinbutton->get_value(); + _node->set_duration(duration); } @@ -64,6 +65,14 @@ NodePropertiesWindow::cancel_clicked() _instance = NULL; } + +void +NodePropertiesWindow::ok_clicked() +{ + apply_clicked(); + cancel_clicked(); +} + void NodePropertiesWindow::set_node(SharedPtr node) diff --git a/src/gui/NodePropertiesWindow.hpp b/src/gui/NodePropertiesWindow.hpp index 461b51b..51ab42b 100644 --- a/src/gui/NodePropertiesWindow.hpp +++ b/src/gui/NodePropertiesWindow.hpp @@ -34,8 +34,9 @@ private: void set_node(SharedPtr node); - void ok_clicked(); + void apply_clicked(); void cancel_clicked(); + void ok_clicked(); static NodePropertiesWindow* _instance; @@ -43,6 +44,7 @@ private: Gtk::SpinButton* _note_spinbutton; Gtk::SpinButton* _duration_spinbutton; + Gtk::Button* _apply_button; Gtk::Button* _cancel_button; Gtk::Button* _ok_button; }; diff --git a/src/gui/machina.glade b/src/gui/machina.glade index fb61bd8..b5349be 100644 --- a/src/gui/machina.glade +++ b/src/gui/machina.glade @@ -46,6 +46,7 @@ True + Open a saved machine gtk-open True @@ -55,6 +56,7 @@ True + Save machine gtk-save True @@ -64,6 +66,7 @@ True + Save machine to file gtk-save-as True @@ -79,6 +82,7 @@ True + Import a MIDI file _Import MIDI... True @@ -101,6 +105,7 @@ True + Export a MIDI file _Export MIDI... True @@ -129,7 +134,7 @@ True - Export a DOT file for rendering diagram with the GraphViz suite of tools + Export a DOT file for rendering with GraphViz Export _GraphViz... True @@ -158,6 +163,7 @@ True + Exit Machina gtk-quit True @@ -836,12 +842,12 @@ Connect some nodes up and double click one. You'll get it. 8 - Node Properties + dialog1 GTK_WINDOW_TOPLEVEL GTK_WIN_POS_NONE False False - True + False True True True @@ -852,16 +858,29 @@ Connect some nodes up and double click one. You'll get it. True - + True False 8 - + True GTK_BUTTONBOX_END + + + True + True + True + gtk-apply + True + GTK_RELIEF_NORMAL + True + -10 + + + True @@ -879,6 +898,7 @@ Connect some nodes up and double click one. You'll get it. True True + True True gtk-ok True @@ -908,7 +928,6 @@ Connect some nodes up and double click one. You'll get it. True - False True 1 0 @@ -986,8 +1005,8 @@ Connect some nodes up and double click one. You'll get it. True - False True + True 1 2 True diff --git a/src/gui/main.cpp b/src/gui/main.cpp index c522898..e1e86ea 100644 --- a/src/gui/main.cpp +++ b/src/gui/main.cpp @@ -51,6 +51,11 @@ main(int argc, char** argv) cout << "No quantization." << endl; machine = file_driver->learn(filename); } + + if (!machine) { + cout << "Not a MIDI file. Attempting to load as Machina file." << endl; + machine = Loader().load(filename); + } } if (!machine) -- cgit v1.2.1