Reduce file size - simplify code

[Bloody_geek]

Hello everyone,
I'm learning a python right now and just experimented a bit by writing a small program.
My problem is that the outcomming file size is pretty huge. It's ~150MB which I dont think is an appropriate size for a program of this kind. It's just a simple calculator for plotting circles.
Can someone give me a hint how to minimize the size? Maybe code or compiling changes will do it, but I'm not sure how to do it since I'm a beginner.
I know the code ist "complicated" and can be simplified for sure. But I wanted to keep it easy and didn't want to implement functions I've not learned yet.
Some advice for some cosmetic changes to reduce file and code size would be great!
greetings

import tkinter
from tkinter import *
from tkinter import messagebox
import math
from mpmath import *
from sympy import *
import numpy as np
import matplotlib
import matplotlib.pyplot as plt
matplotlib.use("TkAgg")
from matplotlib.figure import Figure
from matplotlib.backends.backend_tkagg import (FigureCanvasTkAgg, NavigationToolbar2Tk)

    ###Berechnung der Werte bei gegebenem/gegebener Kreisbogen/Kreissehne
    ###bzw. Kreissektor/Kreissehne mit der Newton Methode

def newton(f,Df,x0,epsilon,max_iter):
   xn = x0
   for n in range(0,max_iter):
        fxn = f(xn)
        if abs(fxn) < epsilon:
            return xn
        Dfxn = Df(xn)
        if Dfxn == 0:
            return -1
        xn = xn - fxn/Dfxn
   return -1
   
def deploy(f,x0,epsilon,max_iter,acc):
     xn = x0
     for n in range(0,max_iter):
            fxn = f(xn)
            if abs(fxn) < epsilon:
                return xn
            xn = xn + (acc)
     return -1

###Funktion für neue Eingabe

def delete_entry_fields():
 e1.delete(0,END)
 e2.delete(0,END)
 e3.delete(0,END)
 e4.delete(0,END)
 e5.delete(0,END)
 e1.insert(10,"0")
 e2.insert(10,"0")
 e3.insert(10,"0")
 e4.insert(10,"0")
 e5.insert(10,"0")

###Berechnungen der Werte

