thoughtworksarts / EmoPy

A deep neural net toolkit for emotion analysis via Facial Expression Recognition (FER)
https://thoughtworksarts.io/blog/emopy-emotional-expression-toolkit/
GNU Affero General Public License v3.0
923 stars 264 forks source link

Is there a return value for the predicted value? #43

Closed chizuchizu closed 4 years ago

chizuchizu commented 5 years ago

Hi

When using EmoPy, I noticed that "model.predict ()" had no return value. I want to use the prediction results, but I can't find a setting that returns the prediction results.

Please let me know if you have any.

I am currently making some improvements to the library to get the expected results.

/EmoPy/src/fermodel.py 60th line

# self._print_prediction(prediction[0]) # past
memo = self._print_prediction(prediction[0]) # add
return memo # add

/EmoPy/src/fermodel.py 111th line

return [str(dominant_emotion), normalized_prediction[self.emotion_map[emotion]] * 100]
cptanalatriste commented 4 years ago

Apparently commit https://github.com/thoughtworksarts/EmoPy/commit/3523f4adf8496072d4dd41768475013ab2a3cd1d implements the return values you required.

It was pushed in November 2018. Are you working with the latest version?

chizuchizu commented 4 years ago

Yes, I am. I'd like to down the version but, I'm not sure that it works normally. Thank you for sharing for me.

Why did it remove the return values?

I’m curious about the reason.

cptanalatriste commented 4 years ago

@Chizuchizu not sure about it, I just joined the project :)

Please let me know if you find the new code suitable, so I can close this issue.

microcosm commented 4 years ago

It looks like @Chizuchizu is good here - please re-open if you have a problem @Chizuchizu