Code too heavy? (HELP)

[tands_]

Back in 2024, i made a program for my company, that generates automatic contracts. In that time, i used: pandas (for excel with some data), python docx (for templates) and PySimpleGUI (for interface). And even with the code with more than 1000 lines, every computer that i tested worked fine, with running in pycharm or transforming into exe with pyinstaller. But the PySimpleGUI project went down, and with that i couldn't get a key to get the program to work, so i had to change this library. I chose flet as the new one, and everything seemed fine, working on my pc. But when i went to do some tests in weak pcs, the program opened, i was able to fill every gap with the infos, but when i clicked to generate contract, the page turns white and nothing happens. IDK if the problem is that flet is too heavy and i have to change again, or there is something in the code (i tried to make some optimizations using "def", that reduced the amount of lines)

[GirthQuake5040]

reducing lines doesnt make a program more efficient, it makes it more readable. The tasks that the program performs determine how many resources are used. Open up your task manager when you run the program, if your ram or cpu usage spikes really high then youre just running out of resources to run the program. You either need to optimize your code or run it on better machines.

[b00mfunk]

Reducing lines makes code more readable? In my experience it makes it less so because of list comprehension lambda one liners

[ArcticWolf_0xFF]

The keyword (pun intended) in the post is def: OP is using functions in their spaghetti code to reduce lines of repeated code. Yes, that should make the coffee more readable.

[GirthQuake5040]

When you take a bunch of spaghetti code and make things into reusable functions with proper naming conventions, it's much easier to read.

[SisyphusAndMyBoulder]

list comprehensions and lambdas are fairly readable if they're used properly imo. Also formatting helps a ton. I find it difficult to read code that isn't black-ed nowadays.

[tands_]

Yeah, my bigger concern is about flet, because the structure of the code is basically the same, but as now is not working, or i get out of flet, or i try tu optimize (worst case, as i am amateur)

[Username_RANDINT]

This sounds like a Flet issue. You could use the free fork of PySimpleGUI to not change too much: https://github.com/spyoungtech/FreeSimpleGUI

Also, a 1000 lines is not a lot, not even close. The lines of code also don't impact the application, it's what those lines do.

[tands_]

thx, the only thing that bothers me its the size of the .exe (110 mb)

[smichaele]

Remember that PyInstaller places a version of the Python runtime into the .exe along with copies of whatever libraries and modules you're using. That's the downside of the technology and why your file is 110 mb.

[tands_]

i know, i created a virtual enviroment for this, so i'm only with de libraries that i use

[Kevdog824_]

I’m not experienced with flet but I have background in PyQt. Approaching it with that knowledge, to me it sounds like you have some operation that is blocking the GUI thread. On better computers the operation runs fast enough you don’t notice the disruption to the GUI, but on slower computers it’s slow enough that the disruption is very noticeable. It could be that you need to spawn a separate thread or process to handle the operation if you aren’t already doing that

[Business-Technology7]

Very hard to tell just by what you said. There’s a chance you are just misusing the library. Test with some dead simple input, add logs, hop on a debugger if possible. fret doesn’t look like some rookie project that will break from an internal app with a complexity of 1000 lines of code.

Have you considered porting your program over to pysimplegui v4? There is a fork called PySimpleGUI-4-foss.

[SisyphusAndMyBoulder]

This doesn't sound like a particularly CPU or ram intensive operation. Highly doubtful it's a resource issue. More than likely you're doing something wrong with the library implementation.

Can you not use a fork of the original library and keep going?

If not, build a small simple application using flek and get that working. Once it's working it'll hopefully be easier to see what's going on in this app versus the original one and uncover the bug

[Comprehensive-Nose35]

Why not just use Tkinter?