Merge pull request #12 from Wovchena/remove-negative_prompt-from-signature

Remove negative_prompt from signature

Author: likholat

samples/cpp/video_generation/text2video.cpp

@@ -57,7 +57,7 @@ int main(int32_t argc, char* argv[]) {
     pipe.set_generation_config(config);
     auto output = pipe.generate(
         prompt,
-        "worst quality, inconsistent motion, blurry, jittery, distorted",
+        ov::genai::negative_prompt("worst quality, inconsistent motion, blurry, jittery, distorted"),
         ov::genai::height(512),  // OVLTXPipeline's default
         ov::genai::width(704),  // OVLTXPipeline's default
         ov::genai::num_inference_steps(10),

src/cpp/include/openvino/genai/video_generation/text2video_pipeline.hpp

@@ -28,23 +28,20 @@ class OPENVINO_GENAI_EXPORTS Text2VideoPipeline {
     /**
      * Generates image(s) based on prompt and other image generation parameters
      * @param positive_prompt Prompt to generate image(s) from
-     * @param negative_prompt
      * @param properties Image generation parameters specified as properties. Values in 'properties' override default value for generation parameters.
      * @returns A tensor which has dimensions [num_images_per_prompt, height, width, 3]
      */
     VideoGenerationResult generate(
         const std::string& positive_prompt,
-        const std::string& negative_prompt = "",
         const ov::AnyMap& properties = {}
     );
 
     template <typename... Properties>
     ov::util::EnableIfAllStringAny<VideoGenerationResult, Properties...> generate(
         const std::string& positive_prompt,
-        const std::string& negative_prompt,
         Properties&&... properties
     ) {
-        return generate(positive_prompt, negative_prompt, ov::AnyMap{std::forward<Properties>(properties)...});
+        return generate(positive_prompt, ov::AnyMap{std::forward<Properties>(properties)...});
     }
 
     const VideoGenerationConfig& get_generation_config() const;

src/cpp/src/video_generation/text2video_pipeline.cpp

@@ -452,7 +452,7 @@ class Text2VideoPipeline::LTXPipeline {
         return out_vec[0];  // [B, F, H, W, C]
 }
 
-    VideoGenerationResult generate(const std::string& positive_prompt, const std::string& negative_prompt, const ov::AnyMap& properties = {}) {
+    VideoGenerationResult generate(const std::string& positive_prompt, const ov::AnyMap& properties = {}) {
         const auto gen_start = std::chrono::steady_clock::now();
         m_perf_metrics.clean_up();
 
@@ -473,7 +473,7 @@ class Text2VideoPipeline::LTXPipeline {
             callback = callback_iter->second.as<std::function<bool(size_t, size_t, ov::Tensor&)>>();
         }
 
-        compute_hidden_states(positive_prompt, negative_prompt, merged_generation_config);
+        compute_hidden_states(positive_prompt, merged_generation_config.negative_prompt.value_or(""), merged_generation_config);
 
         size_t num_channels_latents = m_transformer->get_config().in_channels;
         size_t spatial_compression_ratio = m_vae->get_config().patch_size * std::pow(2, std::reduce(m_vae->get_config().spatio_temporal_scaling.begin(), m_vae->get_config().spatio_temporal_scaling.end(), 0));
@@ -637,10 +637,9 @@ Text2VideoPipeline::Text2VideoPipeline(
 
 VideoGenerationResult Text2VideoPipeline::generate(
     const std::string& positive_prompt,
-    const std::string& negative_prompt,
     const ov::AnyMap& properties
 ) {
-    return m_impl->generate(positive_prompt, negative_prompt, properties);
+    return m_impl->generate(positive_prompt, properties);
 }
 
 const VideoGenerationConfig& Text2VideoPipeline::get_generation_config() const {