Pradyot Ranjan
Pradyot Ranjan
> Thanks for taking this on. I don't think it is necessary to have both `npts` and `non_linear_time`. Instead, I think we should deprecate `npts`, require users to pass `dt`...
Should I change name of variable `non_linear_time` to something more accurate or is it ok? Also how to handle logic when function has both `dt` and `non_linear_time` as input? Should...
Now the output of ```py while time < end_time: step_solution = step_solver.step(step_solution, model, dt=500, non_linear_time=[0, 50, 100, 200, 500]) time += dt ``` Looks like:
Should we also remove `npts` from [here](https://github.com/pybamm-team/PyBaMM/blob/ee64eafc7f945fb6684ce22a60169f97f5a0f235/pybamm/simulation.py#L965) ? @rtimms @tinosulzer
I've also added tests.
I've made the changes! Please have a look.
@agriyakhetarpal why is [this](https://github.com/pybamm-team/PyBaMM/actions/runs/8003981310/job/21860421881?pr=3627) CI job failing?
Can you suggest some useful resources that might help. Also if no one is already working on this I would love to !
Thanks a lot for the detailed info! This would really be helpful
Can you please explain why do we use citations and how does it help? For example :[here](https://github.com/pybamm-team/PyBaMM/blob/cf2203c618d6f86209f34d63a0efe10a540545fa/pybamm/models/submodels/particle_mechanics/swelling_only.py#L30). Also how does it register a citation above?