Skip to content

Min/Max Scaler function implementation#8

Open
i-wegner wants to merge 1 commit intomainfrom
feat-scaler
Open

Min/Max Scaler function implementation#8
i-wegner wants to merge 1 commit intomainfrom
feat-scaler

Conversation

@i-wegner
Copy link
Collaborator

@a-sparrowhawk @ModiPriyanshu

Sorry for the delay in this pull request, but I believe everything is working as it should and handles my tests. Please review and leave any comments for me if necessary.

CC: @affans

@a-sparrowhawk
Copy link
Collaborator

Hello Isaac, everything seems to be running properly.

#data = [1] #returns [0]

#data = [] #good returns an empty list

#data = [-1] #returns [0]

#data = [1, 1] #returns [0, 0] which is good

#data = [-1, -1] #returns [0, 0] which is good

#data = [1, 2, 3, 4, 5, 6] #returns [0.0, 0.2, 0.4, 0.6, 0.8, 1.0] which is good

#data = [-1, -2, -3, -4, -5] #returns [1.0, 0.75, 0.5, 0.25, 0.0] which is good

#data = [1, 2, 3, 3, 3, 3, 4] #good

#data = [1, 1, 2, 3, 4, 5] #good

#data = [1, 2, 3, 4, 4] #good

#data = [1, 1, 1, 1] #good

#data = [-1, 0, 3, 4, 5, -8] #good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants