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

Reduce COSMOS memory usage #78

Closed
jmthomas opened this issue Feb 27, 2015 · 2 comments
Closed

Reduce COSMOS memory usage #78

jmthomas opened this issue Feb 27, 2015 · 2 comments
Milestone

Comments

@jmthomas
Copy link
Member

@jmthomas jmthomas commented Feb 27, 2015

No description provided.

jmthomas pushed a commit that referenced this issue Feb 27, 2015
jmthomas pushed a commit that referenced this issue Feb 27, 2015
jmthomas pushed a commit that referenced this issue Feb 27, 2015
@jmthomas
Copy link
Member Author

@jmthomas jmthomas commented Feb 27, 2015

I ran my configuration which spawns 30 targets and which I continuously send commands to all targets. Improvements are pretty dramatic:
22x improvement in memory allocation in Qt.rb and 1/2 the number of objects
75x improvement in memory allocation in cmd_tlm_server_gui.rb and 1/2 the number of objects
2.5x improvement in memory allocation in json_rpc.rb and 1/4 the number of objects

Other pieces of code show more minor improvements. All specs pass.

Loading

jmthomas pushed a commit that referenced this issue Mar 6, 2015
jmthomas pushed a commit that referenced this issue Mar 6, 2015
Update json_drb and tcpip_socket_stream to use single object in read_nonblock to reduce memory usage. Reduce max blocks in cmd_tlm_server_gui.txt to avoid out of memory errors. Other minor performance improvements. #78
jmthomas pushed a commit that referenced this issue Mar 6, 2015
jmthomas pushed a commit that referenced this issue Mar 12, 2015
ryanatball added a commit that referenced this issue Mar 12, 2015
ryanatball added a commit that referenced this issue Mar 12, 2015
jmthomas added a commit that referenced this issue Mar 12, 2015
@ryanatball
Copy link
Member

@ryanatball ryanatball commented Mar 13, 2015

closed by pull request #79

Loading

@ryanatball ryanatball closed this Mar 13, 2015
@ryanatball ryanatball added this to the v3.3.1 milestone Mar 20, 2015
ryanatball pushed a commit that referenced this issue Apr 12, 2021
Merge in COSMOSEE/base from copy_on_write to master

* commit 'e4a9a2343c89ee864a1d16569a5d5825acb17b8a':
  Remove space before star
  Copy on write scripts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants