Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: Differentiate between Python 2 and Python 3 saves #144

Merged
merged 1 commit into from
Oct 20, 2015

Conversation

tcld
Copy link
Contributor

@tcld tcld commented Oct 20, 2015

If this is ok, I will follow up with an update to my PR at worldengine-data.

Mindwerks/worldengine-data#1

@tcld
Copy link
Contributor Author

tcld commented Oct 20, 2015

I remembered that World_pb2.py now supposedly is the same for Python 2 and 3. Is that right? And if so: Well...maybe we don't need separate pickle-files? Since I can't use the Python 2-version of Worldengine I also can't check what it would output and if the files maybe are the same.

@psi29a
Copy link
Member

psi29a commented Oct 20, 2015

protobuf solves the python2/3 problem (one file fits all), but only for protobuf worlds. :)

@tcld
Copy link
Contributor Author

tcld commented Oct 20, 2015

Oh, I'm an idiot. Disregard my last comment.^^ (I was thinking about pickle and somehow mixed the protobuf file in, and then thought "Hey, maybe the problem is non-existent?".)

psi29a added a commit that referenced this pull request Oct 20, 2015
Tests: Differentiate between Python 2 and Python 3 saves
@psi29a psi29a merged commit 930ed27 into Mindwerks:master Oct 20, 2015
@tcld tcld mentioned this pull request Oct 20, 2015
@tcld tcld deleted the tests_py23 branch October 23, 2015 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants