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

Perf: Instructor Dashboard performance regressions #534

Closed
bmtcril opened this issue Dec 4, 2023 · 2 comments
Closed

Perf: Instructor Dashboard performance regressions #534

bmtcril opened this issue Dec 4, 2023 · 2 comments

Comments

@bmtcril
Copy link
Contributor

bmtcril commented Dec 4, 2023

In large scale load testing, the following Instructor Dashboard reports have poor performance, or are timing out:

  • Currently enrolled learners per day
    • This only seems to be slow when no course filter is applied, we could possibly force a course filter here)
  • Transcripts / Captions per video
    • This is pulling directly from xapi_events_all_parsed and should probably have an intermediate model of just these events with associated "enabled" / "disabled" values parsed out
@bmtcril bmtcril changed the title Instructor Dashboard performance regressions Perf: Instructor Dashboard performance regressions Jan 12, 2024
@SoryRawyer
Copy link
Contributor

The transcripts chart should be backed by a smaller MV instead of the larger xapi_events_all_parsed table, so performance should be improved.
I haven't been able to get much more performance out of the enrolled learners per day dataset, so I think requiring a course filter would be a reasonable change to enhance performance.

@bmtcril
Copy link
Contributor Author

bmtcril commented Jan 12, 2024

I think choosing a course is going to be required for v1 anyway 👍 Calling this one done, we'll do another round of testing once we have a v1 RC.

@bmtcril bmtcril closed this as completed Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants