Issue 1322 thermal parameters dependent temperature

[brosaplanella]

Description

The thermal parameters (density, heat capacity and conductivity) can now be functions of temperature.

Fixes #1322

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)

Key checklist:

• No style issues: $ flake8
• All tests pass: $ python run-tests.py --unit
• The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.

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

Merging #1323 (7567868) into develop (85b3a20) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #1323   +/-   ##
========================================
  Coverage    98.12%   98.12%           
========================================
  Files          272      272           
  Lines        15380    15431   +51     
========================================
+ Hits         15091    15142   +51     
  Misses         289      289           

Impacted Files	Coverage Δ
pybamm/models/submodels/thermal/lumped.py	100.00% <ø> (ø)
...mal/pouch_cell/pouch_cell_1D_current_collectors.py	100.00% <ø> (ø)
...mal/pouch_cell/pouch_cell_2D_current_collectors.py	100.00% <ø> (ø)
pybamm/models/submodels/thermal/x_full.py	100.00% <100.00%> (ø)
pybamm/parameters/lead_acid_parameters.py	100.00% <100.00%> (ø)
pybamm/parameters/lithium_ion_parameters.py	100.00% <100.00%> (ø)
pybamm/parameters/thermal_parameters.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 85b3a20...7567868. Read the comment docs.

[valentinsulzer]

Thanks @brosaplanella , just a couple of things to change / clarify

[valentinsulzer]

Each dimensionless function should calculate T_dim from T and then pass this to the dimensional function

[valentinsulzer]

need to make T dimensional here and pass T_dim to the functions

[valentinsulzer]

Thanks, all good now