erratic behavior of graphics items

[sajis997]

Hello forum,

Please Check the attached image. Screenshot-1.jpg. You can see that two elliptical items are connected by an arrow graphics item.

If the user clicks the small ellipse inside the large ellipse an arrow item is instantiated. The arrow item is extended with the mouse move and mouse release event.

I also have over-ridden the following function so that the arrow is adjusted even after the position changes of the elliptical items.

QVariant H3DHierarchyArrowDockGraphicsItem::itemChange(GraphicsItemChange change, const QVariant &value)
{
    switch(change)
    {
    case ItemPositionHasChanged:
        foreach(H3DHierarchyArrowGraphicsItem *arrow, m_hierarchyArrowList)
            arrow->adjust();
        break;
    default:
        break;
    }
 
    return QGraphicsItem::itemChange(change,value);
}

QVariant H3DHierarchyArrowDockGraphicsItem::itemChange(GraphicsItemChange change, const QVariant &value)
{
    switch(change)
    {
    case ItemPositionHasChanged:
        foreach(H3DHierarchyArrowGraphicsItem *arrow, m_hierarchyArrowList)
            arrow->adjust();
        break;
    default:
        break;
    }

    return QGraphicsItem::itemChange(change,value);
}

To copy to clipboard, switch view to plain text mode 

But at some point i get the following output while moving the items inside the scene.
Bizarre.jpg



As you can see the arrow is totally detached from the destination item. And the arrow is attached back while moving the elliptical items - not so pleasant behavior, is it?

To get a functional one i have taken hint from elastic nodes, diagram scene example and voreen editor( Qt based volume rendering editor), but it seems that i have missed some ideas to get it right.

To provide me with useful hint i am also attaching the corresponding source files that you need to look into.

H3DHierarchyArrowGraphicsItem.cppH3DHierarchyArrowDockGraphicsItem.cppArrowGraphicsItem.cpp


If the arrow has both the source and destination item i am doing the intersection test just as in the diagramscene example of Qt so that the destination point always slides on the arc of the ellipse. All the arrows that are created has the small ellipse and another larger ellipse as the destination item.

In inside the adjust() function i am trying to update the destination point of the arrow which seems not to be working. Please make some time to go through it.


Thanks
Sajjad

[sajis997]

Hello forum,

I have not got any reply yet on this issue. Please let me know if my elaboration of the problem is not clear enough.

If two graphical items of different types are connected to a single arrow at both ends, is mandatory to call the adjust() function of arrow graphics item from the both the graphical items?


I am looking forward to some feed-back



Regards
Sajjad

[wysota]

It would help if you provided a minimal compilable example reproducing the problem.

[sajis997]

Hi

I have managed to recreate the same problem i am having in my project by doing the following changes in the diagramscene example:


Inside the diagramitem.h file i added another item as follows:

enum DiagramType
    {
       Step,
       Conditional,
       StartEnd,
       Io ,
       Ellipse
    };

enum DiagramType
    {
       Step,
       Conditional,
       StartEnd,
       Io ,
       Ellipse
    };

To copy to clipboard, switch view to plain text mode 

Inside the diagramitem constructor i have added the following:

case Ellipse:
            {
                //declare an ellipse with the bounding rectange
//                QGraphicsEllipseItem item(-100,-50,200,100);
                QGraphicsEllipseItem item(0,0,200,100);
                myPolygon = item.shape().toFillPolygon();
            }
            break;

case Ellipse:
            {
                //declare an ellipse with the bounding rectange
//                QGraphicsEllipseItem item(-100,-50,200,100);
                QGraphicsEllipseItem item(0,0,200,100);
                myPolygon = item.shape().toFillPolygon();
            }
            break;

To copy to clipboard, switch view to plain text mode 

I am sure not how to do this arrow and elliptical intersection test where in my project i have defined the bounding rectangle and shape for the elliptical item as follows:

