remove or add a way to disable println!("count {}", reverse_count); #28

Activity

1. mdrokz

   added

   bugfix

   bugSomething isn't workingSomething isn't working

   and removed

   bugfix

   on Jan 6, 2024
2. mdrokz commented on Jan 6, 2024

   

   mdrokz

   on Jan 6, 2024

   Owner

   More actions

   First, thanks for your work :)

   I'm trying to silence llama.cpp output and keep only the answer. I've closed stderr temporally while loading the model (this is not a nice approach, but it works).

   unsafe {
       libc::close(libc::STDERR_FILENO);
   }
   
   let llama = LLama::new(model,&options);
   
   unsafe {
       let wr = "w".as_ptr() as *const c_char;
       let fd = libc::fdopen(libc::STDERR_FILENO, wr);
       libc::dup2(fd as i32, libc::STDERR_FILENO);
   }

   But when I call predict I still have an unwanted output count 0. Maybe you can change it to log::debug!("count {}", reverse_count); ?

   I will add logging instead of println! that will be better thanks!
3. BenjaminMassey

   added a commit that references this issue on Jun 21, 2024

   Removed count println from predict(..) as per mdrokz#28

   01255b9
4. mdrokz

   added a commit that references this issue on Jun 27, 2024

   Merge pull request #43 from BenjaminMassey/master

   ...

   49ab940