-
Improvement
-
Resolution: Fixed
-
Major
-
3.4
-
MOODLE_34_STABLE
-
MOODLE_34_STABLE
-
MDL-59106_master -
Each prediction should contain information about when the prediction was done and also the time range that was considered to perform that prediction.
Showing the prediction time should be easy, (we have a analytics_predictions timecreated column) the not that straight forward part is to show the time ranges that were considered to make this prediction. There is a rangeindex field that (in theory) could be used to extract the period of time of the logs we looked at to make that prediction but we should avoid it because analysables (let's say course, it is easy to imagine it in our mind) start and end times (course start date and course end date) can change along time (courses can be reused...) so we need to add 2 new fields to analytics_predictions table, start time and end time, so they are set when we get the prediction. At the moment we are using rangeindex to select the most recent prediction for each element (e.g. if we get predictions for a student after 3 months and another one after 6 months the 6 months one is displayed) we could also do it with analytics_predictions timecreated field though. I see positive and negative aspects for both removing rangeindex (there is just 1 prediction for each sampleid and rangeindex and is cleaner to sort by a incremental number than by a timestamp) and keeping it (it seems kind of redundant). I vote for keeping rangeindex as we don't need to keep these records synced and it is just a few digits int on the other hand joining with timestamps is not a good idea in general (see core_analytics\model::get_predictions)
Feel free to contact UX team about where to fit this information, for what I've talked with Barbara what seems to make more sense is a new column in the insights list table.
- is blocked by
-
MDL-59153 Review tool_analytics and report_insights UI / UX
- Closed