- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6.8k
[rllib] Parallel-data loading and multi-gpu support for IMPALA #2766
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
          
     Merged
      
      
    
  
     Merged
                    Changes from all commits
      Commits
    
    
            Show all changes
          
          
            51 commits
          
        
        Select commit
          Hold shift + click to select a range
      
      33f01bb
              
                wip
              
              
                ericl d6b5b5d
              
                stats
              
              
                ericl 56081a2
              
                fix assert
              
              
                ericl 33c4871
              
                fix par opt
              
              
                ericl 10cb4ac
              
                clip
              
              
                ericl 517b1ab
              
                fix
              
              
                ericl c84906e
              
                gpu queu
              
              
                ericl 0a01023
              
                multiple loader threadsst
              
              
                ericl a33bff0
              
                thread safe
              
              
                ericl 50843da
              
                fix
              
              
                ericl 763f165
              
                icr threads
              
              
                ericl 2d4e332
              
                depth
              
              
                ericl e5f9095
              
                Merge remote-tracking branch 'upstream/master' into impala-multigpu
              
              
                ericl 91df50b
              
                revert local
              
              
                ericl 60f2d4b
              
                fix
              
              
                ericl 9676459
              
                frac gpu
              
              
                ericl 84cc7db
              
                doc
              
              
                ericl 2295a15
              
                Update rllib-training.rst
              
              
                ericl 10ad1f7
              
                yapf
              
              
                ericl 79d9677
              
                Merge branch 'test-frac-gpu' of github.com:ericl/ray into test-frac-gpu
              
              
                ericl ee3b599
              
                remove xray
              
              
                ericl 6cc13b1
              
                replay params
              
              
                ericl 6070872
              
                fix assert
              
              
                ericl e6fe182
              
                Merge remote-tracking branch 'upstream/master' into impala-multigpu
              
              
                ericl 0c4db6d
              
                rep
              
              
                ericl 54acce9
              
                note
              
              
                ericl c1468a1
              
                update
              
              
                ericl 9d16999
              
                fix ceil
              
              
                ericl 8f5e704
              
                Merge branch 'test-frac-gpu' into impala-multigpu
              
              
                ericl d56768e
              
                Merge remote-tracking branch 'upstream/master' into impala-multigpu
              
              
                ericl 113a968
              
                Merge remote-tracking branch 'upstream/master' into impala-multigpu
              
              
                ericl 5e194c4
              
                fix up names
              
              
                ericl df26220
              
                no debug
              
              
                ericl 14f2339
              
                clean up apex stats too
              
              
                ericl 49c38ea
              
                test
              
              
                ericl dacda0f
              
                fix replay sampling
              
              
                ericl 473f3fe
              
                doc
              
              
                ericl 0dd2bae
              
                comments 0
              
              
                ericl 9d2b535
              
                fix lstm case
              
              
                ericl daebf65
              
                set to 99999
              
              
                ericl 8d66ded
              
                fix assert
              
              
                ericl 8979a54
              
                doc
              
              
                ericl 7c81fe9
              
                impala lstm test
              
              
                ericl d94fe31
              
                remove unneeded assert
              
              
                ericl bae28f6
              
                ppo mask
              
              
                ericl 76640e3
              
                doc
              
              
                ericl 507ae93
              
                lint
              
              
                ericl 73005ca
              
                fix mask
              
              
                ericl 6b3f6f3
              
                fix non lstm case
              
              
                ericl c7d893d
              
                fix impala
              
              
                ericl b76c0c4
              
                damnit flake8
              
              
                ericl File filter
Filter by extension
Conversations
          Failed to load comments.   
        
        
          
      Loading
        
  Jump to
        
          Jump to file
        
      
      
          Failed to load files.   
        
        
          
      Loading
        
  Diff view
Diff view
There are no files selected for viewing
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
      
      Oops, something went wrong.
        
    
  
  Add this suggestion to a batch that can be applied as a single commit.
  This suggestion is invalid because no changes were made to the code.
  Suggestions cannot be applied while the pull request is closed.
  Suggestions cannot be applied while viewing a subset of changes.
  Only one suggestion per line can be applied in a batch.
  Add this suggestion to a batch that can be applied as a single commit.
  Applying suggestions on deleted lines is not supported.
  You must change the existing code in this line in order to create a valid suggestion.
  Outdated suggestions cannot be applied.
  This suggestion has been applied or marked resolved.
  Suggestions cannot be applied from pending reviews.
  Suggestions cannot be applied on multi-line comments.
  Suggestions cannot be applied while the pull request is queued to merge.
  Suggestion cannot be applied right now. Please check back later.
  
    
  
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this work when
num_gpus * gpu_fraction> 1?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know what will happen if gpu_fraction > 1, but for < 1 I believe it should work.