DataFrame icon indicating copy to clipboard operation
DataFrame copied to clipboard

DataFrame references the missing class AIMissingCategory

Open olekscode opened this issue 2 years ago • 2 comments

This method requires pharo-ai to be loaded into the image Which is strange because DataFrame should not depend on AI So we need to check, maybe it's the method that was accidentally committed here

dataPreProcessingEncodeWith: anEncoder
	"This method is here to speed up pharo-ai/data-preprocessing algos without coupling both projects."

	| copy cache |
	copy := self copy.
	cache := IdentityDictionary new.
	self columns doWithIndex: [ :dataSerie :columnIndex |
		| category |
		category := cache at: columnIndex ifAbsentPut: [ ((anEncoder categories at: columnIndex) collectWithIndex: [ :elem :index | elem -> index ]) asDictionary ].
		dataSerie doWithIndex: [ :element :rowIndex |
			copy at: rowIndex at: columnIndex put: (category at: element ifAbsent: [ AIMissingCategory signalFor: element ]) ] ].

	^ copy

olekscode avatar Jun 04 '23 14:06 olekscode

This is my fault indeed :)

I'll add to my todo list to fix this.

jecisc avatar Jun 07 '23 09:06 jecisc

This method should be moved to https://github.com/pharo-ai/data-preprocessing and removed from this repository

olekscode avatar Apr 23 '24 10:04 olekscode