Remove brew-sundials from the recommended developer installations

[jsbrittain]

Description

The (Mac recommended) brew install procedure for sundials does not link the cmake args that are required by tox -e dev. Here, we simplify the developer installation procedure for Mac, bringing it in-line with that for linux, by using tox -e pybamm-requires to download, compile and install the SuiteSparse and SUNDIALS libraries.

Fixes #2897

Type of change

Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

• New feature (non-breaking change which adds functionality)
• Optimization (back-end change that speeds up the code)
• Bug fix (non-breaking change which fixes an issue)

Key checklist:

(documentation change only)
[ ] No style issues: $ pre-commit run (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[ ] All tests pass: $ python run-tests.py --all

• The documentation builds: $ python run-tests.py --doctest

Further checks:

• Code is commented, particularly in hard-to-understand areas
• Tests added that prove fix is effective or that feature works

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (4f0a23d) 99.68% compared to head (4e64f21) 99.68%.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2925   +/-   ##
========================================
  Coverage    99.68%   99.68%           
========================================
  Files          273      273           
  Lines        18997    18997           
========================================
  Hits         18938    18938           
  Misses          59       59           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[agriyakhetarpal]

For Linux installation, and in addition to tox -e pybamm-requires – maybe the pybamm_install_odes command can be updated to also install OpenBLAS and not just SUNDIALS in a PR in the future?

This can remove the need for the note about the error A library with BLAS API not found. Please specify library location. -- LAPACK requires BLAS which is mentioned in the Troubleshooting section of the GNU/Linux installation section documentation at GNU-linux.rst

[brosaplanella]

Looks good to me, but I don't use Mac, so I have requested a review from Valentin who uses it

[valentinsulzer]

Looks good except changelog needs updating

[valentinsulzer]

this needs to be moved to the new section