newsworthycharts 1.63.2__py3-none-any.whl → 1.63.4__py3-none-any.whl

newsworthycharts/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "1.63.2"
+__version__ = "1.63.4"
 
 from .chart import Chart
 from .choroplethmap import ChoroplethMap

newsworthycharts/chart.py

@@ -486,35 +486,49 @@ class Chart(object):
         if not self.show_ticks:
             self.category_axis.set_visible(False)
         else:
-            # Increase number of decimals until we have no duplicated y axis ticks,
-            # unless .decimals explicitly set.
-            self._fig.canvas.draw()
-            tl = [x.get_text() for x in self.value_axis.get_ticklabels()]
-            tl_ = [x for (i, x) in enumerate(tl) if tl[i - 1] != x]
             autodetect_decimals = self.decimals is None
-            if len(tl_) < len(tl) and autodetect_decimals:
-                try_dec = 1
-                keep_trying = True
-                while keep_trying:
-                    self.decimals = try_dec
-                    if len(self.data):
-                        self._add_data()
+            if autodetect_decimals:
+                # Are all numbers integers?
+
+                def flatten(list_of_lists):
+                    return [item for sublist in list_of_lists for item in sublist]
+
+                all_integers = False
+                try:
+                    all_integers = all([x.is_integer() for x in flatten(self.data.values)])
+                except AttributeError:
+                    # values is undefined for catCharts
+                    pass
+                if all_integers:
+                    self.decimals = 0
+                else:
+                    # Increase number of decimals until we have no duplicated y axis ticks,
+                    # unless .decimals explicitly set.
                     self._fig.canvas.draw()
                     tl = [x.get_text() for x in self.value_axis.get_ticklabels()]
                     tl_ = [x for (i, x) in enumerate(tl) if tl[i - 1] != x]
-                    if len(tl) == len(tl_):
-                        keep_trying = False
-                    if try_dec > 2:
-                        keep_trying = False
-                    try_dec += 1
-            # If we still have duplicates; remove them!
-            # If we still have duplicates; remove them!
+                    if len(tl_) < len(tl):
+                        try_dec = 1
+                        keep_trying = True
+                        while keep_trying:
+                            self.decimals = try_dec
+                            if len(self.data):
+                                self._add_data()
+                            self._fig.canvas.draw()
+                            tl = [x.get_text() for x in self.value_axis.get_ticklabels()]
+                            tl_ = [x for (i, x) in enumerate(tl) if tl[i - 1] != x]
+                            if len(tl) == len(tl_):
+                                keep_trying = False
+                            if try_dec > 2:
+                                keep_trying = False
+                            try_dec += 1
+            # Remove duplicate y ticks (most should be gone as per above)
             ticks = self.value_axis.get_ticklabels()
             ticks_ = [x for (i, x) in enumerate(ticks) if ticks[i - 1].get_text() != x.get_text()]
             if len(ticks_) < len(ticks):
                 y = [float(x._y) for x in ticks_]
                 if self.decimals == 0:
-                    # Recrate with integeres
+                    # Recrate with integres
                     import math
                     _bottom = min(y)
                     _top = math.ceil(max(y))
@@ -522,6 +536,11 @@ class Chart(object):
                     y = range(_bottom, _top + 1)
                     y = [float(x) for x in y]
                 self.value_axis.set_ticks(y)
+            if self.decimals == 0:
+                # remove non integer ticks
+                y = [float(x._y) for x in ticks]
+                y = [x for x in y if x.is_integer()]
+                self.value_axis.set_ticks(y)
 
         for a in self.annotations:
             self._annotate_point(a["text"], a["xy"], a["direction"])

{newsworthycharts-1.63.2.dist-info → newsworthycharts-1.63.4.dist-info}/METADATA

@@ -1,9 +1,9 @@
 Metadata-Version: 2.1
 Name: newsworthycharts
-Version: 1.63.2
+Version: 1.63.4
 Summary: Matplotlib wrapper to create charts and publish them on Amazon S3
 Home-page: https://github.com/jplusplus/newsworthycharts
-Download-URL: https://github.com/jplusplus/newsworthycharts/archive/1.63.2.tar.gz
+Download-URL: https://github.com/jplusplus/newsworthycharts/archive/1.63.4.tar.gz
 Author: Jens Finnäs and Leo Wallentin, J++ Stockholm
 Author-email: stockholm@jplusplus.org
 License: MIT
