Skip to content
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

Thanks agains (+ some issue) #1

Open
ANaaim opened this issue Feb 27, 2024 · 5 comments
Open

Thanks agains (+ some issue) #1

ANaaim opened this issue Feb 27, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@ANaaim
Copy link

ANaaim commented Feb 27, 2024

Great work :D Should ease some video verification. Thanks again for all this work super helpful.

Some problem i just encounter :

  • When importing video and trc file from the same acqusition (the trc file is the one generated with P2S from the video) the video and the marker are not synchornised. Is there any additional stuff to do in blender to be able to have everything synchronise temporaly ?

  • Maybe the name of the camera when imported from a toml file should not be the number of the camera but its true name :
    For example for us it should be 26578 instead of cam_1 ==> it might be much easier to associate camera in blender.

[cam_1]
name = "26578"
size = [ 1088.0, 1920.0,]
matrix = [ [ 1677.425415046875, 0.0, 550.739013671875,], [ 0.0, 1677.491943359375, 959.772338875,], [ 0.0, 0.0, 1.0,],]
distortions = [ -0.000715875, 0.002123484375, 4.828125e-06, 7.15625e-06,]
rotation = [ -0.0801490876242585, 2.55670003636537, -1.7035997593979935,]
translation = [ -1.1008491007352483, 0.9451061357111141, 2.759258575604535,]
fisheye = false

[cam_2]
name = "26579"
size = [ 1088.0, 1920.0,]
matrix = [ [ 1680.458740234375, 0.0, 545.0693359375,], [ 0.0, 1680.66064453125, 923.3432006875,], [ 0.0, 0.0, 1.0,],]
distortions = [ -0.000690171875, 0.00212715625, 1.7359375e-05, -6.96875e-06,]
rotation = [ 1.651872305424128, -1.3123573659257894, 0.8062765511334361,]
translation = [ 0.40351437570705406, -0.18787945794784702, 4.89854989774333,]
fisheye = false
@ANaaim ANaaim changed the title Thanks agins (+ some issue) Thanks agains (+ some issue) Feb 27, 2024
@davidpagnon
Copy link
Owner

davidpagnon commented Mar 15, 2024

Sorry @ANaaim, I just noticed that I was not "watching" the repository and did not get any notification about it.

  • How are the data unsynchronized? Is it a frame shift (how many frames?), or a framerate issue? In the first case, I'll have to see, maybe this is a problem of Blender starting at frame Thanks agains (+ some issue) #1 while python index starts at #0, I'm not sure. In the second case, did you select the right target_framerate upon importing the trc file? (upper right of the file selector window)
  • You are right for the camera name, I'll do the change now!

@davidpagnon davidpagnon added the enhancement New feature or request label Mar 15, 2024
@ANaaim
Copy link
Author

ANaaim commented Mar 18, 2024

it is a Framerate issue so I will have a look at the option you are talking about because I did not see it.

@ANaaim
Copy link
Author

ANaaim commented Mar 18, 2024

It works when indicating the correct frequencies ==> Would it be possible to read directly the value in the trc to use this value by default ?

@davidpagnon
Copy link
Owner

Yes, I can do this for the trc files, but not for the csv nor mot files as there is no framerate specification.
It is not complicated but I'm quite busy this week, I may not do it before next week or so!

@ANaaim
Copy link
Author

ANaaim commented Mar 22, 2024

No hurry, it is more my though about it as a user rather than a need :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants