]> git.sesse.net Git - casparcg/blobdiff - modules/psd/psd_scene_producer.cpp
[psd] #532 Fixed text position when animating.
[casparcg] / modules / psd / psd_scene_producer.cpp
index cf16622681647b8dfa1e6dd620f3a7b0612bb2e3..0999302819b15637fee912a50aeed640a992beb5 100644 (file)
@@ -131,9 +131,9 @@ public:
 
        spl::shared_ptr<core::scene::scene_producer> scene() { return scene_; }
 
-       void add(caspar::core::scene::layer* layer, layer_tag tags, double adjustment_x, double adjustment_y, bool vector_mask)
+       void add(caspar::core::scene::layer* layer, layer_tag tags, bool vector_mask)
        {
-               layer_record rec{ layer, tags, adjustment_x, adjustment_y, vector_mask };
+               layer_record rec{ layer, tags, 0, 0, vector_mask };
                layers.push_back(rec);
 
                //if the layer is either explicitly tagged as dynamic or at least not tagged as static/rasterized we should try to set it as master
@@ -200,12 +200,11 @@ public:
        }
 };
 
-int64_t get_frame_number(
+boost::rational<int> get_frame_number(
                const core::video_format_desc& format_desc,
                const boost::rational<int>& at_second)
 {
-       return static_cast<int64_t>(
-                       boost::rational_cast<double>(at_second) * format_desc.fps);
+       return at_second * format_desc.framerate;
 }
 
 boost::rational<int> get_rational(const boost::property_tree::wptree& node)
@@ -227,7 +226,7 @@ void create_marks_from_comments(
        for (auto& key : *keylist)
        {
                auto time = get_rational(key.second.get_child(L"time"));
-               auto frame = get_frame_number(format_desc, time);
+               auto frame = boost::rational_cast<int64_t>(get_frame_number(format_desc, time));
                auto text = key.second.get<std::wstring>(L"animKey.Vl  ");
                std::vector<std::wstring> marks;
                boost::split(marks, text, boost::is_any_of(","));
@@ -252,7 +251,7 @@ void create_marks(
        auto time = get_rational(global_timeline.get_child(L"duration"));
        auto remove_at_frame = get_frame_number(format_desc, time);
 
-       scene->add_mark(remove_at_frame, core::scene::mark_action::remove, L"");
+       scene->add_mark(boost::rational_cast<int64_t>(remove_at_frame), core::scene::mark_action::remove, L"");
 
        auto tracklist = global_timeline.get_child_optional(L"globalTrackList");
 
@@ -272,26 +271,27 @@ void create_timelines(
                const spl::shared_ptr<core::scene::scene_producer>& scene,
                const core::video_format_desc& format_desc,
                core::scene::layer& layer,
-               const layer_ptr& psd_layer,
-               double adjustment_x,
-               double adjustment_y)
+               const layer_ptr& psd_layer)
 {
-       auto timeline = psd_layer->timeline_data();
-       auto start = get_rational(timeline.get_child(L"timeScope.Strt"));
-       auto end_offset = get_rational(timeline.get_child(L"timeScope.outTime"));
-       auto end = start + end_offset;
-       auto start_frame = get_frame_number(format_desc, start);
-       auto end_frame = get_frame_number(format_desc, end);
+       auto timeline           = psd_layer->timeline_data();
+       auto start                      = get_rational(timeline.get_child(L"timeScope.Strt"));
+       auto end_offset         = get_rational(timeline.get_child(L"timeScope.outTime"));
+       auto end                        = start + end_offset;
+       auto start_frame        = get_frame_number(format_desc, start);
+       auto end_frame          = get_frame_number(format_desc, end);
 
-       layer.hidden = scene->timeline_frame() < start_frame || scene->timeline_frame() > end_frame;
+       layer.hidden =
+                       scene->timeline_frame() < boost::rational_cast<int64_t>(start_frame)
+                       || scene->timeline_frame() > boost::rational_cast<int64_t>(end_frame);
 
        auto tracklist = timeline.get_child_optional(L"trackList");
 
        if (!tracklist)
                return;
 
-       double original_pos_x = psd_layer->location().x;
-       double original_pos_y = psd_layer->location().y;
+       bool is_text_field              = psd_layer->is_text() && !psd_layer->is_static();
+       double original_pos_x   = is_text_field ? psd_layer->text_pos().x : psd_layer->location().x;
+       double original_pos_y   = is_text_field ? psd_layer->text_pos().y : psd_layer->location().y;
 
        for (auto& track : *tracklist)
        {
@@ -301,14 +301,13 @@ void create_timelines(
                {
                        for (auto& key : track.second.get_child(L"keyList"))
                        {
-                               bool tween = key.second.get<std::wstring>(L"animInterpStyle")
-                                               == L"Lnr ";
-                               auto time = get_rational(key.second.get_child(L"time"));
-                               auto hrzn = key.second.get<double>(L"animKey.Hrzn");
-                               auto vrtc = key.second.get<double>(L"animKey.Vrtc");
-                               auto x = original_pos_x + hrzn + adjustment_x;
-                               auto y = original_pos_y + vrtc + adjustment_y;
-                               auto frame = get_frame_number(format_desc, time);
+                               bool tween      = key.second.get<std::wstring>(L"animInterpStyle") == L"Lnr ";
+                               auto time       = get_rational(key.second.get_child(L"time"));
+                               auto hrzn       = key.second.get<double>(L"animKey.Hrzn");
+                               auto vrtc       = key.second.get<double>(L"animKey.Vrtc");
+                               auto x          = original_pos_x + hrzn;
+                               auto y          = original_pos_y + vrtc;
+                               auto frame      = get_frame_number(format_desc, time);
 
                                if (frame == 0) // Consider as initial value (rewind)
                                {
@@ -320,13 +319,13 @@ void create_timelines(
 
                                if (tween)
                                {
-                                       scene->add_keyframe(layer.position.x, x, frame, L"easeOutSine");
-                                       scene->add_keyframe(layer.position.y, y, frame, L"easeOutSine");
+                                       scene->add_keyframe(layer.position.x, x, boost::rational_cast<int64_t>(frame), L"easeOutSine");
+                                       scene->add_keyframe(layer.position.y, y, boost::rational_cast<int64_t>(frame), L"easeOutSine");
                                }
                                else
                                {
-                                       scene->add_keyframe(layer.position.x, x, frame);
-                                       scene->add_keyframe(layer.position.y, y, frame);
+                                       scene->add_keyframe(layer.position.x, x, boost::rational_cast<int64_t>(frame));
+                                       scene->add_keyframe(layer.position.y, y, boost::rational_cast<int64_t>(frame));
                                }
                        }
                }
@@ -336,11 +335,10 @@ void create_timelines(
 
                        for (auto& key : track.second.get_child(L"keyList"))
                        {
-                               bool tween = key.second.get<std::wstring>(L"animInterpStyle")
-                                               == L"Lnr ";
-                               auto time = get_rational(key.second.get_child(L"time"));
-                               auto opct = key.second.get<double>(L"animKey.Opct.#Prc") / 100.0;
-                               auto frame = get_frame_number(format_desc, time);
+                               bool tween      = key.second.get<std::wstring>(L"animInterpStyle") == L"Lnr ";
+                               auto time       = get_rational(key.second.get_child(L"time"));
+                               auto opct       = key.second.get<double>(L"animKey.Opct.#Prc") / 100.0;
+                               auto frame      = get_frame_number(format_desc, time);
 
                                if (frame == 0) // Consider as initial value (rewind)
                                        opacity.set(opct);
@@ -348,9 +346,9 @@ void create_timelines(
                                frame = start_frame + frame; // translate to global timeline
 
                                if (tween)
-                                       scene->add_keyframe(opacity, opct, frame, L"easeOutSine");
+                                       scene->add_keyframe(opacity, opct, boost::rational_cast<int64_t>(frame), L"linear");
                                else
-                                       scene->add_keyframe(opacity, opct, frame);
+                                       scene->add_keyframe(opacity, opct, boost::rational_cast<int64_t>(frame));
                        }
                }
                else
@@ -405,10 +403,10 @@ spl::shared_ptr<core::frame_producer> create_psd_scene_producer(const core::fram
                        scene_layer.hidden.set(!psd_layer->is_visible());
 
                        if (psd_layer->has_timeline())
-                               create_timelines(root, dependencies.format_desc, scene_layer, psd_layer, 0, 0);
+                               create_timelines(root, dependencies.format_desc, scene_layer, psd_layer);
 
                        if (psd_layer->is_movable())
-                               current.add(&scene_layer, psd_layer->tags(), 0, 0, false);
+                               current.add(&scene_layer, psd_layer->tags(), false);
 
                        if (psd_layer->is_resizable())  //TODO: we could add support for resizable groups with vector masks
                                CASPAR_LOG(warning) << "Groups doesn't support the \"resizable\"-tag.";
@@ -438,8 +436,6 @@ spl::shared_ptr<core::frame_producer> create_psd_scene_producer(const core::fram
                        caspar::core::scene::layer* scene_layer = nullptr;
                        std::shared_ptr<core::frame_producer> layer_producer;
                        auto layer_name = psd_layer->name();
-                       int adjustment_x = 0,
-                               adjustment_y = 0;
 
                        if(psd_layer->is_text() && !psd_layer->is_static())
                        {
@@ -457,10 +453,8 @@ spl::shared_ptr<core::frame_producer> create_psd_scene_producer(const core::fram
                                //text_producer->pixel_constraints().height.set(psd_layer->size().height);
                                core::text::string_metrics metrics = text_producer->measure_string(str);
 
-                               //adjustment_x = -2;    //the 2 offset is just a hack for now. don't know why our text is rendered 2 px to the right of that in photoshop
-                               //adjustment_y = metrics.bearingY;
                                layer_producer = text_producer;
-                               scene_layer = &current.scene()->create_layer(spl::make_shared_ptr(layer_producer), static_cast<int>(psd_layer->text_pos().x) + adjustment_x, static_cast<int>(psd_layer->text_pos().y) + adjustment_y, layer_name);
+                               scene_layer = &current.scene()->create_layer(spl::make_shared_ptr(layer_producer), static_cast<int>(psd_layer->text_pos().x), static_cast<int>(psd_layer->text_pos().y), layer_name);
 
                                int justification = psd_layer->text_data().get(L"EngineDict.ParagraphRun.RunArray..ParagraphSheet.Properties.Justification", 0);
 
@@ -516,7 +510,7 @@ spl::shared_ptr<core::frame_producer> create_psd_scene_producer(const core::fram
                                }
 
                                if(layer_producer)
-                                       scene_layer = &current.scene()->create_layer(spl::make_shared_ptr(layer_producer), psd_layer->location().x + adjustment_x, psd_layer->location().y + adjustment_y, layer_name);
+                                       scene_layer = &current.scene()->create_layer(spl::make_shared_ptr(layer_producer), psd_layer->location().x, psd_layer->location().y, layer_name);
                        }
 
                        if (layer_producer && scene_layer)
@@ -581,10 +575,10 @@ spl::shared_ptr<core::frame_producer> create_psd_scene_producer(const core::fram
                                }
 
                                if (psd_layer->has_timeline())
-                                       create_timelines(root, dependencies.format_desc, *scene_layer, psd_layer, adjustment_x, adjustment_y);
+                                       create_timelines(root, dependencies.format_desc, *scene_layer, psd_layer);
 
                                if (psd_layer->is_movable() || psd_layer->is_resizable() || (psd_layer->is_text() && !psd_layer->is_static()))
-                                       current.add(scene_layer, psd_layer->tags(), -adjustment_x, -adjustment_y, psd_layer->mask().has_vector());
+                                       current.add(scene_layer, psd_layer->tags(), psd_layer->mask().has_vector());
 
                                if (psd_layer->is_placeholder())
                                        scene_layer->use_mipmap.set(true);