@@ -252,6 +252,15 @@ Roadmap
 Changelog
 ---------
 
+- 1.63.4
+
+  - Fixed edge case in integer check
+
+- 1.63.3
+
+  - Fixed bug in decimal auto detection
+  - Remove non integer value ticks when decimals are set to 0
+
 - 1.63.2
 
   - Fixed bug in interval detection in SerialChart, affecting yearly data

{newsworthycharts-1.63.2.dist-info → newsworthycharts-1.63.4.dist-info}/RECORD

@@ -1,7 +1,7 @@
-newsworthycharts/__init__.py,sha256=M7ITQUn4R41FT6KyEqJTzJZswkTBLemG-BC4pKqmYSw,1160
+newsworthycharts/__init__.py,sha256=Y4F1H2k7Q3k6WNKCJmjcTRDKa8hK6_hUhba1GnqZJuI,1160
 newsworthycharts/bubblemap.py,sha256=nkocWmpiFgfjEuJGAsthjY5X7Q56jXWsZHUGXw4PwgE,2587
 newsworthycharts/categoricalchart.py,sha256=LwOZ3VbNy9vzvoK0s77AkbfMt4CXVDSAhnsnBInUIrE,14764
-newsworthycharts/chart.py,sha256=e118gGQbFpRJZl3SWLhVCBatclG80eFdojrxJn8td5g,31664
+newsworthycharts/chart.py,sha256=dUh32lZ0OZhcKCf61BvqFcRAHAV7Wc9ob8Lt1jXSuhE,32558
 newsworthycharts/choroplethmap.py,sha256=bCLf4kcchp1C2djg5AxcOM8BdbaMj0xg7UHrZsDafhI,8013
 newsworthycharts/datawrapper.py,sha256=RRkAVTpfP4updKxUIBaSmKuBi2RUVPaBRF8HDQhlGGA,11250
 newsworthycharts/map.py,sha256=c409jEO4L8Yr780sJRC0RchR44roAlgOUDAkuk1SfRg,6057
@@ -28,8 +28,8 @@ newsworthycharts/rc/newsworthy,sha256=vbYTav1w3DmB5tpaaV7Xl5q7TQJsc9eou6pLCoWc2v
 newsworthycharts/translations/datawrapper_regions.csv,sha256=fzZcQRX6RFMlNNP8mpgfYNdR3Y0QAlQxDXk8FXTaWWI,9214
 newsworthycharts/translations/regions.py,sha256=Nv1McQjggD4S3JRu82rDMTG3pqUVR13E5-FBpSYbm98,239
 newsworthycharts/translations/se_municipalities.csv,sha256=br_mm-IvzQtj_W55_ATREhJ97jWnCweBFlDAVY2EBxA,7098
-newsworthycharts-1.63.2.dist-info/LICENSE.txt,sha256=Sq6kGICrehbhC_FolNdXf0djKjTpv3YqjFCIYsxdQN4,1069
-newsworthycharts-1.63.2.dist-info/METADATA,sha256=Wby-h-OOlS5HDshkU7nYzByEvZT8RYNlvE4w6cTfjjE,29411
-newsworthycharts-1.63.2.dist-info/WHEEL,sha256=cVxcB9AmuTcXqmwrtPhNK88dr7IR_b6qagTj0UvIEbY,91
-newsworthycharts-1.63.2.dist-info/top_level.txt,sha256=dn_kzIj8UgUCMsh1PHdVEQJHVGSsN7Z8YJF-8xXa8n0,17
-newsworthycharts-1.63.2.dist-info/RECORD,,
+newsworthycharts-1.63.4.dist-info/LICENSE.txt,sha256=Sq6kGICrehbhC_FolNdXf0djKjTpv3YqjFCIYsxdQN4,1069
+newsworthycharts-1.63.4.dist-info/METADATA,sha256=kxlCxLX3F38SdbYVz7LDAli8uQNQRVMjz4Qti0GM-yU,29572
+newsworthycharts-1.63.4.dist-info/WHEEL,sha256=cVxcB9AmuTcXqmwrtPhNK88dr7IR_b6qagTj0UvIEbY,91
+newsworthycharts-1.63.4.dist-info/top_level.txt,sha256=dn_kzIj8UgUCMsh1PHdVEQJHVGSsN7Z8YJF-8xXa8n0,17
+newsworthycharts-1.63.4.dist-info/RECORD,,