Jython 2.7, fullscreen, launcher & more #25

Merged
merged 35 commits into from Apr 1, 2013

Conversation

Projects
None yet
2 participants
Contributor

ralfbiedert commented Apr 1, 2013

I have been using Processing.py quite a lot recently. With it, I also implemented some new features and bug fixes. In particular:

• Now uses jython 2.7
• Added support for fullscreen mode
• Added convenience wrappers to launch sketches without the command line
• Optionally bundled JREs 
• Also added support for true "standalone" sketches with a separate launcher 
• Better support for frameworks such as Ani 
## Quick Start ##
-Download the processing.py distribution.
+Download __[Processing.py All-inclusive](http://s.xr.io/processing.py/latest.zip)__ (Windows & Mac, ~170mb).
@jdf

jdf Apr 1, 2013

Owner

Let's change these links to github file download links.

@ralfbiedert

ralfbiedert Apr 1, 2013

Contributor

Problem: github disabled uploads / downloads. (https://github.com/blog/1302-goodbye-uploads)

There should be a place to have the binary downloads as well. Do you have a project server or any other place to put them?

@jdf

jdf Apr 1, 2013

Owner

On Mon, Apr 1, 2013 at 10:01 AM, Ralf Biedert notifications@github.comwrote:

In README.markdown:

Quick Start

-Download the processing.py distribution.
+Download Processing.py All-inclusive (Windows & Mac, ~170mb).

Problem: github disabled uploads / downloads. (
https://github.com/blog/1302-goodbye-uploads)

There should be a place to have the binary downloads as well. Do you have
a project server or any other place to put them?

Bah.

I'll figure out something. Once the project is merged I'll create a place
to put binaries and update the docs.

Jonathan Feinberg jdf@pobox.com http://MrFeinberg.com/

- $ java -jar processing-py.jar examples.py/Basics/Math/noisefield.py
@jdf

jdf Apr 1, 2013

Owner

I'd love for the intro docs to still contain pointers to some of these demos.

@ralfbiedert

ralfbiedert Apr 1, 2013

Contributor

Will do.

README.markdown
- """
@jdf

jdf Apr 1, 2013

Owner

Could you restore this stuff?

I very much like what you've done to the README, but I'm wondering if you'd do an edit pass to it now to integrate what you've done with the original project, so that it doesn't look like a fork anymore, but is really a core contribution to the original project. Let me know what you think.

@ralfbiedert

ralfbiedert Apr 1, 2013

Contributor

I can rework it. However, since the README is the "main entry point" into the project at the moment I recall that I found it a little confusing when I had my first look. I'll restore and rework a few things and submit it again.

@jdf

jdf Apr 1, 2013

Owner

Ah, that's very helpful feedback. Well, don't make it confusing again; do what you think is most helpful for the newcomer, and I can always find a different place to put the heavy verbiage. Thanks!

+ """Returns the script's path, or the path for some data close to the script."""
+ import java.lang.System as System
+ return System.getProperty("python.main.root") + "/" + name
+
@jdf

jdf Apr 1, 2013

Owner

Please add a newline here.

Owner

jdf commented Apr 1, 2013

Maybe add a unit test for the new native float thingy?

Contributor

ralfbiedert commented Apr 1, 2013

Addressed most of the issues (apart from the download links), fixed some other bugs as well.

Also, can you review the 'set' test please? It looks as if the unit test was wrong, at least CPython reports MySet in both cases.

(And the @pythonusage is merely experimental … might as well remove it, since all of that might be covered by python-unit tests as well)

@ghost ghost assigned jdf Apr 1, 2013

jdf added a commit that referenced this pull request Apr 1, 2013

Merge pull request #25 from ralfbiedert/master
Jython 2.7, fullscreen, launcher & more

@jdf jdf merged commit 6de2184 into jdf:master Apr 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment