Rename IRow to Row, and IStatefulRow to StatefulRow.

Author: TomFinley

src/Microsoft.ML.Api/CustomMappingTransformer.cs

@@ -111,7 +111,7 @@ public Mapper(CustomMappingTransformer<TSrc, TDst> parent, Schema inputSchema)
                 _typedSrc = TypedCursorable<TSrc>.Create(_host, emptyDataView, false, _parent.InputSchemaDefinition);
             }
 
-            public Delegate[] CreateGetters(IRow input, Func<int, bool> activeOutput, out Action disposer)
+            public Delegate[] CreateGetters(Row input, Func<int, bool> activeOutput, out Action disposer)
             {
                 disposer = null;
                 // If no outputs are active, we short-circuit to empty array of getters.
@@ -147,7 +147,7 @@ public Delegate[] CreateGetters(IRow input, Func<int, bool> activeOutput, out Ac
                 return result;
             }
 
-            private Delegate GetDstGetter<T>(IRow input, int colIndex, Action refreshAction)
+            private Delegate GetDstGetter<T>(Row input, int colIndex, Action refreshAction)
             {
                 var getter = input.GetGetter<T>(colIndex);
                 ValueGetter<T> combinedGetter = (ref T dst) =>

src/Microsoft.ML.Api/DataViewConstructionUtils.cs

@@ -125,11 +125,11 @@ protected override TRow GetCurrentRowObject()
         }
 
         /// <summary>
-        /// A row that consumes items of type <typeparamref name="TRow"/>, and provides an <see cref="IRow"/>. This
+        /// A row that consumes items of type <typeparamref name="TRow"/>, and provides an <see cref="Row"/>. This
         /// is in contrast to <see cref="IRowReadableAs{TRow}"/> which consumes a data view row and publishes them as the output type.
         /// </summary>
         /// <typeparam name="TRow">The input data type.</typeparam>
-        public abstract class InputRowBase<TRow> : IRow
+        public abstract class InputRowBase<TRow> : Row
             where TRow : class
         {
             private readonly int _colCount;

src/Microsoft.ML.Api/StatefulFilterTransform.cs

@@ -150,7 +150,7 @@ private sealed class Cursor : RootCursorBase
 
             private readonly RowCursor<TSrc> _input;
             // This is used to serve getters for the columns we produce.
-            private readonly IRow _appendedRow;
+            private readonly Row _appendedRow;
 
             private readonly TSrc _src;
             private readonly TDst _dst;

src/Microsoft.ML.Api/TypedCursor.cs

@@ -13,7 +13,7 @@
 namespace Microsoft.ML.Runtime.Api
 {
     /// <summary>
-    /// This interface is an <see cref="IRow"/> with 'strongly typed' binding.
+    /// This interface is an <see cref="Row"/> with 'strongly typed' binding.
     /// It can populate the user-supplied object's fields with the values of the current row.
     /// </summary>
     /// <typeparam name="TRow">The user-defined type that is being populated while cursoring.</typeparam>
@@ -29,7 +29,7 @@ internal interface IRowReadableAs<TRow>
     }
 
     /// <summary>
-    /// This interface is an <see cref="IRow"/> with 'strongly typed' binding.
+    /// This interface is an <see cref="Row"/> with 'strongly typed' binding.
     /// It can accept values of type <typeparamref name="TRow"/> and present the value as a row.
     /// </summary>
     /// <typeparam name="TRow">The user-defined type that provides the values while cursoring.</typeparam>
@@ -179,7 +179,7 @@ public RowCursor<TRow> GetRandomizedCursor(int randomSeed)
             return GetCursor(x => false, randomSeed);
         }
 
-        public IRowReadableAs<TRow> GetRow(IRow input)
+        public IRowReadableAs<TRow> GetRow(Row input)
         {
             return new RowImplementation(new TypedRow(this, input));
         }
@@ -256,7 +256,7 @@ public static TypedCursorable<TRow> Create(IHostEnvironment env, IDataView data,
         private abstract class TypedRowBase
         {
             protected readonly IChannel Ch;
-            private readonly IRow _input;
+            private readonly Row _input;
             private readonly Action<TRow>[] _setters;
 
             public long Batch => _input.Batch;
@@ -265,7 +265,7 @@ private abstract class TypedRowBase
 
             public Schema Schema => _input.Schema;
 
-            public TypedRowBase(TypedCursorable<TRow> parent, IRow input, string channelMessage)
+            public TypedRowBase(TypedCursorable<TRow> parent, Row input, string channelMessage)
             {
                 Contracts.AssertValue(parent);
                 Contracts.AssertValue(parent._host);
@@ -284,12 +284,12 @@ public TypedRowBase(TypedCursorable<TRow> parent, IRow input, string channelMess
 
             public ValueGetter<UInt128> GetIdGetter() => _input.GetIdGetter();
 
-            private Action<TRow> GenerateSetter(IRow input, int index, InternalSchemaDefinition.Column column, Delegate poke, Delegate peek)
+            private Action<TRow> GenerateSetter(Row input, int index, InternalSchemaDefinition.Column column, Delegate poke, Delegate peek)
             {
                 var colType = input.Schema.GetColumnType(index);
                 var fieldType = column.OutputType;
                 var genericType = fieldType;
-                Func<IRow, int, Delegate, Delegate, Action<TRow>> del;
+                Func<Row, int, Delegate, Delegate, Action<TRow>> del;
                 if (fieldType.IsArray)
                 {
                     Ch.Assert(colType.IsVector);
@@ -348,7 +348,7 @@ private Action<TRow> GenerateSetter(IRow input, int index, InternalSchemaDefinit
             // REVIEW: The converting getter invokes a type conversion delegate on every call, so it's inherently slower
             // than the 'direct' getter. We don't have good indication of this to the user, and the selection
             // of affected types is pretty arbitrary (signed integers and bools, but not uints and floats).
-            private Action<TRow> CreateConvertingVBufferSetter<TSrc, TDst>(IRow input, int col, Delegate poke, Delegate peek, Func<TSrc, TDst> convert)
+            private Action<TRow> CreateConvertingVBufferSetter<TSrc, TDst>(Row input, int col, Delegate poke, Delegate peek, Func<TSrc, TDst> convert)
             {
                 var getter = input.GetGetter<VBuffer<TSrc>>(col);
                 var typedPoke = poke as Poke<TRow, TDst[]>;
@@ -370,7 +370,7 @@ private Action<TRow> CreateConvertingVBufferSetter<TSrc, TDst>(IRow input, int c
                 };
             }
 
-            private Action<TRow> CreateDirectVBufferSetter<TDst>(IRow input, int col, Delegate poke, Delegate peek)
+            private Action<TRow> CreateDirectVBufferSetter<TDst>(Row input, int col, Delegate poke, Delegate peek)
             {
                 var getter = input.GetGetter<VBuffer<TDst>>(col);
                 var typedPoke = poke as Poke<TRow, TDst[]>;
@@ -408,7 +408,7 @@ private Action<TRow> CreateDirectVBufferSetter<TDst>(IRow input, int col, Delega
                 };
             }
 
-            private static Action<TRow> CreateConvertingActionSetter<TSrc, TDst>(IRow input, int col, Delegate poke, Func<TSrc, TDst> convert)
+            private static Action<TRow> CreateConvertingActionSetter<TSrc, TDst>(Row input, int col, Delegate poke, Func<TSrc, TDst> convert)
             {
                 var getter = input.GetGetter<TSrc>(col);
                 var typedPoke = poke as Poke<TRow, TDst>;
@@ -422,7 +422,7 @@ private static Action<TRow> CreateConvertingActionSetter<TSrc, TDst>(IRow input,
                 };
             }
 
-            private static Action<TRow> CreateDirectSetter<TDst>(IRow input, int col, Delegate poke, Delegate peek)
+            private static Action<TRow> CreateDirectSetter<TDst>(Row input, int col, Delegate poke, Delegate peek)
             {
                 // Awkward to have a parameter that's always null, but slightly more convenient for generalizing the setter.
                 Contracts.Assert(peek == null);
@@ -437,7 +437,7 @@ private static Action<TRow> CreateDirectSetter<TDst>(IRow input, int col, Delega
                 };
             }
 
-            private Action<TRow> CreateVBufferToVBufferSetter<TDst>(IRow input, int col, Delegate poke, Delegate peek)
+            private Action<TRow> CreateVBufferToVBufferSetter<TDst>(Row input, int col, Delegate poke, Delegate peek)
             {
                 var getter = input.GetGetter<VBuffer<TDst>>(col);
                 var typedPoke = poke as Poke<TRow, VBuffer<TDst>>;
@@ -472,7 +472,7 @@ public ValueGetter<TValue> GetGetter<TValue>(int col)
 
         private sealed class TypedRow : TypedRowBase
         {
-            public TypedRow(TypedCursorable<TRow> parent, IRow input)
+            public TypedRow(TypedCursorable<TRow> parent, Row input)
                 : base(parent, input, "Row")
             {
             }

src/Microsoft.ML.Core/Data/IDataView.cs

@@ -102,7 +102,7 @@ public interface IDataView
         /// should return the "same" row as would have been returned through the regular serial cursor,
         /// but all rows should be returned by exactly one of the cursors returned from this cursor.
         /// The cursors can have their values reconciled downstream through the use of the
-        /// <see cref="IRow.Batch"/> property.
+        /// <see cref="Row.Batch"/> property.
         /// </summary>
         /// <param name="consolidator">This is an object that can be used to reconcile the
         /// returned array of cursors. When the array of cursors is of length 1, it is legal,
@@ -142,7 +142,7 @@ public interface IRowCursorConsolidator
     /// A logical row. May be a row of an <see cref="IDataView"/> or a stand-alone row. If/when its contents
     /// change, its <see cref="Position"/> value is changed.
     /// </summary>
-    public abstract class IRow
+    public abstract class Row
     {
         /// <summary>
         /// This is incremented when the underlying contents changes, giving clients a way to detect change.
@@ -169,7 +169,7 @@ public abstract class IRow
         public abstract long Batch { get; }
 
         /// <summary>
-        /// A getter for a 128-bit ID value. It is common for objects to serve multiple <see cref="IRow"/>
+        /// A getter for a 128-bit ID value. It is common for objects to serve multiple <see cref="Row"/>
         /// instances to iterate over what is supposed to be the same data, for example, in a <see cref="IDataView"/>
         /// a cursor set will produce the same data as a serial cursor, just partitioned, and a shuffled cursor
         /// will produce the same data as a serial cursor or any other shuffled cursor, only shuffled. The ID
@@ -216,12 +216,12 @@ public enum CursorState
 
     /// <summary>
     /// The basic cursor base class to cursor through rows of an <see cref="IDataView"/>. Note that
-    /// this is also an <see cref="IRow"/>. The <see cref="IRow.Position"/> is incremented by <see cref="MoveNext"/>
+    /// this is also an <see cref="Row"/>. The <see cref="Row.Position"/> is incremented by <see cref="MoveNext"/>
     /// and <see cref="MoveMany"/>. When the cursor state is <see cref="CursorState.NotStarted"/> or
-    /// <see cref="CursorState.Done"/>, <see cref="IRow.Position"/> is <c>-1</c>. Otherwise,
-    /// <see cref="IRow.Position"/> >= 0.
+    /// <see cref="CursorState.Done"/>, <see cref="Row.Position"/> is <c>-1</c>. Otherwise,
+    /// <see cref="Row.Position"/> >= 0.
     /// </summary>
-    public abstract class RowCursor : IRow, IDisposable
+    public abstract class RowCursor : Row, IDisposable
     {
         /// <summary>
         /// Returns the state of the cursor. Before the first call to <see cref="MoveNext"/> or
@@ -245,11 +245,11 @@ public abstract class RowCursor : IRow, IDisposable
         public abstract bool MoveMany(long count);
 
         /// <summary>
-        /// Returns a cursor that can be used for invoking <see cref="IRow.Position"/>, <see cref="State"/>,
+        /// Returns a cursor that can be used for invoking <see cref="Row.Position"/>, <see cref="State"/>,
         /// <see cref="MoveNext"/>, and <see cref="MoveMany"/>, with results identical to calling those
         /// on this cursor. Generally, if the root cursor is not the same as this cursor, using the
         /// root cursor will be faster. As an aside, note that this is not necessarily the case of
-        /// values from <see cref="IRow.GetIdGetter"/>.
+        /// values from <see cref="Row.GetIdGetter"/>.
         /// </summary>
         public abstract RowCursor GetRootCursor();
         public abstract void Dispose();

src/Microsoft.ML.Core/Data/ISchemaBindableMapper.cs

@@ -66,7 +66,7 @@ public interface ISchemaBoundRowMapper : ISchemaBoundMapper, IRowToRowMapper
     }
 
     /// <summary>
-    /// This interface maps an input <see cref="IRow"/> to an output <see cref="IRow"/>. Typically, the output contains
+    /// This interface maps an input <see cref="Row"/> to an output <see cref="Row"/>. Typically, the output contains
     /// both the input columns and new columns added by the implementing class, although some implementations may
     /// return a subset of the input columns.
     /// This interface is similar to <see cref="ISchemaBoundRowMapper"/>, except it does not have any input role mappings,
@@ -93,26 +93,26 @@ public interface IRowToRowMapper
         Func<int, bool> GetDependencies(Func<int, bool> predicate);
 
         /// <summary>
-        /// Get an <see cref="IRow"/> with the indicated active columns, based on the input <paramref name="input"/>.
+        /// Get an <see cref="Row"/> with the indicated active columns, based on the input <paramref name="input"/>.
         /// The active columns are those for which <paramref name="active"/> returns true. Getting values on inactive
         /// columns of the returned row will throw. Null predicates are disallowed.
         ///
-        /// The <see cref="IRow.Schema"/> of <paramref name="input"/> should be the same object as
+        /// The <see cref="Row.Schema"/> of <paramref name="input"/> should be the same object as
         /// <see cref="InputSchema"/>. Implementors of this method should throw if that is not the case. Conversely,
         /// the returned value must have the same schema as <see cref="OutputSchema"/>.
         ///
-        /// This method creates a live connection between the input <see cref="IRow"/> and the output <see
-        /// cref="IRow"/>. In particular, when the getters of the output <see cref="IRow"/> are invoked, they invoke the
-        /// getters of the input row and base the output values on the current values of the input <see cref="IRow"/>.
-        /// The output <see cref="IRow"/> values are re-computed when requested through the getters.
+        /// This method creates a live connection between the input <see cref="Row"/> and the output <see
+        /// cref="Row"/>. In particular, when the getters of the output <see cref="Row"/> are invoked, they invoke the
+        /// getters of the input row and base the output values on the current values of the input <see cref="Row"/>.
+        /// The output <see cref="Row"/> values are re-computed when requested through the getters.
         ///
         /// The optional <paramref name="disposer"/> should be invoked by any user of this row mapping, once it no
-        /// longer needs the <see cref="IRow"/>. If no action is needed when the cursor is Disposed, the implementation
+        /// longer needs the <see cref="Row"/>. If no action is needed when the cursor is Disposed, the implementation
         /// should set <paramref name="disposer"/> to <c>null</c>, otherwise it should be set to a delegate to be
         /// invoked by the code calling this object. (For example, a wrapping cursor's <see cref="IDisposable.Dispose"/>
         /// method. It's best for this action to be idempotent - calling it multiple times should be equivalent to
         /// calling it once.
         /// </summary>
-        IRow GetRow(IRow input, Func<int, bool> active, out Action disposer);
+        Row GetRow(Row input, Func<int, bool> active, out Action disposer);
     }
 }

src/Microsoft.ML.Core/Data/MetadataUtils.cs

@@ -495,7 +495,7 @@ public static bool TryGetCategoricalFeatureIndices(Schema schema, int colIndex,
             return cols;
         }
 
-        private sealed class MetadataRow : IRow
+        private sealed class MetadataRow : Row
         {
             private readonly Schema.Metadata _metadata;
 
@@ -514,11 +514,11 @@ public MetadataRow(Schema.Metadata metadata)
         }
 
         /// <summary>
-        /// Presents a <see cref="Schema.Metadata"/> as a an <see cref="IRow"/>.
+        /// Presents a <see cref="Schema.Metadata"/> as a an <see cref="Row"/>.
         /// </summary>
         /// <param name="metadata">The metadata to wrap.</param>
         /// <returns>A row that wraps an input metadata.</returns>
-        public static IRow MetadataAsRow(Schema.Metadata metadata)
+        public static Row MetadataAsRow(Schema.Metadata metadata)
         {
             Contracts.CheckValue(metadata, nameof(metadata));
             return new MetadataRow(metadata);

src/Microsoft.ML.Core/Data/Schema.cs

@@ -14,7 +14,7 @@
 namespace Microsoft.ML.Data
 {
     /// <summary>
-    /// This class represents the <see cref="Schema"/> of an object like, for interstance, an <see cref="IDataView"/> or an <see cref="IRow"/>.
+    /// This class represents the <see cref="Schema"/> of an object like, for interstance, an <see cref="IDataView"/> or an <see cref="Row"/>.
     /// On the high level, the schema is a collection of 'columns'. Each column has the following properties:
     /// - Column name.
     /// - Column type.

src/Microsoft.ML.Data/Data/DataViewUtils.cs

@@ -644,7 +644,7 @@ private RowCursor[] SplitCore(out IRowCursorConsolidator consolidator, IChannelP
             /// <summary>
             /// An in pipe creator intended to be used from the splitter only.
             /// </summary>
-            private InPipe CreateInPipe<T>(IRow input, int col)
+            private InPipe CreateInPipe<T>(Row input, int col)
             {
                 Contracts.AssertValue(input);
                 Contracts.Assert(0 <= col && col < _schema.ColumnCount);
@@ -654,7 +654,7 @@ private InPipe CreateInPipe<T>(IRow input, int col)
             /// <summary>
             /// An in pipe creator intended to be used from the splitter only.
             /// </summary>
-            private InPipe CreateIdInPipe(IRow input)
+            private InPipe CreateIdInPipe(Row input)
             {
                 Contracts.AssertValue(input);
                 return CreateInPipeCore(_schema.ColumnCount + (int)ExtraIndex.Id, input.GetIdGetter());
@@ -849,7 +849,7 @@ public void SetAll(OutPipe[] pipes)
 
             /// <summary>
             /// This helps a cursor present the results of a <see cref="BatchColumn"/>. Practically its role
-            /// really is to just provide a stable delegate for the <see cref="IRow.GetGetter{T}(int)"/>.
+            /// really is to just provide a stable delegate for the <see cref="Row.GetGetter{T}(int)"/>.
             /// There is one of these created per column, per output cursor, i.e., in splitting
             /// there are <c>n</c> of these created per column, and when consolidating only one of these
             /// is created per column.
@@ -1332,7 +1332,7 @@ public static ValueGetter<ReadOnlyMemory<char>>[] PopulateGetterArray(RowCursor
             return getters;
         }
 
-        public static ValueGetter<ReadOnlyMemory<char>> GetSingleValueGetter<T>(IRow cursor, int i, ColumnType colType)
+        public static ValueGetter<ReadOnlyMemory<char>> GetSingleValueGetter<T>(Row cursor, int i, ColumnType colType)
         {
             var floatGetter = cursor.GetGetter<T>(i);
             T v = default(T);
@@ -1362,7 +1362,7 @@ public static ValueGetter<ReadOnlyMemory<char>> GetSingleValueGetter<T>(IRow cur
             return getter;
         }
 
-        public static ValueGetter<ReadOnlyMemory<char>> GetVectorFlatteningGetter<T>(IRow cursor, int colIndex, ColumnType colType)
+        public static ValueGetter<ReadOnlyMemory<char>> GetVectorFlatteningGetter<T>(Row cursor, int colIndex, ColumnType colType)
         {
             var vecGetter = cursor.GetGetter<VBuffer<T>>(colIndex);
             var vbuf = default(VBuffer<T>);

src/Microsoft.ML.Data/Data/IRowSeekable.cs

@@ -24,7 +24,7 @@ public interface IRowSeekable
     /// For <see cref="RowSeeker"/>, when the state is valid (that is when <see cref="MoveTo(long)"/>
     /// returns <see langword="true"/>), it returns the current row index. Otherwise it's -1.
     /// </summary>
-    public abstract class RowSeeker : IRow, IDisposable
+    public abstract class RowSeeker : Row, IDisposable
     {
         public abstract void Dispose();