From dff8242572531764017fb39867a1abf05325a727 Mon Sep 17 00:00:00 2001
From: "Ryan S. DeFever" <r.s.defever@gmail.com>
Date: Fri, 19 Mar 2021 18:44:29 -0400
Subject: [PATCH 1/2] Fix missing f-strings and add more descriptive error
 message

---
 ele/element.py | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/ele/element.py b/ele/element.py
index 7784215..e8e86f7 100644
--- a/ele/element.py
+++ b/ele/element.py
@@ -75,7 +75,9 @@ def element_from_symbol(symbol):
         If no match is found
     """
     if not isinstance(symbol, str):
-        raise TypeError("`symbol` ({symbol}) must be a string")
+        raise TypeError(
+            f"`string` ({symbol}) must be a string.  Provided {type(symbol).__name__}."
+        )
 
     symbol = symbol.capitalize()
     matched_element = symbol_dict.get(symbol)
@@ -106,7 +108,9 @@ def element_from_name(name):
         If no match is found
     """
     if not isinstance(name, str):
-        raise TypeError("`name` ({name}) must be a string")
+        raise TypeError(
+            f"`string` ({name}) must be a string.  Provided {type(name).__name__}."
+        )
 
     name = name.lower()
     matched_element = name_dict.get(name)
@@ -137,7 +141,9 @@ def element_from_atomic_number(atomic_number):
         If no match is found
     """
     if not isinstance(atomic_number, int):
-        raise TypeError("`atomic_number` ({atomic_number}) must be an int")
+        raise TypeError(
+            f"`string` ({atomic_number}) must be an integer.  Provided {type(atomic_number).__name__}."
+        )
 
     matched_element = atomic_dict.get(atomic_number)
     if matched_element is None:
@@ -172,8 +178,10 @@ def element_from_mass(mass, exact=True, duplicates="error"):
     matched_element : element.Element or tuple of element.Element
         The matching element(s) from the periodic table
     """
-    if not isinstance(mass, (float, int)):
-        raise TypeError("`mass` ({mass}) must be a float")
+    if not isinstance(mass, (int, float)):
+        raise TypeError(
+            f"`string` ({mass}) must be a float.  Provided {type(mass).__name__}."
+        )
 
     if duplicates.lower() not in ["error", "all", "none"]:
         raise TypeError(
@@ -233,7 +241,7 @@ def infer_element_from_string(string):
     """
     if not isinstance(string, str):
         raise TypeError(
-            f"`string` ({string}) must be a string.  Provided {type(string).__name__}"
+            f"`string` ({string}) must be a string.  Provided {type(string).__name__}."
         )
 
     try:

From a38dfeae8689674a79f76cc42f53dc60f468aaeb Mon Sep 17 00:00:00 2001
From: "Ryan S. DeFever" <r.s.defever@gmail.com>
Date: Thu, 1 Apr 2021 08:33:41 -0400
Subject: [PATCH 2/2] Switch error description from "float" to "number"

---
 ele/element.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ele/element.py b/ele/element.py
index e8e86f7..c88a32f 100644
--- a/ele/element.py
+++ b/ele/element.py
@@ -180,7 +180,7 @@ def element_from_mass(mass, exact=True, duplicates="error"):
     """
     if not isinstance(mass, (int, float)):
         raise TypeError(
-            f"`string` ({mass}) must be a float.  Provided {type(mass).__name__}."
+            f"`string` ({mass}) must be a number.  Provided {type(mass).__name__}."
         )
 
     if duplicates.lower() not in ["error", "all", "none"]: