Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[STRAT-109] - Implement Correction Function For SOC #131

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

[STRAT-109] - Implement Correction Function For SOC #131

wants to merge 22 commits into from

Conversation

vik01
Copy link
Contributor

@vik01 vik01 commented Jun 12, 2021 •
edited

  • Created SOC correction function which uses current SOC and OCV values to correct the SOC given.
  • Added tests scripts and test files with SOC and OCV values.

@vik01
Merge pull request #1 from uw-midsun/master
2cf7164 
Updating Vik01 fork repo
@vik01
Merge pull request #2 from uw-midsun/master
44d0fee 
Pulling from main repo
@vik01
Merge pull request #3 from uw-midsun/master
9303219 
Pulling from head repo to fork repo
@vik01
Possible changes to the SOC program
7e9c84f
@vik01
First Set of Changes to the values
740a517
@vik01
Added a linear interpolation method - in Progress
5ac0d36
@vik01
STRAT108: Changed linear approximation algorithm and added sorting al…
9fe3014 
…gorithm to better sort soc and vocv array
@vik01
Zipped soc and vocv values for better sorting. Changed to python buil…
6fed0b9 
…t sorting.
@vik01
Changed up get_cell_ocv program to include binary search and made sli…
0394bb0 
…ght adjustmenst to binary search for guessing between cases
@vik01
deleted code snippet that called the deleted linear searching get_cel…
37d593d 
…l_ocv function
@vik01
Merge pull request #4 from uw-midsun/master
58584a4 
Merge from master in main repo to strat108 in fork
@vik01
Added Commenting to SoC_OCV.py. Added if __name__ trigger spacing.
ebc1bc2
@vik01
merged BatTestPlot.py and BatTestPlotLinear.py. Deleted BatTestPlotLi…
f16829e 
…near.py
@vik01
[STRAT-109] reformatting file structure
b832d89
@vik01
[STRAT-109] Added correction function for SOC. Added additional data …
26a4a70 
…for testing purposes.
@vik01
Merge branch 'master' into STRAT109
d837ce9
e-wai
e-wai requested changes Jun 26, 2021
View reviewed changes
Copy link
Collaborator

@e-wai e-wai left a comment •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still a bit unclear - how does this SoC_OCV curve interact with the CoulombCounter class during the race?

Also, 22 000 lines woah 👀

soc/SoC_OCV.py Show resolved Hide resolved
soc/SoC_OCV.py Outdated Show resolved Hide resolved
soc/SoC_OCV.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@e-wai e-wai e-wai requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@vik01 @e-wai @rac99