Skip to content

Commit

Permalink
Update get_ionic_radii test
Browse files Browse the repository at this point in the history
  • Loading branch information
morganjwilliams committed Nov 1, 2024
1 parent a7be1ff commit c5f7611
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pyrolite/geochem/ind.py
Original file line number Diff line number Diff line change
Expand Up @@ -429,7 +429,7 @@ def get_ionic_radii(
Returns
--------
:class:`pandas.Series` | :class:`float`
:class:`pandas.Series` | :class:`numpy.ndarray` | :class:`float`
Series with viable ion charge and coordination, with associated radii in
angstroms. If the ion charge and coordiation are completely specified and
found in the table, a single value will be returned instead.
Expand Down
5 changes: 3 additions & 2 deletions test/geochem/geochem_ind.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import unittest

import numpy as np
import periodictable as pt

from pyrolite.geochem.ind import (
Expand Down Expand Up @@ -258,11 +259,11 @@ def test_ree_radii(self):

def test_ree_radii_list(self):
radii = get_ionic_radii(self.ree, charge=3, coordination=8)
self.assertTrue(isinstance(radii, list))
self.assertTrue(isinstance(radii, np.ndarray))

def test_ree_radii_list_whittaker_muntus(self):
radii = get_ionic_radii(self.ree, charge=3, coordination=8, source="whittaker")
self.assertTrue(isinstance(radii, list))
self.assertTrue(isinstance(radii, np.ndarray))


class TestByIncompatibility(unittest.TestCase):
Expand Down

0 comments on commit c5f7611

Please sign in to comment.