gabehf/sqm-dashboards
1
0
Fork 0
mirror of https://github.com/edcommonwealth/sqm-dashboards.git synced 2026-03-07 21:48:16 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

perf: Add another index to surveyitemresponses. Implement bullet suggestions. Memoize a function in measure.rb

Browse source
This commit is contained in:
rebuilt 2023-06-10 10:19:05 -07:00
parent eee0cb14a3
commit 452876ef09
4 changed files with 19 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/controllers/overview_controller.rb
View file

@ -25,10 +25,10 @@ class OverviewController < SqmApplicationController
end
def measures
@measures ||= Measure.all.includes(%i[scales admin_data_items category])
@measures ||= Measure.all.includes(%i[scales admin_data_items category subcategory])
end
def subcategories
@subcategories ||= Subcategory.all
@subcategories ||= Subcategory.all.includes(:measures)
end
end

17
app/models/measure.rb
View file

@ -25,14 +25,15 @@ class Measure < ActiveRecord::Base
end
def student_survey_items_with_sufficient_responses(school:, academic_year:)
SurveyItem.where(id: SurveyItem.joins('inner join survey_item_responses on survey_item_responses.survey_item_id = survey_items.id')
.student_survey_items
.where("survey_item_responses.school": school,
"survey_item_responses.academic_year": academic_year,
"survey_item_responses.survey_item_id": survey_items.student_survey_items)
.group('survey_items.id')
.having('count(*) >= 10')
.count.keys)
@student_survey_items_with_sufficient_responses ||=
SurveyItem.where(id: SurveyItem.joins('inner join survey_item_responses on survey_item_responses.survey_item_id = survey_items.id')
.student_survey_items
.where("survey_item_responses.school": school,
"survey_item_responses.academic_year": academic_year,
"survey_item_responses.survey_item_id": survey_items.student_survey_items)
.group('survey_items.id')
.having('count(*) >= 10')
.count.keys)
end
def teacher_scales

6
db/migrate/20230610165508_add_survey_item_index_to_survey_item_responses.rb Normal file
View file

@ -0,0 +1,6 @@
class AddSurveyItemIndexToSurveyItemResponses < ActiveRecord::Migration[7.0]
def change
add_index :survey_item_responses, %i[school_id academic_year_id survey_item_id],
name: 'by_school_year_and_survey_item'
end
end

3
db/schema.rb
View file

@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.
ActiveRecord::Schema[7.0].define(version: 2023_04_21_034505) do
ActiveRecord::Schema[7.0].define(version: 2023_06_10_165508) do
# These are extensions that must be enabled in order to support this database
enable_extension "plpgsql"
@ -447,6 +447,7 @@ ActiveRecord::Schema[7.0].define(version: 2023_04_21_034505) do
t.index ["academic_year_id"], name: "index_survey_item_responses_on_academic_year_id"
t.index ["gender_id"], name: "index_survey_item_responses_on_gender_id"
t.index ["response_id"], name: "index_survey_item_responses_on_response_id"
t.index ["school_id", "academic_year_id", "survey_item_id"], name: "by_school_year_and_survey_item"
t.index ["school_id", "academic_year_id"], name: "index_survey_item_responses_on_school_id_and_academic_year_id"
t.index ["school_id", "survey_item_id", "academic_year_id", "grade"], name: "index_survey_responses_on_grade"
t.index ["school_id"], name: "index_survey_item_responses_on_school_id"