def calc():
 n1 = float(e1.get())
 n2 = float(e2.get())
 n3 = float(e3.get())
 n4 = float(e4.get())
 n5 = float(e5.get())
 if n1 > 0 and n2 > 0 and n3 == 0 and n4 == 0 and n5 == 0 and n2 <= 360:
   e3.delete(0,END)
   e4.delete(0,END)
   e5.delete(0,END)
   r1 = n1 * math.pi * n2 / 180
   e3.insert(10,round(r1,3))
   r2 = n1 * 2 * math.sin(n2*math.pi/180/2)
   e4.insert(10,round(r2,3))
   r3 = n1 ** 2 * math.pi * n2/360
   e5.insert(10,round(r3,3))
 elif n2 > 0 and n3 > 0 and n1 == 0 and n4 == 0 and n5 == 0 and n2 <= 360:
   e1.delete(0,END)
   e4.delete(0,END)
   e5.delete(0,END)
   r1 = n3 / (n2 / 180 * math.pi)
   e1.insert(10,round(r1,3))
   r2 = r1 * 2 * math.sin(n2*math.pi/180/2)
   e4.insert(10,round(r2,3))
   r3 = r1 ** 2 * math.pi * n2/360
   e5.insert(10,round(r3,3))
 elif n3 > 0 and n4 > 0 and n3 > n4 and n1 == 0 and n2 == 0 and n5 == 0:
   e1.delete(0,END)                  
   e2.delete(0,END)                      
   e5.delete(0,END)
   m1 = n3
   m2 = n4
   f = lambda x: 2*x*math.sin((m1/2)/x)-m2
   Df = lambda x: 2*math.sin((m1/2)/x)-((n1*math.cos((m1/2)/x))/(x))
   approxi = newton(f,Df,4,1e-3,100)
   e1.insert(10,round(abs(approxi),3))
   r1 = n3 / approxi * 57.2958
   e2.insert(10,round(abs(r1),3))
   r3 = approxi ** 2 * math.pi * r1/360
   e5.insert(10,round(abs(r3),3))
 elif n4 > 0 and n5 > 0 and n1 == 0 and n2 == 0 and n3 == 0:
   e1.delete(0,END)                  
   e2.delete(0,END)                      
   e3.delete(0,END)
   f = lambda x: (n4/math.sin(x/2)/2)**2*math.pi*x/(2*math.pi)-n5
   if n4 < 1 and n5 < 1:
    approxi = deploy(f,0.001,0.01,90000,0.0001)
   if n4 > n5 and n4 <= 2.546:
    approxi = deploy(f,0.0001,0.0001,900000,0.0001)
   if n4 >= 1 and n4 >= 1 and n4 < 60 and n5 < 100:
    approxi = deploy(f,0.0001,1,90000,0.001)
   if n4 >= 60 or n5 >= 100:
    approxi = deploy(f,0.0001,100,90000,0.001)
   if approxi == -1:
    messagebox.showerror("Ungültige Eingabe", "Für die gegebenen Werte ist keine Berechnung möglich.")
   r1 = approxi * 180 / math.pi
   e2.insert(10,round(r1,3))
   r2 = n4 / (2*sin(r1*math.pi/180/2))
   e1.insert(10,round(r2,3))
   r3 = r2 * 2 * math.pi * r1/360
   e3.insert(10,round(r3,3))
 elif n3 > 0 and n5 > 0 and n1 == 0 and n2 == 0 and n4 == 0 and n2 <= 360:
   e1.delete(0,END)
   e2.delete(0,END)
   e4.delete(0,END)
   r1 = 360 * (n3 ** 2) / (4 * math.pi * n5)
   e2.insert(10,round(r1,3))
   r2 = 360 * n3 / (2 * math.pi * r1)
   e1.insert(10,round(r2,3))
   r3 = r2 * 2 * math.sin(r1*math.pi/180/2)
   e4.insert(10,round(r3,3))
 elif n1 > 0 and n3 > 0 and n2 == 0 and n4 == 0 and n5 == 0 and 2 * n1 * math.pi >= n3:
   e2.delete(0,END)
   e4.delete(0,END)
   e5.delete(0,END)
   r1 = n3 / n1 * (180 / math.pi)
   e2.insert(10,round(r1,3))
   r2 = n1 * 2 * math.sin(r1*math.pi/180/2)
   e4.insert(10,round(r2,3))
   r3 = n1 ** 2 * math.pi * r1/360
   e5.insert(10,round(r3,3))
 elif n1 > 0 and n4 > 0 and n2 == 0 and n3 == 0 and n5 == 0 and n1 * 2 >= n4:
   e2.delete(0,END)
   e3.delete(0,END)
   e5.delete(0,END)
   a = (2 * n1 - math.sqrt(4 * (n1 ** 2) - (n4 ** 2))) / 2
   r1 = 2 * math.asin(n4 / (2 * n1)) * (180 / math.pi)
   e2.insert(10,round(r1,3))
   r2 = n1 * math.pi * r1 / 180
   e3.insert(10,round(r2,3))
   r3 = n1 ** 2 * math.pi * r1/360
   e5.insert(10,round(r3,3))
 elif n1 > 0 and n5 > 0 and n2 == 0 and n3 == 0 and n4 == 0 and math.pi * n1 ** 2 >= n5:
   e2.delete(0,END)
   e3.delete(0,END)
   e4.delete(0,END)
   r1 = n5 / (n1 ** 2) / math.pi * 360
   e2.insert(10,round(r1,3))
   r2 = 2 * n1 * math.pi * r1 / 360
   e3.insert(10,round(r2,3))
   r3 = n1 * 2 * math.sin(r1*math.pi/180/2)
   e4.insert(10,round(r3,3))
 elif n2 > 0 and n4 > 0 and n1 == 0 and n3 == 0 and n5 == 0 and n2 <= 360:
   e1.delete(0,END)
   e3.delete(0,END)
   e5.delete(0,END)
   r1 = n4 / (2*sin(n2*math.pi/180/2))
   e1.insert(10,round(r1,3))
   r2 = r1 * math.pi * n2 / 180
   e3.insert(10,round(r2,3))
   r3 = r1 ** 2 * math.pi * n2/360
   e5.insert(10,round(r3,3))
 elif n2 > 0 and n5 > 0 and n1 == 0 and n3 == 0 and n4 == 0 and n2 <= 360:
   e1.delete(0,END)
   e3.delete(0,END)
   e4.delete(0,END)
   r1 = math.sqrt(n5 / math.pi / n2 * 360)
   e1.insert(10,round(r1,3))
   r2 = r1 * math.pi * n2 / 180
   e3.insert(10,round(r2,3))
   r3 = r1 * 2 * math.sin(n2*math.pi/180/2)
   e4.insert(10,round(r3,3))
 elif n4 > n5 and n4 > 2.546:
    messagebox.showerror("Ungültige Eingabe", "Die Kreissektorfläche kann die Sekantenlänge nicht derart unterschreiten.")
 elif n1 * 2 < n4:
    messagebox.showerror("Ungültige Eingabe", "Die Kreissehne überschreitet den Kreisdurchmesser.")
 elif n1 ** 2 * math.pi < n5:
    messagebox.showerror("Ungültige Eingabe", "Der Flächeninhalt des Kreisauschnitts überschreitet den des gesamten Kreises.")
 elif n1 > 0 and n3 > 0 and n2 == 0 and n4 == 0 and n5 == 0 and 2 * n1 * math.pi < n3:
    messagebox.showerror("Ungültige Eingabe", "Das angegebene Bogenmaß überschreitet den Kreisumfang.")
 elif n2 > 0 and n5 > 0 and n1 == 0 and n3 == 0 and n4 == 0 and n2 > 360:
    messagebox.showerror("Ungültige Eingabe", "Der maximale Sektorwinkel kann 360° nicht überschreiten.")
 elif n2 > 0 and n4 > 0 and n1 == 0 and n3 == 0 and n5 == 0 and n2 > 360: 
    messagebox.showerror("Ungültige Eingabe", "Der maximale Sektorwinkel kann 360° nicht überschreiten.")
 elif n2 > 0 and n3 > 0 and n1 == 0 and n4 == 0 and n5 == 0 and n2 > 360:
    messagebox.showerror("Ungültige Eingabe", "Der maximale Sektorwinkel kann 360° nicht überschreiten.")
 elif n1 > 0 and n2 > 0 and n3 == 0 and n4 == 0 and n5 == 0 and n2 > 360:
    messagebox.showerror("Ungültige Eingabe", "Der maximale Sektorwinkel kann 360° nicht überschreiten.")
 elif n1 > 0 and n2 > 0 and n3 > 0 and n4 > 0 and n5 > 0 or\
    n1 > 0 and n2 > 0 and n3 > 0 and n4 > 0 or\
    n1 > 0 and n2 > 0 and n3 > 0 and n5 > 0 or\
    n1 > 0 and n2 > 0 and n4 > 0 and n5 > 0 or\
    n1 > 0 and n3 > 0 and n4 > 0 and n5 > 0 or\
    n1 > 0 and n2 > 0 and n3 > 0 or\
    n1 > 0 and n2 > 0 and n4 > 0 or\
    n1 > 0 and n2 > 0 and n4 > 0 or\
    n1 > 0 and n2 > 0 and n5 > 0 or\
    n1 > 0 and n3 > 0 and n4 > 0 or\
    n1 > 0 and n3 > 0 and n5 > 0 or\
    n2 > 0 and n3 > 0 and n4 > 0 or\
    n3 > 0 and n4 > 0 and n5 > 0 or\
    n2 > 0 and n4 > 0 and n5 > 0 or\
    n2 > 0 and n3 > 0 and n5 > 0 or\
    n1 > 0 and n4 > 0 and n5 > 0 or\
    n1 == 0 and n2 == 0 and n3 == 0 and n4 == 0 and n5 == 0 or\
    n1 == 0 and n2 == 0 and n3 == 0 and n4 == 0 or\
    n1 == 0 and n2 == 0 and n3 == 0 and n5 == 0 or\
    n1 == 0 and n2 == 0 and n5 == 0 and n4 == 0 or\
    n1 == 0 and n5 == 0 and n3 == 0 and n4 == 0 or\
    n5 == 0 and n2 == 0 and n3 == 0 and n4 == 0:
     messagebox.showerror("Ungültige Eingabe", "Bitte exakt zwei Eingaben tätigen.")
 else:
    messagebox.showerror("Ungültige Eingabe", "Achten Sie auf die in der Beschreibung angegebene Syntax.")


    ###Darstellung der Grafik
     
 rest = math.pi*(float(e1.get())**2)

 labels = 'Kuchenstück', 'Rest'
 sizes = [float(e5.get()), rest-float(e5.get())]
    
 explode = (0.1, 0)

 fig, ax1 = plt.subplots()
 ax1.pie(sizes, explode=explode, labels=labels, shadow=True, startangle=90)
 ax1.axis('equal') 
 canvas1 = FigureCanvasTkAgg(fig, master=root)  # A tk.DrawingArea.
 canvas1.draw()
 canvas1.get_tk_widget().grid(columnspan=10, rowspan=10, row=0, column=7, ipadx=0, pady=30)
 canvas1.pack

 

    ###HauptfensterGUI


