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

Remove `jvm-heap-size` command-line option #2048

Closed
0x00b1 opened this issue Jun 1, 2016 · 3 comments

Comments

@0x00b1
Copy link
Collaborator

commented Jun 1, 2016

No description provided.

@braymp

This comment has been minimized.

Copy link
Contributor

commented Jun 2, 2016

Is the memory allocated to the JVM no longer an issue? Because having this flag is critical for being able to load large image sets; see #787, http://forum.cellprofiler.org/t/cannot-load-large-images-java-heap-space-error/1632, and http://forum.cellprofiler.org/t/java-exception-on-pressing-update-inside-metadata-module/2362 for a few examples.

@0x00b1

This comment has been minimized.

Copy link
Collaborator Author

commented Jun 2, 2016

Unfortunately, it’s still an issue. You should use:

set JAVA_OPTS="-Xms128m -Xmx256m"

rather than --jvm-heap-size. For example:

JAVA_OPTS="-Xmx256m" cellprofiler

It’s a useful change from a usability perspective because:

  • there’s one way to modify the setting and
  • it obeys your standard platform-wide setting.
@braymp

This comment has been minimized.

Copy link
Contributor

commented Jun 2, 2016

Probably good to document somewhere then. It's almost certain to come up at
some point.

On Thu, Jun 2, 2016 at 8:27 AM, Allen Goodman notifications@github.com
wrote:

Unfortunately, it’s still an issue. You should use:

set JAVA_OPTS="-Xms128m -Xmx256m"

rather than --jvm-heap-size. For example:

JAVA_OPTS="-Xmx256m" cellprofiler


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#2048 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABIaiPYu76lI-1Qm31zUL2LV1q5CkK1Hks5qHswkgaJpZM4Irzxw
.

@0x00b1 0x00b1 referenced this issue Jun 2, 2016
13 of 15 tasks complete

@0x00b1 0x00b1 closed this in de16df7 Jun 7, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.