search

jpype-project / jpype

JPype is cross language bridge to allow Python programs full access to Java class libraries.
http://www.jpype.org
Apache License 2.0
1.12k stars 183 forks source link

findjvm.py: print() is a function in Python 3 #1152

Closed cclauss closed 11 months ago

cclauss commented 1 year ago 
- print "MessageRate =", float(NUMMSGS) / (time.time() - t0)
+ print("MessageRate =", float(NUMMSGS) / (time.time() - t0))

And similar changes to avoid SyntaxErrors on Python 3.

codecov[bot] commented 1 year ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7363ce8) 87.84% compared to head (88107df) 87.85%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1152 +/- ## ======================================= Coverage 87.84% 87.85% ======================================= Files 112 112 Lines 10276 10276 Branches 4032 4032 ======================================= + Hits 9027 9028 +1 Misses 698 698 + Partials 551 550 -1 ``` [see 1 file with indirect coverage changes](https://app.codecov.io/gh/jpype-project/jpype/pull/1152/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jpype-project)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

marscher commented 11 months ago

Thanks for fixing that. Did you actually try to run these examples? Do they provide any meaningful output? I think they are not used/updated for many years.