Skip to content

Commit 29cd1ba

Browse files
committed
reveiw comments
1 parent f1236c6 commit 29cd1ba

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/Microsoft.ML.Data/Transforms/ColumnConcatenatingEstimator.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
namespace Microsoft.ML.Transforms
1313
{
1414
/// <summary>
15-
/// Concatenates columns in an <see cref="IDataView"/> together.
15+
/// Concatenates columns in an <see cref="IDataView"/> into one single column. Estimator for the <see cref="ColumnConcatenatingTransformer"/>.
1616
/// </summary>
1717
public sealed class ColumnConcatenatingEstimator : IEstimator<ColumnConcatenatingTransformer>
1818
{
@@ -25,7 +25,7 @@ public sealed class ColumnConcatenatingEstimator : IEstimator<ColumnConcatenatin
2525
/// </summary>
2626
/// <param name="env">The local instance of <see cref="IHostEnvironment"/>.</param>
2727
/// <param name="outputColumnName">The name of the resulting column.</param>
28-
/// <param name="inputColumnNames">The columns to concatenate together.</param>
28+
/// <param name="inputColumnNames">The columns to concatenate into one single column.</param>
2929
internal ColumnConcatenatingEstimator(IHostEnvironment env, string outputColumnName, params string[] inputColumnNames)
3030
{
3131
Contracts.CheckValue(env, nameof(env));

src/Microsoft.ML.Data/Transforms/ColumnConcatenatingTransformer.cs

+3-1
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
using Microsoft.ML.Model;
1616
using Microsoft.ML.Model.Onnx;
1717
using Microsoft.ML.Model.Pfa;
18+
using Microsoft.ML.Transforms;
1819
using Newtonsoft.Json.Linq;
1920

2021
[assembly: LoadableClass(ColumnConcatenatingTransformer.Summary, typeof(IDataTransform), typeof(ColumnConcatenatingTransformer), typeof(ColumnConcatenatingTransformer.TaggedOptions), typeof(SignatureDataTransform),
@@ -34,7 +35,8 @@ namespace Microsoft.ML.Data
3435
using PfaType = PfaUtils.Type;
3536

3637
/// <summary>
37-
/// Concatenates columns in an <see cref="IDataView"/> together.
38+
/// Concatenates columns in an <see cref="IDataView"/> into one single column. Please see <see cref="ColumnConcatenatingEstimator"/> for
39+
/// constructing <see cref="ColumnConcatenatingTransformer"/>.
3840
/// </summary>
3941
public sealed class ColumnConcatenatingTransformer : RowToRowTransformerBase
4042
{

0 commit comments

Comments
 (0)