QRectF RootGraphicsItem::boundingRect() const
{
    //!get the bounding rectangle of the enclosing text
    QRectF rect = m_textGraphicsItem->boundingRect();
 
 
    //the following condition decides if the boundary needs
    //to be changed
    if( rect.width() < m_drawRectMinimumWidth )
        rect.setWidth(m_drawRectMinimumWidth);
 
    if(rect.height() < m_drawRectMinimumHeight)
        rect.setHeight(m_drawRectMinimumHeight);
 
    //set the rectangle width with the text item spacing
    rect.setWidth(rect.width() + getTextItemSpacing() );
 
    //and return the rectangle
    return rect;
}
 
//the following function is for better collission detection
QPainterPath RootGraphicsItem::shape() const
{
    QRectF ellipse_rect = boundingRect();
 
    QPainterPath path;
 
    path.addEllipse(ellipse_rect);
 
    return path;
}

QRectF RootGraphicsItem::boundingRect() const
{
    //!get the bounding rectangle of the enclosing text
    QRectF rect = m_textGraphicsItem->boundingRect();


    //the following condition decides if the boundary needs
    //to be changed
    if( rect.width() < m_drawRectMinimumWidth )
        rect.setWidth(m_drawRectMinimumWidth);

    if(rect.height() < m_drawRectMinimumHeight)
        rect.setHeight(m_drawRectMinimumHeight);

    //set the rectangle width with the text item spacing
    rect.setWidth(rect.width() + getTextItemSpacing() );

    //and return the rectangle
    return rect;
}

//the following function is for better collission detection
QPainterPath RootGraphicsItem::shape() const
{
    QRectF ellipse_rect = boundingRect();

    QPainterPath path;

    path.addEllipse(ellipse_rect);

    return path;
}

To copy to clipboard, switch view to plain text mode 

There are some affine transformation issues need to be taken care of - i guess and i am not sure how to do it. Do i have call some translation command before calling the ellipse drawing function ? I tried this as well and it became messier.


Looking forward to some useful hint over this .



Thanks
Sajjad

[wysota]

And where do you call prepareGeometryChange()?

[sajis997]

Hi

i have inserted the function call prepareGeometryChange inside the following function at arrow.cpp of the updated diagram scene example to recreate the problem.

void Arrow::updatePosition()
{
    prepareGeometryChange();
 
    //declare a new line
    QLineF line(mapFromItem(myStartItem, 0, 0), mapFromItem(myEndItem, 0, 0));
    setLine(line);
}

void Arrow::updatePosition()
{
    prepareGeometryChange();

    //declare a new line
    QLineF line(mapFromItem(myStartItem, 0, 0), mapFromItem(myEndItem, 0, 0));
    setLine(line);
}

To copy to clipboard, switch view to plain text mode 

And the above function is called as follows:

QVariant DiagramItem::itemChange(GraphicsItemChange change,
                     const QVariant &value)
{
    //if the dialgram items change is about the item positional change
    //then we need to notify the component items about the changes
    if (change == QGraphicsItem::ItemPositionChange)
    {
        //all the arrows connected to
        //it need to update its position
 
        foreach (Arrow *arrow, arrows)
        {
            arrow->updatePosition();
        }
    }
 
    return value;
}

QVariant DiagramItem::itemChange(GraphicsItemChange change,
                     const QVariant &value)
{
    //if the dialgram items change is about the item positional change
    //then we need to notify the component items about the changes
    if (change == QGraphicsItem::ItemPositionChange)
    {
        //all the arrows connected to
        //it need to update its position
       
        foreach (Arrow *arrow, arrows)
        {
            arrow->updatePosition();
        }
    }

    return value;
}

To copy to clipboard, switch view to plain text mode 

But i am having the same problem - The arrow does not attach to the intersection point. I am attaching all the sources now. I hope that it will be more

illustrative to solve the issue. If i change the code in diagramitem.cpp as follows the problem is solved here as i mentioned in my previous post.

QGraphicsEllipseItem item(-100,-50,200,100);
                myPolygon = item.shape().toFillPolygon();

QGraphicsEllipseItem item(-100,-50,200,100);
                myPolygon = item.shape().toFillPolygon();

To copy to clipboard, switch view to plain text mode 

But in my original project, the graphical items bounding rectangles starting point is from (0,0,width,height). And i do want to implement this intersection test there as well. Please make some time to go through this and help me solve the problem.



