docs: Update changelog

Author: jpmckinney

AUTHORS.rst

@@ -47,4 +47,4 @@ agate is made by a community. The following individuals have contributed code, d
 * `Tim Gates <https://github.com/timgates42>`_
 * `castorf <https://github.com/castorf>`_
 * `Julien Enselme <https://github.com/Jenselme>`__
-
+* `Scott Gigante <https://github.com/scottgigante>`__

CHANGELOG.rst

@@ -1,6 +1,7 @@
 1.10.0 - April 27. 2024
 -----------------------
 
+-  feat: :meth:`.Table.from_csv` reads the file line by line. If ``sniff_limit=None``, it reads the file into memory once, instead of twice. If ``column_types`` is a :class:`.TypeTester`, it reads the file into memory. (#778)
 -  fix: Fix :meth:`.TableSet.print_structure` for nested tablesets. (#765)
 
    .. code-block:: python

agate/table/__init__.py

@@ -82,7 +82,7 @@ def __init__(self, rows, column_names=None, column_types=None, row_names=None, _
             try:
                 first_row = rows[0]
             except TypeError:
-                # it's an iterator
+                # rows is an iterator.
                 first_row = next(rows)
                 rows = chain([first_row], rows)
             self._column_names = tuple(utils.letter_name(i) for i in range(len(first_row)))
@@ -108,8 +108,7 @@ def __init__(self, rows, column_names=None, column_types=None, row_names=None, _
                     raise ValueError('Column types must be instances of DataType.')
 
         if isinstance(column_types, TypeTester):
-            # if rows is streaming, we must bring it in-memory
-            # note: we could do better here.
+            # In case rows is an iterator, in which case need to read it all into memory.
             rows = tuple(rows)
             self._column_types = column_types.run(rows, self._column_names)
         else:

agate/table/from_csv.py

@@ -62,12 +62,12 @@ def from_csv(cls, path, column_names=None, column_types=None, row_names=None, sk
         handle = f
 
         if sniff_limit is None:
-            # avoid reading the file twice
+            # Reads to the end of the tile, but avoid reading the file twice.
             handle = StringIO(f.read())
             kwargs['dialect'] = csv.Sniffer().sniff(handle.getvalue())
         elif sniff_limit > 0:
+            # Reads only the start of the file.
             kwargs['dialect'] = csv.Sniffer().sniff(f.read(sniff_limit))
-            # return to the start of the file
             f.seek(0)
 
         reader = csv.reader(handle, header=header, **kwargs)
@@ -84,7 +84,6 @@ def from_csv(cls, path, column_names=None, column_types=None, row_names=None, sk
             rows = itertools.islice(reader, row_limit)
 
         return Table(rows, column_names, column_types, row_names=row_names)
-
     finally:
         if close:
             f.close()

docs/conf.py

@@ -12,7 +12,7 @@
 
 project = 'agate'
 copyright = '2017, Christopher Groskopf'
-version = '1.9.1'
+version = '1.10.0'
 release = version
 
 # -- General configuration ---------------------------------------------------

setup.py

@@ -5,7 +5,7 @@
 
 setup(
     name='agate',
-    version='1.9.1',
+    version='1.10.0',
     description='A data analysis library that is optimized for humans instead of machines.',
     long_description=long_description,
     long_description_content_type='text/x-rst',