Skip to main content
3 votes

Calculation of liquid propellant nozzles

I don't really understand any of this, so I can only speak as a programmer, but the code looks good and fairly well structured. Naming conventions make sense in plain English even to an outsider. If ...
Kate's user avatar
  • 8,303
2 votes

Solve a system of linear equations with Gauss algorithm

Not only are names like a1 and b2 non-descriptive, but they all of these objects should probably be in lists or dictionaries ...
Chris's user avatar
  • 4,329
2 votes

Solve a system of linear equations with Gauss algorithm

UX When I run the code, the GUI title has the acronym "LGS". It would be helpful to spell out the acronym somewhere in the window. Also, I do not know what I should enter into each field. ...
toolic's user avatar
  • 15.7k
2 votes

Google drive backup robot

Tools You could run code development tools to automatically find some style issues with your code. ruff recommends splitting imports onto separate lines: ...
toolic's user avatar
  • 15.7k
2 votes

Simple Neural Network from scratch using NumPy

Here are some general coding style suggestions. DRY The following expression is repeated in 2 functions: 1 / (1 + np.exp(-x)) Call one function from the other to ...
toolic's user avatar
  • 15.7k
2 votes
Accepted

Calculation of liquid propellant nozzles

Overview The code layout is good, and you used meaningful names for classes, functions and variables. It seems like the code is already structured for future enhancements. Long lines There are a ...
toolic's user avatar
  • 15.7k
2 votes

Yet Another Python Weather Visualizer using Open-Meteo

Some minor points from me. Intuitive function names Function names should be descriptive, so c_to_f should be named ...
Kate's user avatar
  • 8,303
2 votes

Timer with input, sound and flashing

Portability When I copy the code out of the question and try to run it, I get syntax errors due to indentation. Perhaps your indentation mixes single spaces with tab characters on the same line, and ...
toolic's user avatar
  • 15.7k
1 vote

Web scraping data from website using Selenium

Portability When I run the code, I get a syntax error on lines like this: ...
toolic's user avatar
  • 15.7k
1 vote

Calculation of Correlation Time

Here are some general coding style suggestions. Unused code This variable is set but never used: start_time = time.time() It can be deleted, which means this line ...
toolic's user avatar
  • 15.7k
1 vote

Snake game in pygame with varying velocity

Imports The import lines should precede all executable code; move the init call: ...
toolic's user avatar
  • 15.7k
1 vote

Solve a system of linear equations with Gauss algorithm

I somewhat doubt that the original poster is still around, but for posterity: 700 lines is too long for such a program. It's too hard-coded (a, b, ...), and not flexible enough - it only supports up ...
Reinderien's user avatar
  • 71.1k

Only top scored, non community-wiki answers of a minimum length are eligible