-
-
Notifications
You must be signed in to change notification settings - Fork 572
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename lead-acid and lithium-ion #1464
Rename lead-acid and lithium-ion #1464
Conversation
8949c55
to
fa73210
Compare
…in parameters See this slack discussion for more details https://pybamm.slack.com/archives/C010LMK2ZB9/p1619188113091100
fa73210
to
d413205
Compare
I guess I have renamed everything, @tinosulzer @rtimms could you please review it, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @priyanshuone6
@@ -104,6 +104,9 @@ KLU_module_deps | |||
# setup | |||
setup.log | |||
|
|||
# test | |||
test.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why was this added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tinosulzer This file is generated every time someone runs the run-tests.py
, so I thought to put it in the gitignore as it is only used locally and should not be pushed. Should I undo this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, no problem that's fine
Description
Renamed lead-acid to lead_acid and lithium-ion to lithium_ion as discussed over slack.
Add warnings and tests
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.
Key checklist:
$ flake8
$ python run-tests.py --unit
$ cd docs
and then$ make clean; make html
You can run all three at once, using
$ python run-tests.py --quick
.Further checks: