You need to put something into the sizer even if there are no images.
I haven't edited your code but uprated a previous answer instead, because your code is mixing positional placement and sizers, which is bound to end up giving you trouble.
Below is sample code for a video to be displayed on the left and on the middle and on the right are 2 separate scrolling panels, monitoring 2 separate directories.
I hope it helps.
import wx
import wx.lib.scrolledpanel
import glob
import wx.media
class windowclass(wx.Frame):
def __init__(self, *args, **kwargs):
super(windowclass, self).__init__(*args, **kwargs)
self.panel1 = wx.Panel(self, -1, size=(350,550))
self.panel1.SetBackgroundColour('#FDDF99')
self.h1 = wx.StaticText(self.panel1, -1, "Video")
self.Media = wx.media.MediaCtrl()
self.Media.Create(self.panel1, style=wx.SIMPLE_BORDER, szBackend="")
self.slider = wx.Slider(self.panel1, -1, 0, 0, 10)
self.slider.SetMinSize((340, -1))
self.Media.Load("/home/rolf/BBB.ogv")
self.Media.SetInitialSize((340,300))
self.panel2 = wx.lib.scrolledpanel.ScrolledPanel(self,-1, size=(350,550))
self.h2 = wx.StaticText(self.panel2, -1, "Known")
self.panel2.SetupScrolling()
self.panel2.SetBackgroundColour('#FFFFFF')
self.panel2.SetMinSize((350,550))
self.panel3 = wx.lib.scrolledpanel.ScrolledPanel(self,-1, size=(350,550))
self.h3 = wx.StaticText(self.panel3, -1, "Unknown")
self.panel3.SetupScrolling()
self.panel3.SetBackgroundColour('#FFFFFF')
self.panel3.SetMinSize((350,550))
self.timer = wx.Timer(self)
self.Bind(wx.EVT_TIMER, self.basicgui, self.timer)
self.vtimer = wx.Timer(self)
self.Bind(wx.EVT_TIMER, self.video, self.vtimer)
self.dict1 = {}
self.dict2 = {}
self.sizer1 = wx.BoxSizer( wx.VERTICAL )
self.sizer1.Add(self.h1,0,wx.ALIGN_CENTER)
self.sizer1.Add(self.Media)
self.sizer1.Add(self.slider)
self.sizer2 = wx.BoxSizer( wx.VERTICAL )
self.sizer2.Add(self.h2,0,wx.ALIGN_CENTER)
self.sizer3 = wx.BoxSizer( wx.VERTICAL )
self.sizer3.Add(self.h3,0,wx.ALIGN_CENTER)
self.panel1.SetSizer(self.sizer1)
self.panel2.SetSizer(self.sizer2)
self.panel3.SetSizer(self.sizer3)
self.mainsizer = wx.BoxSizer( wx.HORIZONTAL )
self.mainsizer.Add(self.panel1,0,wx.EXPAND)
self.mainsizer.Add(self.panel2,0,wx.EXPAND)
self.mainsizer.Add(self.panel3,0,wx.EXPAND)
self.SetSizer(self.mainsizer)
self.Bind(wx.EVT_CLOSE, self.Stop)
self.timer.Start(3000)
self.vtimer.Start(1000)
self.Media.Play()
self.slider.SetRange(0, self.Media.Length())
self.Show()
def Stop(self, event):
self.Media.Stop()
self.Destroy()
def video(self,event):
offset = self.Media.Tell()
self.slider.SetValue(offset)
def basicgui(self,event):
self.GetJpgList("./image", self.dict1)
image_cnt = len(self.dict1)
items = self.sizer2.GetChildren()
#if the image count is different perform processing
update = False
if len(items)-1 != image_cnt:
#Clear sizer of it's items
self.sizer2.Clear(delete_windows=True)
h2 = wx.StaticText(self.panel2, -1, "Known")
self.sizer2.Add(h2,0,wx.ALIGN_CENTER)
# add images to the sizer
if image_cnt != 0:
for item, bitmap in self.dict1.items():
control = wx.StaticBitmap(self.panel2, -1, bitmap)
self.sizer2.Add( control, 0, wx.CENTER|wx.ALL, 5 )
else:
control = wx.StaticBitmap(self.panel2, -1, wx.NullBitmap)
self.sizer2.Add( control, 0, wx.CENTER|wx.ALL, 5 )
#reset scrolling
self.panel2.SetupScrolling(scrollToTop=False)
update = True
self.GetJpgList("./image1", self.dict2)
image_cnt = len(self.dict2)
items = self.sizer3.GetChildren()
#if the image count is different perform processing
if len(items)-1 != image_cnt:
#Clear sizer of it's items
self.sizer3.Clear(delete_windows=True)
h3 = wx.StaticText(self.panel3, -1, "Unknown")
self.sizer3.Add(h3,0,wx.ALIGN_CENTER)
# add images to the sizer
if image_cnt != 0:
for item, bitmap in self.dict2.items():
control = wx.StaticBitmap(self.panel3, -1, bitmap)
self.sizer3.Add( control, 0, wx.CENTER|wx.ALL, 5 )
else:
control = wx.StaticBitmap(self.panel2, -1, wx.NullBitmap)
self.sizer2.Add( control, 0, wx.CENTER|wx.ALL, 5 )
#reset scrolling
self.panel3.SetupScrolling(scrollToTop=False)
update = True
if update == True:
self.Layout()
self.Show()
def GetJpgList(self, the_dir, the_dict):
jpgs = glob.glob(the_dir+"/*.jpg")
#Build a dictionary of the images
#this way we only have to create them once
for i in jpgs:
#if image already in dict bail
if i in the_dict:
continue
print("adding", i)
image = wx.Image(i)
image = image.Scale(300, 200, wx.IMAGE_QUALITY_HIGH)
bitmap = wx.Bitmap(image)
the_dict[i] = bitmap
#make a list of any deleted images
del_list = []
for i in the_dict:
if i not in jpgs:
del_list.append(i)
#remove deleted images from the dictionary
for i in del_list:
the_dict.pop(i)
print("deleting",i)
return
def main():
app = wx.App()
windowclass(None)
app.MainLoop()
main()
it works but it is refreshing panel continuously and i don't want that continuous refresh. what should i do.
My code or yours? If it's mine, put the video update in another
Timerand only perform theLayout()andShow()if updating the sizers.My mistake! I added in the "known", "unknown" texts into the sizers, so the test for the number of items in the sizer vs the number of jpgs, was always out by 1. See my edited code above.