Remove redundant column names collection from DataFrameColumnCollection

[asmirnov82]

column names collection was used to get column name by column index. As DataFrameColumnCollection is already collection of columns that can be accessed by index - this[i].Name can be used instead to get name. Storing dedicated collection of names is excessive,

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (main@150e68f). Click here to learn what that means.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6701   +/-   ##
=======================================
  Coverage        ?   68.77%           
=======================================
  Files           ?     1215           
  Lines           ?   251723           
  Branches        ?    26256           
=======================================
  Hits            ?   173132           
  Misses          ?    71773           
  Partials        ?     6818           

Flag	Coverage Δ
Debug	68.77% <100.00%> (?)
production	63.29% <100.00%> (?)
test	88.88% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...crosoft.Data.Analysis/DataFrameColumnCollection.cs	79.82% <100.00%> (ø)

[JakeRadMSFT]

@asmirnov82 thanks so much for all your contributions to DataFrame.

I’m on vacation but I can review in about a week and I’m the resident expert at the moment.

Would you be open to porting them to another branch?

#6664

I have a fairly big PR to modernize DataFrame to .NET 6 and Generic Math. If you could open a PR into that branch also… it would help a bunch.

[asmirnov82]

Hi, @JakeRadMSFT. Many thanks for you quick response. I hope you're having a great time enjoying your vacation!

I created PR with porting all my changes to a new logic with general math. I also had to merge the latest main branch from the original machinelearning repo to avoid merging conflicts.

You may check, when you have time: https://github.com/JakeRadMSFT/machinelearning/pull/2/commits

[JakeRadMSFT]

Hi, @JakeRadMSFT. Many thanks for you quick response. I hope you're having a great time enjoying your vacation!

I created PR with porting all my changes to a new logic with general math. I also had to merge the latest main branch from the original machinelearning repo to avoid merging conflicts.

You may check, when you have time: https://github.com/JakeRadMSFT/machinelearning/pull/2/commits

Thanks so much!

[michaelgsharp]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[michaelgsharp]

LGTM. Thanks!

[JakeRadMSFT]

Merged into Generic Math branch. (This is just a note for myself)