-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to enhance AUs from within Facsvatar? (Fix the notebook GUI) #19
Comments
I haven't checked yet what you're trying to do, but those errors seem to be not specific to FACSvatar, but to Python in general. It seems that your text editor is set to use tabs instead of 4 spaces. In the case of Thank you for including the Traceback output of the error. It will be even more readable if you put the code between 2x ```
|
Do I understand correctly that what you want to achieve is what's demonstrated in this video from 00:14 to 00:27?: |
Ah of course, thanks for the advice. Yes, I want to exaggerate some of the AUs that I feel don't show well enough in the Blender model. '''self.multiplier[index] = number''' |
Sorry I mean
|
Hey, how are you doing with this? Sorry for my late reply. Basically what this GUI is doing is sending a command to the bridge module to change the multiplier values. Specifically this piece of code: https://github.com/NumesSanguis/FACSvatar/blob/master/modules/process_bridge/main.py#L190-L223 |
Hi, no problem. It seems to work now when I add lines to the set_new_multiplier function. Thanks! |
Glad to hear you got it to work :) Yes, the GUI is exactly what is shown in the video (the sliders) I linked: Although that was with the older code base of FACSvatar (when the GUI was still working). |
Hi Numes, Since I asked you about the AU multipliers I have been working on many things. So I wonder if you had any time to look at the gui, or whether you know of a way to specify multipliers directly in the Bridge module. Perhaps by adding an array of numbers in the piece of code you mentioned? https://github.com/NumesSanguis/FACSvatar/blob/master/modules/process_bridge/main.py#L190-L223 I would appreciate it if you could share any thoughts you have on this. |
Hi @Animixer , sorry for my slow reply. I'm currently not actively working at this project. It sounds strange though, the GUI should send a command to the Bridge module, which then internally changes the value of the intensity of the AUs (if the message from the GUI properly arrives). In case it does not work, you can indeed do this manually. First of all, you probably want to be on the v0.4.x branch (newer): https://github.com/NumesSanguis/FACSvatar/tree/v0.4.0 The hacky-approach is to hard code the values (if you don't need real-time modifications). This function sets all multiplier values to 1 (except for AU45, which is set to 1.5): The proper way is to send a message to the bridge module with the new multiplier values (what the GUI is supposed to do). But it seems it is currently not working, which why this issue is still open.The GUI uses this function: Hope this can point you in the right direction! |
Hi there,
The set_new_multiplier function in smooth_data.py multiplies the value of AU45 by 1.5. I can change this value, and this seems to work. But I am having trouble adding multipliers for other AU's. Should this be possible or not?
I have tried this by adding a similar line below the one for AU45, so for example:
self.multiplier[15] = 2.0
But this causes an error in the Process_bridge module:
Traceback (most recent call last):
File "main.py", line 43, in
from smooth_data import SmoothData
File "...\FACSvatar\modules\process_bridge\smooth_data.py", line 40
self.multiplier[15] = 2.0
^
TabError: inconsistent use of tabs and spaces in indentation
I have also tried to change the line for AU45 into self.multiplier[15:17] = [2.0,1.5]
But this gives the following error:
Traceback (most recent call last):
File "main.py", line 43, in
from smooth_data import SmoothData
File "...\FACSvatar\modules\process_bridge\smooth_data.py", line 34
def set_new_multiplier(self no_of_columns=17):
^
SyntaxError: invalid syntax
If it is possible to set additional multipliers, can you show me how to do this?
If not, is there another way to enhance AUs? (other than editing the CSV file from OpenFace)
The text was updated successfully, but these errors were encountered: