astroufsc / chimera

Chimera - Observatory Automation System
http://chimera.sf.net/
GNU General Public License v2.0
36 stars 19 forks source link

Fix issues #164, #160 and #37. #165

Closed tribeiro closed 7 years ago

tribeiro commented 7 years ago

Fixes a memory leak on chimera that allows imageserver to open an unrestricted number of fits files (#164). The user can control the number of images added to the queue (default=10) in the configuration file. Control is done when registering images. This also fixes the “Too many open files” issue, #160 and #37, without the need to track all calls to camera.expose().

wschoenell commented 7 years ago

Sounds good to go for me...

tribeiro commented 7 years ago

So? Should we go ahead on this fix?

ankanaan commented 7 years ago

I asked PH a while ago, he didn't react, so I guess it is good with the big boss :-)

On Wed, Jul 5, 2017 at 10:22 AM, Tiago notifications@github.com wrote:

So? Should we go ahead on this fix?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/astroufsc/chimera/pull/165#issuecomment-313101332, or mute the thread https://github.com/notifications/unsubscribe-auth/ADaVPLdieWm0mDi4hF8dKORNOHdCDtebks5sK44wgaJpZM4M5Oe9 .

wschoenell commented 7 years ago

The solution indeed fixes the mentioned issues. I've been using in LNA and T80 these changes without any problem... Will merge and close the issues.