Regards
Sajjad

[wysota]

I meant prepareGeometryChange() for your RootGraphicsItem class. Its bounding rect depends on the size of the text (and possibly some other things) yet you never inform graphics view the size of the item might have changed.

[sajis997]

In the last post i missed to include the source that i did some changes. Sorry.


arrow.cppdiagramitem.cppdiagramitem.harrow.h



Please go through it and suggest me to get it fixed.


Regards
Sajjad

---

Added after 9 minutes:

I meant prepareGeometryChange() for your RootGraphicsItem class. Its bounding rect depends on the size of the text (and possibly some other things) yet you never inform graphics view the size of the item might have changed.

Yes it is true that the size of the bounding rectangle is based on the size of the text and predefined text spacing. And i am defining the size of the item inside the boundingRect() function, do i need to call the prepareGeometryChange() inside the boundingRect() and even if i do i get the following error:

RootGraphicsItem.cpp:671: error: passing ‘const RootGraphicsItem’ as ‘this’ argument of ‘void QGraphicsItem::prepareGeometryChange()’ discards qualifiers

RootGraphicsItem.cpp:671: error: passing ‘const RootGraphicsItem’ as ‘this’ argument of ‘void QGraphicsItem::prepareGeometryChange()’ discards qualifiers

To copy to clipboard, switch view to plain text mode 

Regards
Sajjad

[sajis997]

I call the prepareGeometryhange() inside the definition of boundary rectangle but it gives me the error as follows:

RootGraphicsItem.cpp:668: error: passing ‘const RootGraphicsItem’ as ‘this’ argument of ‘void QGraphicsItem::prepareGeometryChange()’ discards qualifiers

RootGraphicsItem.cpp:668: error: passing ‘const RootGraphicsItem’ as ‘this’ argument of ‘void QGraphicsItem::prepareGeometryChange()’ discards qualifiers

To copy to clipboard, switch view to plain text mode 

I guess that it is not the right place to call the prepareGeometryChange(), as i am doing it now:

QRectF RootGraphicsItem::boundingRect() const
{
 
    prepareGeometryChange();
 
    //!get the bounding rectangle of the enclosing text
    QRectF rect = m_textGraphicsItem->boundingRect();
 
    //the following condition decides if the boundary needs
    //to be changed
    if( rect.width() < m_drawRectMinimumWidth )
        rect.setWidth(m_drawRectMinimumWidth);
 
    if(rect.height() < m_drawRectMinimumHeight)
        rect.setHeight(m_drawRectMinimumHeight);
 
    //set the rectangle width with the text item spacing
    rect.setWidth(rect.width() + getTextItemSpacing() );
 
    //and return the rectangle
    return rect;
}

QRectF RootGraphicsItem::boundingRect() const
{

    prepareGeometryChange();

    //!get the bounding rectangle of the enclosing text
    QRectF rect = m_textGraphicsItem->boundingRect();

    //the following condition decides if the boundary needs
    //to be changed
    if( rect.width() < m_drawRectMinimumWidth )
        rect.setWidth(m_drawRectMinimumWidth);

    if(rect.height() < m_drawRectMinimumHeight)
        rect.setHeight(m_drawRectMinimumHeight);

    //set the rectangle width with the text item spacing
    rect.setWidth(rect.width() + getTextItemSpacing() );

    //and return the rectangle
    return rect;
}

To copy to clipboard, switch view to plain text mode 

According to the manual it should be called once the set bounding rectangle is about to change. I am deciding the size before setting the bounding rectangle.

Any more suggestion ?

[wysota]

Calling that method from within boundingRect() wouldn't make sense. You need to call it when the actual change occurs (like when the text or spacing is changed).

[sajis997]

Hi

This is what i think as well. I am changing the text later after defining the bounding rectangle.


Could you please figure out any another reason that the arrow item and elliptical item intersection test is not functioning ?



Regards
Sajjad

[wysota]

Could you please figure out any another reason that the arrow item and elliptical item intersection test is not functioning ?

What's wrong with the original reason? Don't you think we should test one thing at a time?

[sajis997]

This post has been merged into this thread!