root = tkinter.Tk()
display1 = Canvas()
display1.config(width=1300, height=600)
display1.grid(row=0, rowspan=300, columnspan=100)
display1.create_rectangle(0, 0, 3000, 3000, fill="azure")
display3 = Canvas()
display3.config(width=700, height=600)
display3.grid(row=0, column=6, rowspan=13, columnspan=12)
display3.create_rectangle(0, 0, 2000, 2000, fill="steel blue")
display2 = Canvas()
display2.config(width=650, height=500)
display2.grid(row=0, column=7, rowspan=10, columnspan=10, ipadx=0)
display2.create_rectangle(0, 0, 2000, 2000, fill="white")
root.title("Kuchenstück-Rechner 2019")
root.geometry("1240x566")
root.iconbitmap('cake1.ico')
root.resizable(width=False, height=False)
img = PhotoImage(file='cakebg.png')
panel = Label(root, bg="azure", image = img)
panel.grid(row=2, column=1, rowspan=9, columnspan=4)

    ###Labels

Label(root, text="Radius", bg="azure").grid(row=0, pady=3)
Label(root, text="Bogenwinkel", bg="azure").grid(row=1, pady=3)
Label(root, text="Bogenlänge", bg="azure").grid(row=2, pady=3)
Label(root, text="Sekantenlänge", bg="azure").grid(row=3, pady=3)
Label(root, text="Flächeninhalt", bg="azure").grid(row=4, pady=3)
Label(root, text="Bitte geben Sie zwei der fünf Werte an, um die übrigen zu berechnen.\n Achten Sie dabei auf die folgende Syntax: ZAHL.NACHKOMMASTELLE.\n Die Werte werden auf die dritte Nachkommastelle gerundet.", bg="steel blue").grid(row=9, column=0, columnspan=3, padx=20, pady=20, ipady=20, ipadx=20)

    ###Entries

