debug ci

Author: romanrizzi

lib/sentiment/post_classification.rb

@@ -5,7 +5,10 @@ module Sentiment
     class PostClassification
       def self.backfill_query(from_post_id: nil, max_age_days: nil)
         available_classifier_names =
-          DiscourseAi::Sentiment::SentimentSiteSettingJsonSchema.values.map(&:model_name).sort
+          DiscourseAi::Sentiment::SentimentSiteSettingJsonSchema
+            .values
+            .map { |mc| mc.model_name.downcase }
+            .sort
 
         base_query =
           Post
@@ -24,7 +27,7 @@ def self.backfill_query(from_post_id: nil, max_age_days: nil)
             .group("posts.id")
             .having(<<~SQL, available_classifier_names)
               COUNT(crs.model_used) = 0
-              OR array_agg(DISTINCT crs.model_used ORDER BY crs.model_used)::text[] IS DISTINCT FROM array[?]
+              OR array_agg(DISTINCT crs.model_used ORDER BY LOWER(crs.model_used))::text[] IS DISTINCT FROM array[?]
             SQL
 
         base_query = base_query.where("posts.id >= ?", from_post_id.to_i) if from_post_id.present?

spec/lib/modules/sentiment/post_classification_spec.rb

@@ -159,6 +159,18 @@ def check_classification_for(post)
         Fabricate(:classification_result, target: classified_post, model_used: cm["model_name"])
       end
 
+      pp Post
+           .where(id: classified_post.id)
+           .joins(<<~SQL)
+      LEFT JOIN classification_results crs
+        ON crs.target_id = posts.id 
+        AND crs.target_type = 'Post'
+        AND crs.classification_type = 'sentiment'
+    SQL
+           .select("array_agg(DISTINCT crs.model_used ORDER BY crs.model_used ASC)")
+      pp "\n"
+      pp DiscourseAi::Sentiment::SentimentSiteSettingJsonSchema.values.map(&:model_name).sort
+
       posts = described_class.backfill_query
       expect(posts).not_to include(classified_post)