Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-60936

Convert enabled time-splitting methods setting to evaluation default time-splitting methods

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.4, 3.7
    • Fix Version/s: 3.7
    • Component/s: Analytics
    • Labels:
    • Testing Instructions:
      Hide

      Prerequisites

      1. Install https://github.com/dmonllao/moodle-local_testanalytics

      from the command line:

      change directory to 'local' and type in the following:

      git clone https://github.com/dmonllao/moodle-local_testanalytics.git testanalytics
      This will add the code into the testanalytics folder under local. Next login to your moodle site and follow the installation instructions.

      Test 1

      1. Log in as admin
      2. Go to "site admin > analytics > analytics settings"
      3. Check that there is a 'defaulttimesplittingsevaluation' setting
      4. Remove "Tenths accumulative" from the list of selected time-splitting methods if it is selected
      5. Go to "site admin > analytics > analytics models"
      6. Select "Actions > Edit" for "Students at risk of dropping out" model
      7. You should see "Tenths accumulative" listed as one of the "Time-splitting method" options you can select
      8. Select that option and save changes
      9. Confirm that the option was saved by clicking again on "Actions > Edit" for that same model

      Test 2

      1. Log in as admin
      2. Go to "site admin > analytics > analytics settings"
      3. Disable 'onlycli' option
      4. Set 'defaulttimesplittingsevaluation' setting to "Tenths" and "Tenths accumulative"
      5. Save changes
      6. Go to "site admin > analytics > analytics models" and click 'Evaluate' for "Discrete" model
      7. You should see two headings starting with "Results using": Results using Tenths time-splitting method and Results using Tenths accumulative time-splitting method
      Show
      Prerequisites Install https://github.com/dmonllao/moodle-local_testanalytics from the command line: change directory to 'local' and type in the following: git clone https://github.com/dmonllao/moodle-local_testanalytics.git testanalytics This will add the code into the testanalytics folder under local. Next login to your moodle site and follow the installation instructions. Test 1 Log in as admin Go to "site admin > analytics > analytics settings" Check that there is a 'defaulttimesplittingsevaluation' setting Remove "Tenths accumulative" from the list of selected time-splitting methods if it is selected Go to "site admin > analytics > analytics models" Select "Actions > Edit" for "Students at risk of dropping out" model You should see "Tenths accumulative" listed as one of the "Time-splitting method" options you can select Select that option and save changes Confirm that the option was saved by clicking again on "Actions > Edit" for that same model Test 2 Log in as admin Go to "site admin > analytics > analytics settings" Disable 'onlycli' option Set 'defaulttimesplittingsevaluation' setting to "Tenths" and "Tenths accumulative" Save changes Go to "site admin > analytics > analytics models" and click 'Evaluate' for "Discrete" model You should see two headings starting with "Results using": Results using Tenths time-splitting method and Results using Tenths accumulative time-splitting method
    • Affected Branches:
      MOODLE_34_STABLE, MOODLE_37_STABLE
    • Fixed Branches:
      MOODLE_37_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-60936_master

      Description

      There is a problem with 'Enabled time-splitting methods' setting.

      Any 3rd party can add all sort of new analytics components: analysers, time splitting methods... people can potentially add a time splitting method and create a model using it without the time spolitting method being part of the enabled time-splitting methods in the site. They need to enable it separately, this is far from nice.

      The purpose of that setting was to limit the amount of time splitting methods a model is evaluated against. Instead of enforcing time-splitting methods to be enabled (if it is not already enforced everywhere) I would vote to update the time-splitting method setting to something like "List of time-splitting methods that will be use to evaluate models".

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  20/May/19

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 hour, 30 minutes
                  1h 30m