e1 = Entry(root, width=9)
e2 = Entry(root, width=9)
e3 = Entry(root, width=9)
e4 = Entry(root, width=9)
e5 = Entry(root, width=9)
e1.insert(10,"0")
e2.insert(10,"0")
e3.insert(10,"0")
e4.insert(10,"0")
e5.insert(10,"0")
e1.grid(row=0, column=1)
e2.grid(row=1, column=1)
e3.grid(row=2, column=1)
e4.grid(row=3, column=1)
e5.grid(row=4, column=1)

    ###Buttons

b1 = Button(root, text='Beenden', bg="azure2", command=root.quit).grid(row=7, column=0, columnspan=2, ipadx=56)
b2 = Button(root, text='Berechnen', bg="azure2", command=calc).grid(row=5, column=0, columnspan=2, ipadx=50)
b3 = Button(root, text='Neue Eingabe', bg="azure2", command=delete_entry_fields).grid(row=6, column=0, columnspan=2, ipadx=38)


root.mainloop()

[ichabod801]

The first thing I see is this function:

def delete_entry_fields():
 e1.delete(0,END)
 e2.delete(0,END)
 e3.delete(0,END)
 e4.delete(0,END)
 e5.delete(0,END)
 e1.insert(10,"0")
 e2.insert(10,"0")
 e3.insert(10,"0")
 e4.insert(10,"0")
 e5.insert(10,"0")

