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.

Sign up for GitHub

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

Jump to bottom

Remove “Maximum memory for Java (MB)” option #1938

Closed
0x00b1 opened this issue Mar 30, 2016 · 4 comments
Closed

Remove “Maximum memory for Java (MB)” option #1938

0x00b1 opened this issue Mar 30, 2016 · 4 comments
Assignees
@AetherUnbound
Labels
Enhancement

Comments

@0x00b1
Copy link
Contributor

0x00b1 commented Mar 30, 2016

CellProfiler should obey the user’s configuration (i.e. -Xms).
@0x00b1 0x00b1 added this to the Refactor cellprofiler.gui.preferencesdlg milestone Mar 30, 2016
@0x00b1 0x00b1 changed the title Remove “Maximum memory for Java (MB)” option from preferencesdlg.py Remove “Maximum memory for Java (MB)” option Mar 30, 2016
@dlogan dlogan mentioned this issue Apr 8, 2016
Closed
@0x00b1 0x00b1 removed this from the Refactor cellprofiler.gui.preferencesdlg milestone Aug 12, 2016
@mcquin mcquin closed this as completed Apr 13, 2017
@mcquin
Copy link
Contributor

mcquin commented Apr 13, 2017

Still a thing. My search was bad.

@mcquin mcquin reopened this Apr 13, 2017
@mcquin mcquin mentioned this issue Jan 24, 2018
Closed
@bethac07
Copy link
Member

bethac07 commented Aug 9, 2018

CellProfiler still (as of 3.1.5) obeys this preference, not the system java options. This is a bummer since we already removed the command-line-switch to override, which means until this is fixed the user needs to manually edit preferences.py on their system.

@AetherUnbound AetherUnbound self-assigned this Sep 17, 2018
@mbarish
Copy link

mbarish commented Mar 22, 2019

Has this issue of memory limits in Java been addressed? In CP 3.1.8, I cannot find a preferences.py file to modify.
Thanks.

@bethac07 bethac07 mentioned this issue Mar 24, 2020
Closed
0x00b1 added a commit to CellProfiler/core that referenced this issue Jul 7, 2020
@0x00b1
Merge pull request #17 from CellProfiler/issues/1938
b7a12b9 
resolves CellProfiler/CellProfiler#1938
@0x00b1
Copy link
Contributor Author

0x00b1 commented Jul 7, 2020

resolved by CellProfiler/core#17

gnodar01 pushed a commit to gnodar01/cp_full that referenced this issue Oct 20, 2023
@0x00b1
resolves CellProfiler/CellProfiler#1938
b6b1290
gnodar01 pushed a commit to gnodar01/cp_full that referenced this issue Oct 20, 2023
@0x00b1
Merge pull request CellProfiler/core#17 from CellProfiler/issues/1938
ff3b58a 
resolves CellProfiler/CellProfiler#1938
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AetherUnbound AetherUnbound

Labels
Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@0x00b1 @mcquin @bethac07 @AetherUnbound @mbarish