Merge two pieces of code

JonB

@john_hobbyist
Seems to be much the same code as you started with, re-pasted. I don't understand how you said you had moved from Qt4 to Qt5 with QtGui changes, yet now you have QtGui.QLabel, which as far as I know should not work....

Anyway, I don't know what path your code is or is not following. It's time for you to put your own print() statements in if you want to understand what is going on....

john_hobbyist

@JonB Yes, I have put "prints" inside the code, that is how I figure out on which parts of code the execution goes...

JonB

@john_hobbyist
But you don't tell me, so you want me to guess?

Does it hit the self.selection.show()? Does it hit the move code? What is your usage of Python is instead of == in if self.mode is "drag_lower_right"?

I leave it to you now.

john_hobbyist

@JonB Sorry! I updated the right code! Please see above 2-3 posts..
When I press the mouse button it goes: 56, 57,58, 59, 62, 63, 56, 57,58, 59, 62, 63....
When I move the mouse, it goes 71, 72, 73, 74, 77, 71, 72, 73, 74, 77...
Either I have loaded an image or not!

JonB

@john_hobbyist
I don't know.

 self.lower_right = QtCore.QPoint(event.pos())
self.upper_left = QtCore.QPoint(event.pos())
self.selection.setGeometry(QtCore.QRect(self.upper_left, self.lower_right).normalized())

Are the coordinates of event.pos() correct for setGeometry()? Start by getting some visible rubber band, e.g. put suitable constants in there instead of relying on mouse positions?

SGaist

@john_hobbyist did you try to just get the rubber band stuff working on a minimal widget ?

You can then apply that knowledge to your more complex application.

john_hobbyist

@SGaist I am searching how do this.... and basically what @JonB suggested...

Update: I noticed that the rectangle works but it is constrained inside tool bar...(upper left corner) not inside the area that I load the image...

mrjj

@john_hobbyist
Hi
Progress :) \o/

Check for RubberbandEnhancedLabel (if you use it still ) what parent you assign to it
when you create it.

john_hobbyist

@mrjj You mean this:

label = RubberbandEnhancedLabel()

but I do not use it. I have it as comment from previous code. Basically the whole unused/commented code is this:

    app = QtWidgets.QApplication([])

    screen_pixmap = QScreen.grabWindow (self, win_id, left, top, width, height)
    
    window = QtGui.QWidget()
    
    layout = QtWidgets.QVBoxLayout(window)
    label = RubberbandEnhancedLabel()
    label.setPixmap(screen_pixmap)
    layout.addWidget(label)
    geometry = app.desktop().availableGeometry()
    window.setFixedSize(geometry.width(), geometry.height())

which I do not use (copied from one of the sources above...). Should I use something from here?

mrjj

@john_hobbyist
Hi
So what IS the blue rectangle then ?

Check the parent where you create it.

john_hobbyist

@mrjj Please look in the previous posts. I have posted everything...

mrjj

@john_hobbyist
Yes but its hard to know what you currently have.
So the blue rectangle is a QRubberband ?

So there place where you create it, what is used as the parent.
It kinda seems like the Toolbar as else it would not clip to it.

is self.selection the RubberBand ?

john_hobbyist

@mrjj Yes, you are right! It is used inside

MyMplCanvas

Yes it is rubberband,

self.selection = QtWidgets.QRubberBand(QtWidgets.QRubberBand.Rectangle, self)

mrjj

@john_hobbyist
Hi
Hmm The reason its up in the Toolbar seems not to be be related to parent

class MyMplCanvas(mpl_qt.FigureCanvasQTAgg):
....
self.selection = QtWidgets.QRubberBand(QtWidgets.QRubberBand.Rectangle, self)

I assume here self is MyMplCanvas and as far as we know MyMplCanvas is the class that draw the image.

So the Rubberband works, but cant be dragged outside that little area ?

john_hobbyist

@mrjj "So the Rubberband works, but cant be dragged outside that little area ?"
Yes!

mrjj

@john_hobbyist
Ok. That is bizarre as that sounds like it is inside the toolbar.
and the mouseXXX functions are under the MyMplCanvas ?

I don't get this as then it should not work if it's placed in toolbar as the mouse events go into
MyMplCanvas.

unde this line
self.selection = QtWidgets.QRubberBand(QtWidgets.QRubberBand.Rectangle, self)

is it possible to
print (self) ?

Im not sure if it just shows the address.

john_hobbyist

@mrjj

Both

def mousePressEvent(self, event):

and

def mouseMoveEvent(self, event):

are under

class MyMplCanvas(mpl_qt.FigureCanvasQTAgg):

Yes, the answer in:

print (self)

is:

<__main__.MyMplCanvas object at 0x7f717bb5f700>

mrjj

@john_hobbyist
Hi
Oh that actually worked. the print(self)
so the parent really is MyMplCanvas.

So how does it show the blue rect ?

You click on the Canvas and it appears up there ?

john_hobbyist

@mrjj Yes I click on the large area (where the images are depicted) and I see the result as I depicted in the figure above, BUT it never completes the operation, meaning that as I move it, the rectangle keeps changing on that small area..

mrjj

@john_hobbyist
Ok. since it seems it should be in the Canvas, I would put some print() under

self.selection.setGeometry(QtCore.QRect(self.upper_left, self.lower_right).normalized())

in mouseMove and inspect the values and see
like
print( QtCore.QRect(self.upper_left, self.lower_right).normalized() )

and see what values it sets. Maybe that can give a hint.