Tweaks to the README and commets.

Author: microbit-carlos

README.md

@@ -3,12 +3,13 @@
 [![MakeCode Project](https://github.com/microbit-foundation/pxt-microbit-ml-runner/actions/workflows/makecode.yml/badge.svg)](https://github.com/microbit-foundation/pxt-microbit-ml-runner/actions/workflows/makecode.yml)
 [![Header Generator Tests](https://github.com/microbit-foundation/pxt-microbit-ml-runner/actions/workflows/header-gen.yml/badge.svg)](https://github.com/microbit-foundation/pxt-microbit-ml-runner/actions/workflows/header-gen.yml)
 
-This project includes [ML4F](https://github.com/microsoft/ml4f) and a wrapper
-to invoke a known type of model. It is left slim to be able to be import it
-in other MakeCode extensions and a MicroPython module.
+This project wraps [ML4F](https://github.com/microsoft/ml4f) to invoke a known
+type of model that requires some data pre-processing.
+The wrapper is provided as a slim library to be able to be import it
+into other MakeCode extensions and as a MicroPython module.
 
 
-## How to use your ML4F model with this extension
+## How to use external ML4F model with this extension
 
 The ML4F wrapper library can be found in the `mlrunner` folder.
 This repository also includes a pre-compiled model (inclusion can be controlled
@@ -51,33 +52,34 @@ cd pxt-microbit-ml-runner
 npm install pxt --no-save
 npx pxt target microbit --no-save
 npx pxt install
-PXT_FORCE_LOCAL=1 PXT_NODOCKER=1 npx pxt
+PXT_FORCE_LOCAL=1 npx pxt
 ```
 
 For the V1 build Yotta can hit the GitHub rate limits quite easily if the
 project is built from a clean state more than once.
-A V2-only build can be performed with the `PXT_COMPILE_SWITCHES=csv---mbcodal`
+A V2-only build can be triggered with the `PXT_COMPILE_SWITCHES=csv---mbcodal`
 environmental variable.
 
 ```
 PXT_FORCE_LOCAL=1 PXT_NODOCKER=1 PXT_COMPILE_SWITCHES=csv---mbcodal npx pxt
 ```
 
 > [!CAUTION]
-> **When updating to this repository, do NOT push changes to the `enums.d.ts`
+> **When updating this repository, do NOT push changes to the `enums.d.ts`
 > or `shims.d.ts` files.**
 >
 > These are autogenerated by MakeCode to contain the enums and function shims
-> from the C++ code to be accessible by TypeScript. However, these are only
-> needed for the test code, and should **not** be shipped.
+> from the C++ code to be accessible via TypeScript. However, these are only
+> needed for the test code, and should **not** be shipped as it will affect
+> its usage as a MakeCode extension.
 > 
-> So, it's recommended to use: `git update-index --skip-worktree <file>`
+> It's recommended to run locally: `git update-index --skip-worktree <file>`
 >
 > Unfortunately, adding `enums.d.ts` and `shims.d.ts` to the `testFiles` entry
 > in `pxt.json` does not work, and they need to be added to `files` (so they
-> are included with the extension files, not just the test files), and so they
-> should be kept empty. Building the project locally includes the test files
-> and will add code in there that should never be pushed.
+> end up included with the extension) and so, they should be kept empty.
+> Building the project locally compiles all the test files, will add code
+> to these `.d.ts` files, which should not be pushed.
 
 
 ## Build flags

mlrunner/mlrunner.c

@@ -255,7 +255,7 @@ int ml_calcPrediction(const ml_actions_t *actions, const float* predictions, con
     if (max_index < 0) {
         return -1;
     }
-    // If the max predictionn is 0, then none were above the threshold
+    // If the max prediction is 0, then none were above the threshold
     if (predictions_above_threshold[max_index] == 0.0f) {
         max_index = -1;
     }

testextension.cpp

@@ -79,9 +79,9 @@ namespace testrunner {
 
         unsigned int time_start = system_timer_current_time_us();
 
-        int32_t ticks_start = ticks_cpu() & 0x7FFFFFFF;
+        uint32_t ticks_start = ticks_cpu();
         float *modelData = mlDataProcessor.getProcessedData();
-        int32_t ticks_end = ticks_cpu() & 0x7FFFFFFF;
+        uint32_t ticks_end = ticks_cpu();
         if (modelData == NULL) {
             DEBUG_PRINT("Failed to processed data for the model\n");
             uBit.panic(TEST_RUNNER_ERROR + 21);