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

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

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.4, 3.7
    • 3.7
    • Analytics
    • MOODLE_34_STABLE, MOODLE_37_STABLE
    • MOODLE_37_STABLE
    • MDL-60936_master
    • 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

    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

              dmonllao David Monllaó
              dmonllao David Monllaó
              Carlos Escobedo Carlos Escobedo
              Adrian Greeve Adrian Greeve
              Janelle Barcega Janelle Barcega
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                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