Put all of the entry fields into one list. Then you can do this with a loop:

for entry_field in entry_fields:
    entry_field.delete(0, END)
    entry_field.insret(10, "0")

This has applications throughout your code. If you are numbering things x1 to x5, you should just put them all in a list.

In the calc function, you have lots of conditionals like this:

if n1 > 0 and n2 > 0 and n3 == 0 and n4 == 0 and n5 == 0 and n2 <= 360:

Again, n1 to n5 should be in a list. You should use that list to generalize that conditional. Just have one of them, which picks which items are to be processed in which way, and then after the conditional do the processing.

The conditional on line 192 is insane. It looks like you are trying to check that at least three of n1-n5 are greater than 0 or at least four are equal to 0. Again, the ns should be in a list, let's call it all_ns:

over_zero = len([n for n in all_ns if n > 0])
zero = len([n for n in all_ns if n == 0])
if over_zero >= 3 or zero >= 4:
    ...

That should replace the entire conditional starting on line 192.

All that said, this is a GUI and you are not using object oriented programming (OOP). You really should learn OOP and how classes work. You can start with the classes tutorial link in my signature below.

[Bloody_geek]

Thank you for your quick answer!

I took your advice and reworked the code. It looks a little bit better by now.
I'm learning OOP right now, so I didn't want to use mechanics I'm not familiar with.
Just wanted to experiment with some functions to get a little peak on what is possible to create, I'm not planning to do sloppy work or learn overhasty.
I know that learning a language takes a lot time, and I'm willing to learn it step by step, so I will check out your OOP guide for sure!

Like I said thank you very much for your hints

[joe_momma]

just to add a couple points you can use just one canvas and create many rectangles.
in here

root = tkinter.Tk()
display1 = Canvas()
display1.config(width=1300, height=600)
display1.grid(row=0, rowspan=300, columnspan=100)
display1.create_rectangle(0, 0, 3000, 3000, fill="azure")
display3 = Canvas()
display3.config(width=700, height=600)
display3.grid(row=0, column=6, rowspan=13, columnspan=12)
display3.create_rectangle(0, 0, 2000, 2000, fill="steel blue")
display2 = Canvas()
display2.config(width=650, height=500)
display2.grid(row=0, column=7, rowspan=10, columnspan=10, ipadx=0)
display2.create_rectangle(0, 0, 2000, 2000, fill="white")

when you clear the entries I added a clear the canvas: you won't need the global after
you create a class.

def delete_entry_fields():
 global canvas1
 e1.delete(0,END)
 e2.delete(0,END)
 e3.delete(0,END)
 e4.delete(0,END)
 e5.delete(0,END)
 e1.insert(10,"0")
 e2.insert(10,"0")
 e3.insert(10,"0")
 e4.insert(10,"0")
 e5.insert(10,"0")
 canvas1._tkcanvas.delete('all')
global canvas1
fig, ax1 = plt.subplots()
ax1.pie(sizes, explode=explode, labels=labels, shadow=True, startangle=90)
ax1.axis('equal') 
canvas1 = FigureCanvasTkAgg(fig, master=root)  # A tk.DrawingArea.
canvas1.draw()
canvas1.get_tk_widget().grid(columnspan=10, rowspan=10, row=0, column=7, ipadx=0,pady=30) 
 

finally try and steer the user to just 2 options maybe use radio or checkboxes to only allow 2 active entries that's got to make your if statements a lot easier. I'm not a fan of the single space indent, imo. Best of luck