Issue330

Issues
Show Unassigned
Show All
Search

Login




Register
Lost your login?

Help
Roundup docs

Title report translator memory usage
Priority feature Status resolved
Superseder Nosy List jendrik, malte
Assigned To jendrik Keywords
Optional summary 


















Created on 2012-04-16.09:19:14 by jendrik, last changed  by jendrik.








 
Messages

 
  

   msg2114 (view)
   Author: jendrik
   Date: 2012-04-16.14:22:01
   
    
   
  

  

   
    
Pushed.

   		
  

 
 
  

   msg2113 (view)
   Author: malte
   Date: 2012-04-16.13:41:27
   
    
   
  

  

   
    
Looks good! I'd maybe change the warning text to "warning: could not determine
peak memory" (removing "translator") -- then this is a general utility function
that could be more easily reused in other Python programs.

Feel free to close the branch and merge.

   		
  

 
 
  

   msg2112 (view)
   Author: jendrik
   Date: 2012-04-16.13:25:03
   
    
   
  

  

   
    
I made the proposed changes and pushed the branch to 
bitbucket/jendrikseipp/downward.

   		
  

 
 
  

   msg2111 (view)
   Author: malte
   Date: 2012-04-16.10:32:53
   
    
   
  

  

   
    
Looks good! I added a few comments to the Rietveld issue.

   		
  

 
 
  

   msg2110 (view)
   Author: jendrik
   Date: 2012-04-16.10:02:18
   
    
   
  

  

   
    
Rietveld issue: http://codereview.appspot.com/6030056/

   		
  

 





History


Date
User
Action
Args



2012-04-16 14:22:01jendriksetstatus: chatting -> resolved
messages:
  + msg2114


2012-04-16 13:41:27maltesetmessages:
  + msg2113


2012-04-16 13:25:03jendriksetmessages:
  + msg2112


2012-04-16 10:32:54maltesetmessages:
  + msg2111


2012-04-16 10:02:18jendriksetstatus: unread -> chatting
messages:
  + msg2110


2012-04-16 09:19:14jendrikcreate