- 
                Notifications
    You must be signed in to change notification settings 
- Fork 104
Dictionary as Monad #448
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
          
     Draft
      
      
            gusty
  wants to merge
  21
  commits into
  master
  
    
      
        
          
  
    
      Choose a base branch
      
     
    
      
        
      
      
        
          
          
        
        
          
            
              
              
              
  
           
        
        
          
            
              
              
           
        
       
     
  
        
          
            
          
            
          
        
       
    
      
from
gusty-dict-applicative
  
      
      
   
  
    
  
  
  
 
  
      
    base: master
Could not load branches
            
              
  
    Branch not found: {{ refName }}
  
            
                
      Loading
              
            Could not load tags
            
            
              Nothing to show
            
              
  
            
                
      Loading
              
            Are you sure you want to change the base?
            Some commits from the old base branch may be removed from the timeline,
            and old review comments may become outdated.
          
          
                
     Draft
            
            Dictionary as Monad #448
Conversation
  
    
      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
    
  
  
    
    e86178a    to
    c02d157      
    Compare
  
    c02d157    to
    25c00cd      
    Compare
  
    | One limitation with this implementation is that it would require the infinite dictionary to be in the first position of the applicative expression, otherwise it would try to iterate through an infinite dictionary. There are ways to improve this: 
 | 
32f5c4a    to
    f2d1afd      
    Compare
  
    5c69948    to
    9ca5706      
    Compare
  
    5b61ffc    to
    ad56a34      
    Compare
  
    429e6fc    to
    8fb276c      
    Compare
  
    018d7cb    to
    25c00cd      
    Compare
  
    984e26f    to
    bfa151d      
    Compare
  
    b64b3fc    to
    88bd6b9      
    Compare
  
    
              
                    Happypig375
  
              
              reviewed
              
                  
                    Jun 13, 2025 
                  
              
              
            
            
| I re-opened it just to experiment, but it doesn't mean that I'm proposing to merge it, as I still think it will break LSP. | 
23de96a    to
    ebc40ef      
    Compare
  
    2769fd0    to
    221911c      
    Compare
  
    5d8211a    to
    bf1d623      
    Compare
  
    
  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment
  
      
  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.
  
    
  
    
A sort of "infinite" dictionary would allow it to act as a Monad and Applicative.