Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CourseUnit summary sum row fakd #1323

Closed
HRemonen opened this issue Jun 18, 2024 · 2 comments
Closed

CourseUnit summary sum row fakd #1323

HRemonen opened this issue Jun 18, 2024 · 2 comments
Labels
bug norppa is broken summary Related to course summary TAU working on

Comments

@HRemonen
Copy link
Contributor

HRemonen commented Jun 18, 2024

The CourseUnit summary sum row is fakd when the "näytä kaikki" is selected from the filters.

The summary average row should calculate the sum of the visible course realisations feedbacks, now it calculates something very different.

The problem

Take for example this CourseUnit:

Image

Why does the average row display that the feedback count is 20 and studentcount is 88 even though there is only one course realisation with the numbers 10/44 correspondingly.

This also gets more interesting when we take some Course with multiple Course realisations:

Image

Where these numbers 496/1598 even comes from? If you sum up the course realisation numbers then 550/964 which means 57% percent of feedbacks given.

This is not the case always

If we filter the summary by "Lukuvuosi" then this is not a issue:

Image

Until there eventually is multiple course realisations:

Image

Like WTF, now the sum row displays the same information as one of the course realisations summary????

@HRemonen HRemonen converted this from a draft issue Jun 18, 2024
@HRemonen HRemonen changed the title CourseUnit summary average row fakd CourseUnit summary sum row fakd Jun 18, 2024
@HRemonen HRemonen added bug norppa is broken summary Related to course summary labels Jun 18, 2024
@HRemonen
Copy link
Contributor Author

@HRemonen
Copy link
Contributor Author

HRemonen commented Oct 1, 2024

The TAU changes came and this can't be right. Currently for https://norppa.helsinki.fi/course-summary/TKT20005?startDate=2022-08-01&endDate=2023-07-31 the course summary page is the following:

Image

AFTER THE CHANGES the same page looks like this:
Image

Where did the CUR rows disappear?


Also displaying all of the CUR's for a CU does not work anymore. Before:
Image

And after the changes:
Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug norppa is broken summary Related to course summary TAU working on
Projects
Archived in project
Development

No branches or pull requests

1 participant