[Rotation of an object using a QList of QQuaternion doesn't work]

[JonB]

@appdev
The question was: how many times do you go through the loop?
Did you get your qDebug() line once or 24 times --- you didn't say.
You could at least have printed qDebug() << i<quaternion_w.size() above the for line....
If that is 1 you only execute matrix.rotate(quaternioncsv[i]); once, with i == 0.
How will that do 24 operations?

Assuming this is the problem, I really don't see why you couldn't figure it with qDebug() statements or expect a one-time loop to do 24 operations....

[appdev]

@jsulm I can do one rotation like this :

quaternion = QQuaternion(w, x, y, z);
matrix.rotate(quaternion)

where w, x, y and z are the quaternion values that I specify there.

and yes exactly I want to do an animation.

@JonB Okay, I did that :

qDebug() << i<quaternion_w.size()

I get 24 which is the size of quaternion_w. and also 24 lines and not just one line.

Well I thought that a loop will fill the QList of QQuaternion and at the same time executes the rotation of the object.
For example it will fill in the first field of the QList and then executes the first rotation, then fill in the second field and executes the second rotation ..etc.. so that I can have an animation by the end.

[JonB]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

I get 24 which is the size of quaternion_w. and also 24 lines and not just one line.

OK, fair enough, that is really what I wanted to know. Sorry if I suspected that was not the case in your situation.

For example it will fill in the first field of the QList and then executes the first rotation, then fill in the second field and executes the second rotation ..etc.. so that I can have an animation by the end.

That I have no idea about --- wouldn't know what a "quaternion` was if it slapped me in the face!

But you have to admit that your original

It seems that the for loop isn't being executed and I don't have any reason why.

was not the case, given the qDebug() statements.

[Christian Ehrlicher]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

and yes exactly I want to do an animation.

But how does this correlate to 'matrix' at all? Where do you use it? Currently you simply add all quaternions and then do nothing...

[appdev]

@Christian-Ehrlicher matrix is the modelview matrix.
When I use rotate function, it multiplies the modelview matrix with the matrix that corresponds to the quaternion.

What's confusing is that when I run the application, the object does change its orientation but only one time.

[Christian Ehrlicher]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

its orientation but only one time.

When you don't change quaternion_w - what should change?

[appdev]

I'm sorry, I didn't understand.

Why should I change quaternion_w ?

quaternioncsv is the QList that contains values of QQuaternion and that I will be using to rotate my object.

[Christian Ehrlicher]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

quaternioncsv is the QList that contains values of QQuaternion and that I will be using to rotate my object.

But where? I only see you iterate over a list and modify a variable, nothing more...

[appdev]

Okay, here is the code:

for (int i=0; i<quaternion_w.size();++i)
     {

    quaternioncsv.append(QQuaternion(quaternion_w[i],quaternion_x[i], 
    quaternion_y[i], quaternion_z[i]));
 
    matrix.rotate(quaternioncsv[i]); // the object should rotate

     }

So first thing first I fill the QList "quaternioncsv" with values from other QLists then I use the rotate function that multiplies the modelview matrix by the rotation matrix that corresponds to every quaternion of the QList " quaternioncsv".

The rotate function executes the rotation.

[Christian Ehrlicher]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

matrix.rotate(quaternioncsv[i]); // the object should rotate

there is no object, just a matrix and you don't paint anything after you set the matrix to a new value...

[appdev]

The object is represented by that matrix which is the modelview matrix.

this works perfectly:

quaternion = QQuaternion(0.8536, 0.3536, -0.1464, 0.3536);
matrix.rotate(quaternion)

I tried this with different quaternion values and the object does change its orientation.

If I'm not painting anything after setting the matrix to a new value, how come the code lines I wrote above, make the object rotate. I'm confused.

[Christian Ehrlicher]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

make the object rotate. I'm confused.

Because then you enter the event loop again, a paintEvent is triggered and your stuff is painted...

Use a QTimer, set the desired timeout and advance your rotation to the next state in the connected slot.

[appdev]

Using QTimer and setting the desired timeout is fine but advancing the rotation to the next state isn't really clear ? shouldn't I use something like a loop to make it advance ?

[Christian Ehrlicher]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

shouldn't I use something like a loop to make it advance ?

Remember the current value, add one to get to the next and use this during the next execution of your slot. How else should it work?

[jsulm]

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

but advancing the rotation to the next state isn't really clear ?

In each timeout slot call you take next quaternion from your list and do the rotation.

[appdev]

Okay but don't I need to iterate throught the QList ?

[jsulm]

@appdev As already explained: on each timeout slot call you take next element from the list. Isn't this "iteration"?

[appdev]

But an iteration needs a loop to iterate through the whole QList so I need to use a for loop right ?

[appdev]

I tried something but it didn't work:
I added a new function to my code: it's actually a SLOT

void MainWidget::rotate_csv(QMatrix4x4 m)
{
    for (int i=0; i<quaternioncsv.size();++i)
    {
        m.rotate(quaternioncsv[i]);
    }
}

It iterates through the QList of quaternions and multiplies the matrix "m" with the rotation matrix of quaternioncsv[i].

Then in my paintGL() function I did this:

    this->m_timer=new QTimer(this);
    m_timer->setInterval(10000);
    QTimer::connect(m_timer,SIGNAL(timeout()),this,SLOT(rotate_csv(matrix)));
    m_timer->start();

So according to Qt documentation, the rotate_csv will be called every 10 seconds.
When I run my application, the object remains at its initial orientation. It doesn't start rotating.

[JonB]

@appdev
First, SLOT(rotate_csv(matrix)) isn't even legal. Why you cannot do yourself a favor and change over to New Signal Slot Syntax (unless you are on Qt4, which you do not say), which would catch this, is beyond me.... If you really won't change over, why don't you test the result returned from connect()?

When I run my application, the object remains at its initial orientation. It doesn't start rotating.

Did you put a qDebug() statement inside MainWidget::rotate_csv() to see how many times it was called? I have advised you to do this....

@appdev said in [Rotation of an object using a QList of QQuaternion doesn't work]:

But an iteration needs a loop to iterate through the whole QList so I need to use a for loop right ?

No, this is not what the others are asking you to do. They are asking you to perform one quaternion in the list per each timeout. No for loop!

They are asking you to write something like:

connect(m_timer, &QTimer::timeout, this, &MainWidget::rotate_csv);

m_iterations = 0;
m_timer->start();

void MainWidget::rotate_csv()
{
    if (m_iterations >= quaternioncsv.count())
        return;
    matrix.rotate(quaternioncsv[m_iterations]);
    m_iterations++;
}