Skip to content

Commit

Permalink
Merge pull request #148 from KingsburyLab/bugfix
Browse files Browse the repository at this point in the history
DB: add missing units to radius_ionic for Cr+3 and Ni+2
  • Loading branch information
rkingsbury authored Jul 25, 2024
2 parents dc437ae + bb67628 commit 6eabdc4
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## Unreleased

### Fixed

- Database: `size.radius_ionic` was missing units for `Ni[+2]` and `Cr[+3]`. Correct units have been added.

## [1.0.3] - 2024-07-20

### Fixed
Expand Down
4 changes: 2 additions & 2 deletions src/pyEQL/database/pyeql_db.json
Original file line number Diff line number Diff line change
Expand Up @@ -6698,7 +6698,7 @@
"n_elements": 1,
"size": {
"radius_ionic": {
"value": "0.755",
"value": "0.755",
"reference": "pymatgen",
"data_type": "experimental"
},
Expand Down Expand Up @@ -24663,7 +24663,7 @@
"n_elements": 1,
"size": {
"radius_ionic": {
"value": "0.83",
"value": "0.83",
"reference": "pymatgen",
"data_type": "experimental"
},
Expand Down

0 comments on commit 6eabdc4

Please sign in to comment.