Updated all senders of #columns in the DataFrame package to use #asArrayOfColumns for performance reasons.

Author: Joshua-Dias-Barreto

src/DataFrame-IO-Tests/DataFrameCsvReaderTest.class.st

@@ -106,10 +106,10 @@ DataFrameCsvReaderTest >> testReadFromString [
 	actualDataFrame := DataFrameCsvReader new readFromString: TestCsvStrings commaCsvString.
 
 	self
-		assertCollection: actualDataFrame columns first
+		assertCollection: actualDataFrame asArrayOfColumns first
 		hasSameElements: #('1:10 am' '1:30 am' '1:50 am' '2:10 am' '2:30 am').
 	self
-		assertCollection: actualDataFrame columns last
+		assertCollection: actualDataFrame asArrayOfColumns last
 		hasSameElements: #('rain' 'rain' 'snow' '-' 'rain')
 ]
 

src/DataFrame-IO/DataFrameTypeDetector.class.st

@@ -146,7 +146,7 @@ DataFrameTypeDetector >> detectColumnTypeAndConvert: anArray [
 { #category : #'public API' }
 DataFrameTypeDetector >> detectTypesAndConvert: aDataFrame [
 
-	aDataFrame columns with: aDataFrame columnNames do: [ :column :columnName |
+	aDataFrame asArrayOfColumns with: aDataFrame columnNames do: [ :column :columnName |
 		| thisColumnType |
 		"Get the user given column type for this column name and if it wasn't
 			 given then use the default type detection"

src/DataFrame/DataFrame.class.st

@@ -543,7 +543,7 @@ DataFrame >> average [
 { #category : #'data-types' }
 DataFrame >> calculateDataTypes [
 
-	self columns doWithIndex: [ :column :i | self dataTypes at: (self columnNames at: i) put: column calculateDataType ]
+	self asArrayOfColumns doWithIndex: [ :column :i | self dataTypes at: (self columnNames at: i) put: column calculateDataType ]
 ]
 
 { #category : #comparing }
@@ -1052,8 +1052,10 @@ DataFrame >> hasNils [
 	"Returns true if there is atleast one nil value in the data frame. Returns false if there are no nil values in the dataframe"
 
 	| arrayOfColumns |
-	arrayOfColumns := self columns.
-	arrayOfColumns collect: [ :column | column hasNil ifTrue: [ ^ true ] ].
+	arrayOfColumns := self asArrayOfColumns.
+	1 to: self numberOfColumns do: [ :column |
+		1 to: self numberOfRows do: [ :row |
+		((arrayOfColumns at: column) at: row) ifNil: [ ^ true ] ] ].
 	^ false
 ]
 
@@ -1394,7 +1396,7 @@ DataFrame >> normalized [
 		'DataFrame will remove the dependency over normalization in the next version. You can use pharo-ai/data-preprocessing project to normalize your DataFrame and even more!'.
 	normalizers := (1 to: self anyOne size) collect: [ :e | self class defaultNormalizerClass new ].
 
-	normalizedColumns := self columns with: normalizers collect: [ :col :normalizer | col normalizedUsing: normalizer ].
+	normalizedColumns := self asArrayOfColumns with: normalizers collect: [ :col :normalizer | col normalizedUsing: normalizer ].
 
 	^ self class withColumns: normalizedColumns columnNames: self columnNames
 ]