hanyuone / Ways-to-NOT-make-money

A basic Python (2.x/3.x) incremental game.
8 stars 3 forks source link

print("one clicked") will not work for py2 use eval('print("one click… #110

Closed bortkiewicz closed 8 years ago

bortkiewicz commented 8 years ago

…ed")') instead.

THE CURRENT FRAMES.PY WILL NOT WORK FOR PY2. @Cyber-Shadow I'm just letting you know.

hanyuone commented 8 years ago

So... are the pythons merged? What's the deal with gamePy3.py?

Also, what is this magics with frames and normal_upgrade?

bortkiewicz commented 8 years ago

uh... what do you mean?

hanyuone commented 8 years ago

Why is gamePy3.py still there? Should we get rid of it?

Also, the parameter callback isn't in bf. There's just a lot of issues, which are all practically @joshgit's fault, if I'm brutally honest.

hanyuone commented 8 years ago

As well as that, @joshgit, @testitem, the -illions are in save_and_load, but not in game_model or game. Should we put millions into all of them or get rid of it (and put it back in if the problem persists)?

joshgit commented 8 years ago

@DerpfacePython I don't see -illions in my save_and_load file. Can you show me the exact line? Did a bad merge happen?

What are you talking about re: "Also, the parameter callback isn't in bf" ? Can you show us a line number or stack trace?

What is gamePy3.py?

I'm suspicious about these problems suddenly cropping up at this point, and it looks like you guys have made other changes that could be triggering these problems.

Are we getting rid of Python 2 support?

Cyber-Shadow commented 8 years ago

Do everyone a favor by getting rid of the python 3 version.

joshgit commented 8 years ago

@Cyber-Shadow That is stupid.

hanyuone commented 8 years ago

@joshgit, @Cyber-Shadow has a point - we're merging the Py2 and Py3 versions anyway. I'm just not sure if @testitem is finished with merging it yet.


And no, the problems "cropping up" is ultimately because of you, @joshgit, for not checking the code properly. The line is here: bf = ButtonFrame(bottom_frame, {'one': lambda: eval("print('one clicked')"), 'two': lambda: eval("print('two clicked')")})

We (i.e. @testitem and I) had to fix this because you didn't check the code for whether it works or not..

And there are two arguments in bf's ButtonFrame.

The ButtonFrame class's __init__ requires three arguments: master, name_action_list, callback.


EDIT: Not all the problems are with you, @joshgit (but most of them still are, anyway): the -illions thing was my fault, I included in auto_updater.

joshgit commented 8 years ago

So there's the problem with ButtonFrame, which you said you fixed, and the problem with -illions which you caused... What are the remaining problems?

hanyuone commented 8 years ago

The ButtonFrame issue isn't fixed yet - @testitem fixed it so that the print statement would work properly with the lambda. There's still no third argument on bf, so I was just wondering what it is.

joshgit commented 8 years ago

That's in the GameFrame class, which isn't used right now, right? What would you use for the 3rd argument?

hanyuone commented 8 years ago

I have no idea, TBH, @joshgit. I thought you put it there.

bortkiewicz commented 8 years ago

WELL THE PY2EXE WON'T LET ME COMPILE GameFrame WITH CODE ERRORS, @joshgit .

bortkiewicz commented 8 years ago

I merged Py2 and Py3, @DerpfacePython . @joshgit , yell at me if there are errors.

hanyuone commented 8 years ago

So wait... who made frames?

bortkiewicz commented 8 years ago

@joshgit did. His pull request (#97 ) was superseded by mine (#107 )

joshgit commented 8 years ago

@testitem just add an argument...

hanyuone commented 8 years ago

@joshgit Shouldn't you add an argument, since you made frames and understand the situation better than both of us can?

joshgit commented 8 years ago

No, just read the code. :D

bortkiewicz commented 8 years ago

wait.. i meant #108 .

@joshgit , does game.py work for both Python 2 and Python 3?

bortkiewicz commented 8 years ago

EDIT: Not all the problems are with you, @joshgit (but most of them still are, anyway): the -illions thing was my fault, I included in auto_updater.

The -illions thing aren't needed in Python 3 because Python 3 doesn't convert huge numbers into different data types. The OverflowError happened when Python 2 didn't convert the huge number (How the heck that happened I still do not truly know,) into a long int. We could actually fix it by only storing x, y, and z in the equation (money = x10^y + z), also displaying it in the same format. The stumbling block is when y gets huge, x is a whole number and z is a number like 32187637246234. Then z generates an OverflowError... The -illions thing is better. But what if the money amount goes higher than 2147483647 quintillion? The only way to fix this bug is to use the Python 3 version since Python 3 allows integers above 2147483647.

bortkiewicz commented 8 years ago

(Continued from previous post)

Why is @Cyber-Shadow hate Python 3 so much? Python 3 can make fixing the -illions bug way easier than fixing it in Python 2.

SIDE NOTE: Wow why the heck did I change my passcode into a thirty-digit one?

hanyuone commented 8 years ago

@testitem It's kind of an in-joke between me and @Cyber-Shadow, TBH.

bortkiewicz commented 8 years ago

oh.

bortkiewicz commented 8 years ago

Is there anything we have to add for 0.7.3? (I got rid of the version.major.minor.patch format since we merged the game.) If not, I'll merge the pull request. @DerpfacePython @joshgit _IF NO ONE RESPONDS BY 12:00 PM 5-30 EDT, I WILL MERGE THE PULL REQUEST WITHOUT NOTICE._

bortkiewicz commented 8 years ago

So no one replied.

bortkiewicz commented 8 years ago

I guess no one saw this anyways. @DerpfacePython are you going to merge this or NOT?

EDIT: GitHub said I cant merge the pull request, What does that mean?

This branch has no conflicts with the base branch Only those with write access to this repository can merge pull requests.

hanyuone commented 8 years ago

Hello, sorry about not replying. I'm slightly busy at the moment, with exams going on - I didn't really have time to reply.

Your new account wasn't invited to be a collaborator... whoops. I invited you now.

bortkiewicz commented 8 years ago

Oh. Thanks!