diff --git a/wandb/summary.py b/wandb/summary.py
index <HASH>..<HASH> 100644
--- a/wandb/summary.py
+++ b/wandb/summary.py
@@ -44,10 +44,9 @@ class SummarySubDict(object):
     def __setattr__(self, k, v):
         k = k.strip()
         if k.startswith("_"):
-            return object.__setattr__(self, k, v)
+            object.__setattr__(self, k, v)
         else:
             self[k] = v
-            return v
 
     def __getattr__(self, k):
         k = k.strip()
@@ -95,6 +94,7 @@ class SummarySubDict(object):
         return self._dict.get(k, default)
 
     def __getitem__(self, k):
+        k = k.strip()
         self.get(k)  # load the value into _dict if it should be there
         return self._dict[k]
Summaries: address Jeff's comments. Add a missing key.strip() and get rid of the unnecessary return in __setitem__.