Topic: main roads becoming normal again
| 
	Tibor | 
	
	 
    
        Posted at:
        
            
    2018-04-30, 20:33 UTC+2.0
        
    
	 I pushed updated version to launchpad  
		Top
	
	
		 
		Quote
	
	
 | 
            
| 
	GunChleoc | 
	
	 
    
        Posted at:
        
            
    2018-05-01, 08:54 UTC+2.0
        
    
	 We are not getting any AppVeyor builds at the moment, because somebody had initiated a merge request for an empty branch, which totally screwed up our build bot. Launchpad doesn't offer any continuous integration options, so SirVer has hacked together a bot that is running on his Raspberry Pi. There is no really clean way of doing this, so it is pretty fragile. I notified him yesterday, and we have to wait for him to find the time to get it running again. The first order of business after Build 20 will be to move to GitHub so that we can integrate Travis and AppVeyor directly and retire our little hard-working bunnybot. Busy indexing nil values  
		Top
	
	
		 
		Quote
	
	
 | 
            
| 
	GunChleoc | 
	
	 
    
        Posted at:
        
            
    2018-05-02, 09:39 UTC+2.0
        
    
	 Release builds are ready now, debug builds are still building: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_road_promotions-3235 Busy indexing nil values  
		Top
	
	
		 
		Quote
	
	
 | 
            
| 
	ypopezios | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 01:25 UTC+2.0
        
    
	 I conducted my tests. What works: 
 What doesn't work: 
 For both the things that don't work, the reason is that I limited my work to specific files (having already moved beyond the initial method). I tried to do the most out of those files, but it turned out that the involved code is more spread than necessary, which needs a good understanding of the codebase, and in particular of the scheduling/signaling system (which probably needs to be reworked, but I wouldn't go there without serious help). The lines for those two things (100, 411 and 412 in file tribes/carrier.cc) can get commented out (although their effect is minimal). 
            Edited:
            
                
    2018-05-03, 13:01 UTC+2.0
            
         
	
    
	 
		Top
	
	
		 
		Quote
	
	
 | 
            
	hessenfarmer 
			
		
	
	 | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 07:24 UTC+2.0
        
    
	 Could you provide replays?  
		Top
	
	
		 
		Quote
	
	
 | 
            
| 
	Tibor | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 08:17 UTC+2.0
        
    
	 Description looks good to me, I have not tested it though...  
		Top
	
	
		 
		Quote
	
	
 | 
            
| 
	GunChleoc | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 09:14 UTC+2.0
        
    
	 I have added some small code style tweaks to the branch. I have also seen some more stuff in the Road class where my code style brain went "eek", e.g. the completely superfluous carrier type - that's a problem for another day though. Busy indexing nil values  
		Top
	
	
		 
		Quote
	
	
 | 
            
	einstein13 
			
		
	
	 | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 19:34 UTC+2.0
        
    
	 Hi, I can't stick to all the discussion (I don't have a time to read all 19 pages). But I have a question about old improvement to the roads: is there any idea how to solve manual upgrade/downgrade of the road? I know that this could solve many problems for some players. einstein13  
		Top
	
	
		 
		Quote
	
	
 | 
            
| 
	GunChleoc | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 21:26 UTC+2.0
        
    
	
 Not in the current discussion, no. It is something worth thinking about, but the question is how it would interact with the automatic system. Micromanagement by the player here would need to be an option for good players and something that newer players shouldn't have to worry about. If somebody wanted to take on the backend, I could take care of the UI. Busy indexing nil values  
		Top
	
	
		 
		Quote
	
	
 | 
            
	einstein13 
			
		
	
	 | 
	
	 
    
        Posted at:
        
            
    2018-05-03, 22:30 UTC+2.0
        
    
	 As I understood previous system, a player could add some points to any road and instantly upgrade it. That idea didn't change the logic because further behaviour was as before. So if there is some type of points that counts how the road should look like, it is always possible to do that. Newbies don't have to change anything, their roads will upgrade anyway (but after some time), but advanced players can predict bottlenecks faster and know how to solve any traffic jams faster. Downgrading would behave like always. Micromanagement here don't have to be done, but can be (other opinion needed here). einstein13  
		Top
	
	
		 
		Quote
	
	
 | 
            

        
		
		
		
		
		
		