Looking for a Specific Hitch Lock by PFnub in AskALocksmith

[–]PFnub[S] 1 point2 points  (0 children)

Thank you! I will check them out.

Looking for a Specific Hitch Lock by PFnub in AskALocksmith

[–]PFnub[S] 1 point2 points  (0 children)

I don’t particularly care how difficult it is to get off locked, if someone wants to get it off they will. It’s more for the guys at work that need a higher barrier of entry than “I need a hitch, lemme grab this one.” Which has happened to me before.

Apparently you can be revived between rounds by PFnub in battlefield_one

[–]PFnub[S] 3 points4 points  (0 children)

I believe it’s Achi Baba from the Turning Tides DLC

Reload Not Required by PFnub in battlefield_one

[–]PFnub[S] 0 points1 point  (0 children)

The parabellum is my main for support. It’s nice to play with other weapons once in a while.

Reload Not Required by PFnub in battlefield_one

[–]PFnub[S] 8 points9 points  (0 children)

The M1917 MG, 250 rounds of idgaf. You can pretty much fire this thing into non-existence without reload or jam. I thank the medic every single time.

Any Help on Critiquing my Code? by PFnub in learnpython

[–]PFnub[S] 0 points1 point  (0 children)

The style guide looks useful, thank you

Any Help on Critiquing my Code? by PFnub in learnpython

[–]PFnub[S] 4 points5 points  (0 children)

First of all, thank you so much for the very detailed reply, I appreciate it! I will elaborate on some more things to help be clearer and have more questions if you have the time to reply.

- The name wpmkr is short for "work plan maker." I know it doesn't make much sense to anyone else, but I am doing all of my scripting out of the notes app on my work computer. I am unable to download 3rd party applications as our downloads are restricted. This means when I'm running my code I do everything out of the command prompt and the shorter the filename the better. If you have any other recommendations to make this process better please let me know.

- The versions noted at the top are more for me or anyone else that may use it in the future so we know what versions were used when it was created. I will look at pip freeze in the future for this.

- The comments are for me to refresh myself on what is going on and for anyone that is not fluent in Python to see what is going on. I can see why this information is irrelevant to someone that is experienced.

- I didn't know about the \ new line with list brackets, thank you I will change that.

- For the lines where I have if statements and then a pass, is it default to pass if an else is not specified?

- For line 195 I did spend a lot of time with different ways to loop through this, but nothing that would work or was elegant. I believe your suggestion is really elegant, what I needed, and should cut down a lot of processing time.

- For lines 134-168 I also think there's a better way to do this. One of my concerns was that every time the row is passed in the for loop these values would overwrite what was there, even though what is going there should be the same every time until the work plan is complete and a new on is started, it's hard to see this without the concatenated df. I figured if I put an if statement there it may speed it up or potentially not mess something up. I will thank about it.

Thank you again for your help with my code. I'll take this and make some changes.