Hello forum,

With the help of the forum i have made some changes to the diagram scene example where i added elliptical item as follows:

case Ellipse:
            {
                //declare an ellipse with the bounding rectange
                QGraphicsEllipseItem item(-100,-50,200,100);
                myPolygon = item.shape().toFillPolygon();
            }
            break;

case Ellipse:
            {
                //declare an ellipse with the bounding rectange
                QGraphicsEllipseItem item(-100,-50,200,100);
                myPolygon = item.shape().toFillPolygon();
            }
            break;

To copy to clipboard, switch view to plain text mode 

I wanted to use the very same idea to implement the intersection test in another project. In my project i have the graphical item has the following definitions for bounding rectange and shape functions:

QRectF RootGraphicsItem::boundingRect() const
{
    //!get the bounding rectangle of the enclosing text
    QRectF rect = m_textGraphicsItem->boundingRect();
 
 
    //the following condition decides if the boundary needs
    //to be changed
    if( rect.width() < m_drawRectMinimumWidth )
        rect.setWidth(m_drawRectMinimumWidth);
 
    if(rect.height() < m_drawRectMinimumHeight)
        rect.setHeight(m_drawRectMinimumHeight);
 
    //set the rectangle width with the text item spacing
    rect.setWidth(rect.width() + getTextItemSpacing() );
 
    //and return the rectangle
    return rect;
}
 
//the following function is for better collission detection
QPainterPath RootGraphicsItem::shape() const
{
    QRectF ellipse_rect = boundingRect();
 
    QPainterPath path;
 
    path.addEllipse(ellipse_rect);
 
    return path;
}

QRectF RootGraphicsItem::boundingRect() const
{
    //!get the bounding rectangle of the enclosing text
    QRectF rect = m_textGraphicsItem->boundingRect();


    //the following condition decides if the boundary needs
    //to be changed
    if( rect.width() < m_drawRectMinimumWidth )
        rect.setWidth(m_drawRectMinimumWidth);

    if(rect.height() < m_drawRectMinimumHeight)
        rect.setHeight(m_drawRectMinimumHeight);

    //set the rectangle width with the text item spacing
    rect.setWidth(rect.width() + getTextItemSpacing() );

    //and return the rectangle
    return rect;
}

//the following function is for better collission detection
QPainterPath RootGraphicsItem::shape() const
{
    QRectF ellipse_rect = boundingRect();

    QPainterPath path;

    path.addEllipse(ellipse_rect);

    return path;
}

To copy to clipboard, switch view to plain text mode 

But if i try to add the arrow item to my root graphics item using the line intersection example i get erratic behavior - the arrow is not attached to the item and disappears once you move the item inside the scene. You can also recreate the problem in the diagram scene example if you do the following changes

case Ellipse:
            {
                //declare an ellipse with the bounding rectange
//                QGraphicsEllipseItem item(-100,-50,200,100);
                QGraphicsEllipseItem item(0,0,200,100);
                myPolygon = item.shape().toFillPolygon();
            }
            break;

case Ellipse:
            {
                //declare an ellipse with the bounding rectange
//                QGraphicsEllipseItem item(-100,-50,200,100);
                QGraphicsEllipseItem item(0,0,200,100);
                myPolygon = item.shape().toFillPolygon();
            }
            break;

To copy to clipboard, switch view to plain text mode 

You also need to add the following line in the mainwindow.cpp and some additional changes in diagramitem.h:

layout->addWidget(createCellWidget(tr("Ellipse"),DiagramItem::Ellipse),2,0);

layout->addWidget(createCellWidget(tr("Ellipse"),DiagramItem::Ellipse),2,0);

To copy to clipboard, switch view to plain text mode 

enum DiagramType
    {
       Step,
       Conditional,
       StartEnd,
       Io ,
       Ellipse
    };

enum DiagramType
    {
       Step,
       Conditional,
       StartEnd,
       Io ,
       Ellipse
    };

To copy to clipboard, switch view to plain text mode 

I think i need to adjust some transformational issue here. But i have not found any good reference to it. The manual is not illustrative enough.

Any reference to this issue will be very helpful



Regards
Sajjad