O, and I see one of the CI tests fails because there are no embedded docs. Can you add a file solveODE.js there and list it in src/expression/embeddedDocs.js? Thanks!

Hi Jos, thank you for your through review.

Everything is very clear, I will work on that, I just included a few comments regarding error messages.

Thank you very much for the review. Here is an overview of the changes

• It has embedded docs
• Included the returned object in the jsdocs
• It works with matrices and returns matrices
• Throws some more specific errors regarding
  • max iter reached
  • tspan has no numeric or unit values
  • method name
• Simplifies most tests using functions previously implemented
• Included a more challenging function to test f(t, y) = y - t instead of f(t, y) = y

Here is the bad part:

• It doesn't work with big numbers yet

Now it works with big numbers

These are the recent changes:

• The logic was simplified a bit by forcing:
  • maxStep and firstStep to be positive
  • minStep to be positive or zero
• Included tests for time related variables like tspan[0], tspan[1], minStep, maxStep and firstStep
• I encountered an error where I thought isNumeric included only number and bigNumber. So I used something else
• Fixed a typo with deltaB

Some drawbacks is that now instead of just solving some cases it throws an error, bit it might be ok.

Thanks for the updates David. All looks good to go 👌

That's great news! Thanks!

Published now in v11.9.0.