From c4125b32ee10fcda54be12a7cba6746c5684e673 Mon Sep 17 00:00:00 2001 From: Manas Solanki Date: Fri, 5 May 2017 12:54:25 +0530 Subject: [PATCH 1/3] Course wise assessment reports --- .../course_wise_assessment_report/__init__.py | 0 .../course_wise_assessment_report.js | 21 ++++++++ .../course_wise_assessment_report.json | 23 ++++++++ .../course_wise_assessment_report.py | 54 +++++++++++++++++++ 4 files changed, 98 insertions(+) create mode 100644 erpnext/schools/report/course_wise_assessment_report/__init__.py create mode 100644 erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js create mode 100644 erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.json create mode 100644 erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py diff --git a/erpnext/schools/report/course_wise_assessment_report/__init__.py b/erpnext/schools/report/course_wise_assessment_report/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js new file mode 100644 index 0000000000..d61d59a5fc --- /dev/null +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js @@ -0,0 +1,21 @@ +// Copyright (c) 2016, Frappe Technologies Pvt. Ltd. and contributors +// For license information, please see license.txt + +frappe.query_reports["Course wise Assessment Report"] = { + "filters": [ + { + "fieldname":"assessment_group", + "label": __("Assessment Group"), + "fieldtype": "Link", + "options": "Assessment Group", + "reqd": 1 + }, + { + "fieldname":"student_group", + "label": __("Student Group"), + "fieldtype": "Link", + "options": "Student Group", + "reqd": 1 + }, + ] +} diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.json b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.json new file mode 100644 index 0000000000..6b089d276f --- /dev/null +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.json @@ -0,0 +1,23 @@ +{ + "add_total_row": 0, + "apply_user_permissions": 1, + "creation": "2017-05-05 14:46:13.776133", + "disabled": 0, + "docstatus": 0, + "doctype": "Report", + "idx": 0, + "is_standard": "Yes", + "modified": "2017-05-05 14:47:18.080385", + "modified_by": "Administrator", + "module": "Schools", + "name": "Course wise Assessment Report", + "owner": "Administrator", + "ref_doctype": "Assessment Result", + "report_name": "Course wise Assessment Report", + "report_type": "Script Report", + "roles": [ + { + "role": "Academics User" + } + ] +} \ No newline at end of file diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py new file mode 100644 index 0000000000..e1a5a134e2 --- /dev/null +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py @@ -0,0 +1,54 @@ +# Copyright (c) 2013, Frappe Technologies Pvt. Ltd. and contributors +# For license information, please see license.txt + +from __future__ import unicode_literals +import frappe +from frappe import _ +from collections import defaultdict + +def execute(filters=None): + + assessment_group = filters.get("assessment_group") + student_group = filters.get("student_group") + course = frappe.db.get_value("Student Group", student_group, "course") + if not course: + frappe.throw(_("Student Group {0} is not linked with any course").format(student_group)) + + assessment_plan = frappe.db.sql('''select ap.name, apc.assessment_criteria, apc.maximum_score as max_score + from `tabAssessment Plan` ap, `tabAssessment Plan Criteria` apc + where ap.assessment_group=%s and ap.student_group=%s and ap.name=apc.parent and ap.docstatus=1 + order by apc.assessment_criteria''', (assessment_group, student_group), as_dict=1) + assessment_plan_list = set([d["name"] for d in assessment_plan]) + + assessment_result = frappe.db.sql('''select ar.student, ard.assessment_criteria, ard.grade, ard.score + from `tabAssessment Result` ar, `tabAssessment Result Detail` ard + where ar.assessment_plan in (%s) and ar.name=ard.parent and ar.docstatus=1 + order by ard.assessment_criteria''' %', '.join(['%s']*len(assessment_plan_list)), tuple(assessment_plan_list), as_dict=1) + + result_dict = defaultdict(list) + for result in assessment_result: + result_dict[result.student].append(result.grade) + result_dict[result.student].append(result.score) + + student_list = frappe.db.sql('''select sgs.group_roll_number, sgs.student, sgs.student_name + from `tabStudent Group` sg, `tabStudent Group Student` sgs + where sg.name = sgs.parent and sg.name = %s + order by sgs.group_roll_number asc''', (student_group), as_list=1) + + data = [] + for student in student_list: + tmp_list = student + result_dict[student[1]] + data.append(tmp_list) + + return get_column(assessment_plan), data + +def get_column(assessment_plan): + columns = [ + _("Group Roll No") + "::80", + _("Student ID") + ":Link/Student:90", + _("Student Name") + "::160", + ] + for d in assessment_plan: + columns.append(d.get("assessment_criteria") + "::110") + columns.append("Max Score(" + str(int(d.get("max_score"))) + ")::110") + return columns From 85480b3af73dd01d63c1bf2f391012a1dd67dfff Mon Sep 17 00:00:00 2001 From: Manas Solanki Date: Fri, 5 May 2017 15:04:59 +0530 Subject: [PATCH 2/3] Added report in config/schools.py --- erpnext/config/schools.py | 9 +++- .../course_wise_assessment_report.js | 7 ++- .../course_wise_assessment_report.py | 44 +++++++++++++------ 3 files changed, 44 insertions(+), 16 deletions(-) diff --git a/erpnext/config/schools.py b/erpnext/config/schools.py index c54f808bbc..b984578ca1 100644 --- a/erpnext/config/schools.py +++ b/erpnext/config/schools.py @@ -137,7 +137,14 @@ def get_data(): { "type": "doctype", "name": "Assessment Result Tool" - } + }, + { + "type": "report", + "is_query_report": True, + "name": "Course wise Assessment Report", + "doctype": "Assessment Result" + }, + ] }, { diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js index d61d59a5fc..a359a5f82b 100644 --- a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js @@ -15,7 +15,12 @@ frappe.query_reports["Course wise Assessment Report"] = { "label": __("Student Group"), "fieldtype": "Link", "options": "Student Group", - "reqd": 1 + }, + { + "fieldname":"course", + "label": __("Course"), + "fieldtype": "Link", + "options": "Course", }, ] } diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py index e1a5a134e2..ce029d715e 100644 --- a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py @@ -7,18 +7,34 @@ from frappe import _ from collections import defaultdict def execute(filters=None): + print "======================================" assessment_group = filters.get("assessment_group") student_group = filters.get("student_group") - course = frappe.db.get_value("Student Group", student_group, "course") - if not course: - frappe.throw(_("Student Group {0} is not linked with any course").format(student_group)) - assessment_plan = frappe.db.sql('''select ap.name, apc.assessment_criteria, apc.maximum_score as max_score + if student_group: + course = frappe.db.get_value("Student Group", student_group, "course") + if not course: + frappe.throw(_("Student Group {0} is not linked with any course").format(student_group)) + # student_group_list = [student_group] + else: + course = filters.get("course") + if not course: + frappe.throw(_("Please select Student Group or Course")) + # student_group_list = frappe.get_list("Student Group", fields=["name"], filters={"program":program, "course":course}) + + # find assessment plan according to the student group list + assessment_plan = frappe.db.sql('''select ap.name, ap.student_group, apc.assessment_criteria, apc.maximum_score as max_score from `tabAssessment Plan` ap, `tabAssessment Plan Criteria` apc - where ap.assessment_group=%s and ap.student_group=%s and ap.name=apc.parent and ap.docstatus=1 - order by apc.assessment_criteria''', (assessment_group, student_group), as_dict=1) + where ap.assessment_group=%s and ap.course=%s and ap.name=apc.parent and ap.docstatus=1 + order by apc.assessment_criteria''', (assessment_group, course), as_dict=1) + print assessment_plan assessment_plan_list = set([d["name"] for d in assessment_plan]) + student_group_list = set([d["student_group"] for d in assessment_plan]) + assessment_criteria_list = set([(d["assessment_criteria"],d["max_score"]) for d in assessment_plan]) + assessment_plan + if not assessment_plan_list: + frappe.throw(_("No assessment plan linked with this assessment group")) assessment_result = frappe.db.sql('''select ar.student, ard.assessment_criteria, ard.grade, ard.score from `tabAssessment Result` ar, `tabAssessment Result Detail` ard @@ -32,23 +48,23 @@ def execute(filters=None): student_list = frappe.db.sql('''select sgs.group_roll_number, sgs.student, sgs.student_name from `tabStudent Group` sg, `tabStudent Group Student` sgs - where sg.name = sgs.parent and sg.name = %s - order by sgs.group_roll_number asc''', (student_group), as_list=1) + where sg.name = sgs.parent and sg.name in (%s) + order by sgs.group_roll_number asc''' %', '.join(['%s']*len(student_group_list)), tuple(student_group_list), as_list=1) data = [] for student in student_list: tmp_list = student + result_dict[student[1]] data.append(tmp_list) - return get_column(assessment_plan), data + return get_column(assessment_criteria_list), data -def get_column(assessment_plan): +def get_column(assessment_criteria): columns = [ - _("Group Roll No") + "::80", + _("Batch Roll No") + "::60", _("Student ID") + ":Link/Student:90", _("Student Name") + "::160", ] - for d in assessment_plan: - columns.append(d.get("assessment_criteria") + "::110") - columns.append("Max Score(" + str(int(d.get("max_score"))) + ")::110") + for d in assessment_criteria: + columns.append(d[0] + "::110") + columns.append("Score(" + str(int(d[1])) + ")::100") return columns From bb68f8ca01e128975cb6cd1ec2f2b67fdcf6a0ee Mon Sep 17 00:00:00 2001 From: Manas Solanki Date: Thu, 29 Jun 2017 16:33:53 +0530 Subject: [PATCH 3/3] cleanup for the assessment report and added chart --- .../course_wise_assessment_report.js | 28 ++-- .../course_wise_assessment_report.py | 132 ++++++++++++------ 2 files changed, 110 insertions(+), 50 deletions(-) diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js index a359a5f82b..42b19ebf06 100644 --- a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.js @@ -8,19 +8,27 @@ frappe.query_reports["Course wise Assessment Report"] = { "label": __("Assessment Group"), "fieldtype": "Link", "options": "Assessment Group", - "reqd": 1 - }, - { - "fieldname":"student_group", - "label": __("Student Group"), - "fieldtype": "Link", - "options": "Student Group", - }, + "reqd": 1, + "get_query": function() { + return{ + filters: { + 'is_group': 0 + } + }; + } + }, { "fieldname":"course", "label": __("Course"), "fieldtype": "Link", "options": "Course", - }, + "reqd": 1 + }, + { + "fieldname":"student_group", + "label": __("Student Group"), + "fieldtype": "Link", + "options": "Student Group" + } ] -} +}; diff --git a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py index ce029d715e..b5a2fc1f07 100644 --- a/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py +++ b/erpnext/schools/report/course_wise_assessment_report/course_wise_assessment_report.py @@ -7,64 +7,116 @@ from frappe import _ from collections import defaultdict def execute(filters=None): - print "======================================" + args = frappe._dict() + args["assessment_group"] = filters.get("assessment_group") + if args["assessment_group"] == "All Assessment Groups": + frappe.throw(_("Please select the assessment group other than 'All Assessment Groups'")) - assessment_group = filters.get("assessment_group") - student_group = filters.get("student_group") - - if student_group: - course = frappe.db.get_value("Student Group", student_group, "course") - if not course: - frappe.throw(_("Student Group {0} is not linked with any course").format(student_group)) - # student_group_list = [student_group] + args["course"] = filters.get("course") + args["student_group"] = filters.get("student_group") + if args["student_group"]: + cond = "and ap.student_group=%(student_group)s" else: - course = filters.get("course") - if not course: - frappe.throw(_("Please select Student Group or Course")) - # student_group_list = frappe.get_list("Student Group", fields=["name"], filters={"program":program, "course":course}) + cond = '' + + # find all assessment plan linked with the filters provided + assessment_plan = frappe.db.sql(''' + select + ap.name, ap.student_group, apc.assessment_criteria, apc.maximum_score as max_score + from + `tabAssessment Plan` ap, `tabAssessment Plan Criteria` apc + where + ap.assessment_group=%(assessment_group)s and ap.course=%(course)s and + ap.name=apc.parent and ap.docstatus=1 {0} + order by + apc.assessment_criteria'''.format(cond), (args), as_dict=1) - # find assessment plan according to the student group list - assessment_plan = frappe.db.sql('''select ap.name, ap.student_group, apc.assessment_criteria, apc.maximum_score as max_score - from `tabAssessment Plan` ap, `tabAssessment Plan Criteria` apc - where ap.assessment_group=%s and ap.course=%s and ap.name=apc.parent and ap.docstatus=1 - order by apc.assessment_criteria''', (assessment_group, course), as_dict=1) - print assessment_plan assessment_plan_list = set([d["name"] for d in assessment_plan]) - student_group_list = set([d["student_group"] for d in assessment_plan]) - assessment_criteria_list = set([(d["assessment_criteria"],d["max_score"]) for d in assessment_plan]) - assessment_plan if not assessment_plan_list: frappe.throw(_("No assessment plan linked with this assessment group")) - assessment_result = frappe.db.sql('''select ar.student, ard.assessment_criteria, ard.grade, ard.score + student_group_list = set([d["student_group"] for d in assessment_plan]) + assessment_result = frappe.db.sql('''select ar.student, ard.assessment_criteria, ard.grade, ard.score from `tabAssessment Result` ar, `tabAssessment Result Detail` ard where ar.assessment_plan in (%s) and ar.name=ard.parent and ar.docstatus=1 - order by ard.assessment_criteria''' %', '.join(['%s']*len(assessment_plan_list)), tuple(assessment_plan_list), as_dict=1) + order by ard.assessment_criteria''' %', '.join(['%s']*len(assessment_plan_list)), + tuple(assessment_plan_list), as_dict=1) - result_dict = defaultdict(list) + result_dict = defaultdict(dict) + kounter = defaultdict(dict) for result in assessment_result: - result_dict[result.student].append(result.grade) - result_dict[result.student].append(result.score) + result_dict[result.student].update({frappe.scrub(result.assessment_criteria): result.grade, + frappe.scrub(result.assessment_criteria)+"_score": result.score}) + if result.grade in kounter[result.assessment_criteria]: + kounter[result.assessment_criteria][result.grade] += 1 + else: + kounter[result.assessment_criteria].update({result.grade: 1}) - student_list = frappe.db.sql('''select sgs.group_roll_number, sgs.student, sgs.student_name + student_list = frappe.db.sql('''select sgs.student, sgs.student_name from `tabStudent Group` sg, `tabStudent Group Student` sgs where sg.name = sgs.parent and sg.name in (%s) - order by sgs.group_roll_number asc''' %', '.join(['%s']*len(student_group_list)), tuple(student_group_list), as_list=1) + order by sgs.group_roll_number asc''' %', '.join(['%s']*len(student_group_list)), + tuple(student_group_list), as_dict=1) - data = [] for student in student_list: - tmp_list = student + result_dict[student[1]] - data.append(tmp_list) + student.update(result_dict[student.student]) + data = student_list - return get_column(assessment_criteria_list), data + columns = get_column(list(set([(d["assessment_criteria"],d["max_score"]) for d in assessment_plan]))) + + grading_scale = frappe.db.get_value("Assessment Plan", list(assessment_plan_list)[0], "grading_scale") + grades = frappe.db.sql_list('''select grade_code from `tabGrading Scale Interval` where parent=%s''', + (grading_scale)) + assessment_criteria_list = list(set([d["assessment_criteria"] for d in assessment_plan])) + chart = get_chart_data(grades, assessment_criteria_list, kounter) + + return columns, data, None, chart def get_column(assessment_criteria): - columns = [ - _("Batch Roll No") + "::60", - _("Student ID") + ":Link/Student:90", - _("Student Name") + "::160", - ] + columns = [{ + "fieldname": "student", + "label": _("Student ID"), + "fieldtype": "Link", + "options": "Student", + "width": 90 + }, + { + "fieldname": "student_name", + "label": _("Student Name"), + "fieldtype": "Data", + "width": 160 + }] for d in assessment_criteria: - columns.append(d[0] + "::110") - columns.append("Score(" + str(int(d[1])) + ")::100") + columns.append({ + "fieldname": frappe.scrub(d[0]), + "label": d[0], + "fieldtype": "Data", + "width": 110 + }) + columns.append({ + "fieldname": frappe.scrub(d[0]) +"_score", + "label": "Score(" + str(int(d[1])) + ")", + "fieldtype": "Float", + "width": 100 + }) return columns + +def get_chart_data(grades, assessment_criteria_list, kounter): + grades = sorted(grades) + chart_data = [] + chart_data.append(["x"] + assessment_criteria_list) + for grade in grades: + tmp = [grade] + for ac in assessment_criteria_list: + if grade in kounter[ac]: + tmp.append(kounter[ac][grade]) + else: + tmp.append(0) + chart_data.append(tmp) + return { + "data": { + "x": "x", + "columns": chart_data + }, + "chart_type": 'bar', + }