Merge pull request #482 from scholarly-python-package/develop

Release v1.7.10

Author: arunkannawadi

scholarly/author_parser.py

@@ -123,8 +123,12 @@ def _fill_indices(self, soup, author):
     def _fill_counts(self, soup, author):
         years = [int(y.text)
                  for y in soup.find_all('span', class_='gsc_g_t')]
-        cites = [int(c.text)
-                 for c in soup.find_all('span', class_='gsc_g_al')]
+
+        cites = [0]*len(years)
+        for c in soup.find_all('a', class_='gsc_g_a'):
+            i = int(c['style'].split(':')[-1])
+            cites[-i] = int(c.find('span', class_='gsc_g_al').text)
+
         author['cites_per_year'] = dict(zip(years, cites))
 
     def _fill_public_access(self, soup, author):

setup.py

@@ -5,7 +5,7 @@
 
 setuptools.setup(
     name='scholarly',
-    version='1.7.10-alpha',
+    version='1.7.10',
     author='Steven A. Cholewiak, Panos Ipeirotis, Victor Silva, Arun Kannawadi',
     author_email='steven@cholewiak.com, panos@stern.nyu.edu, vsilva@ualberta.ca, arunkannawadi@astro.princeton.edu',
     description='Simple access to Google Scholar authors and citations',

test_module.py

@@ -560,6 +560,15 @@ def test_bin_citations_by_year(self):
             self.assertLessEqual(y_lo, y_hi)
             self.assertLessEqual(sum(cpy[y] for y in range(y_lo, y_hi+1)), 1000)
 
+    def test_cites_per_year(self):
+        """Test that the cites_per_year is correctly filled in,
+           including any gap years.
+        """
+        author = scholarly.search_author_id('DW_bVcEAAAAJ')
+        scholarly.fill(author, sections=['counts'])
+        cpy = {2014: 1, 2015: 2, 2016: 2, 2017: 0, 2018: 2, 2019: 1, 2020: 12, 2021: 21, 2022: 35}
+        for year, count in cpy.items():
+            self.assertEqual(author['cites_per_year'][year], count)
 
 class TestScholarlyWithProxy(unittest.TestCase):
     @classmethod