don't return run invisibly #35

Activity

1. bart6114

   added

   enhancement

   on Dec 8, 2015
2. Enchufa2 commented on Dec 8, 2015

   

   Enchufa2

   on Dec 8, 2015

   Member

   More actions

   Mmmh... not specially. I supposed that it could be desirable to be able to do something like this:

   env %>% run(15) %>% onestep() %>% onestep()

   or this:

   env <- simmer() %>%
     add_resource("server") %>%
     add_generator("test0", t0, function() 1) %>%
     run(100) %>%
     add_generator("test1", t0, function() 2) %>%
     run(200) %>%
     del_generator("test0") %>%
     run(300)
3. bart6114 commented on Dec 8, 2015

   

   bart6114

   on Dec 8, 2015

   MemberAuthor

   More actions

   If we replace all - not only $run() - invisible(self) with self in the Simmer R6 class we can still do this. And if it isn't saved in a variable it simply calls the $print function:

   > simmer() %>% run(15) %>% onestep() %>% onestep()
   Simmer environment: anonymous | now: 0 | next: Inf
   

   and

   > env<- simmer() %>% run(15) %>% onestep() %>% onestep()
   > 
   
4. Enchufa2

   closed this as completedin 857117con Dec 8, 2015
5. Enchufa2 commented on Dec 8, 2015

   

   Enchufa2

   on Dec 8, 2015

   Member

   More actions

   Certainly. When you're right, you're right. 😄
6. Enchufa2

   added this to the v3.1.0 milestone on Dec 8, 2015