Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-20 Thread 宋文武
l...@gnu.org (Ludovic Courtès) writes: > iyzs...@member.fsf.org (宋文武) skribis: > >> From d0c49ac1341c21d0efb069afb1521f61541e3eb4 Mon Sep 17 00:00:00 2001 >> From: =?UTF-8?q?=E5=AE=8B=E6=96=87=E6=AD=A6?= >> Date: Sat, 16 Sep 2017 14:10:18 -0600 >> Subject: [PATCH] download: Don't report the progr

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-19 Thread Ludovic Courtès
iyzs...@member.fsf.org (宋文武) skribis: > From d0c49ac1341c21d0efb069afb1521f61541e3eb4 Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?=E5=AE=8B=E6=96=87=E6=AD=A6?= > Date: Sat, 16 Sep 2017 14:10:18 -0600 > Subject: [PATCH] download: Don't report the progress too fast. > > * guix/utils.scm (): New reco

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-19 Thread Ludovic Courtès
Hello! iyzs...@member.fsf.org (宋文武) skribis: > l...@gnu.org (Ludovic Courtès) writes: [...] >> To address these, I would use ‘abort-to-prompt’ & co., possibly with >> “suspendable ports”, but this is a 2.2 feature. (It may be that we >> should use Fibers directly.) > > Sure, I’d like to try fi

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-15 Thread 宋文武
iyzs...@member.fsf.org (宋文武) writes: > l...@gnu.org (Ludovic Courtès) writes: > >> [...] >> Thoughts? > > How about replace the use of thread from ‘progress-reporter/file’ by a > ‘rate-limited’ render procedure, and then change the elapsed time to > ETA, which is okay without updating in seconds?

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-14 Thread 宋文武
l...@gnu.org (Ludovic Courtès) writes: > [...] >> Date: Fri, 8 Sep 2017 22:49:03 +0800 >> Subject: [PATCH] download: Report the progress asynchronously in another >> thread. >> >> * guix/utils.scm (): New record type. >> (call-with-progress-reporter): New procedure. >> * guix/build/download.scm (

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-10 Thread Ludovic Courtès
Hi! iyzs...@member.fsf.org (宋文武) skribis: > Hi, sorry for my late response. No problem, it was worth waiting. ;-) > l...@gnu.org (Ludovic Courtès) writes: [...] >> (Also, longer-term, we’d want to do that other way around, which is to >> update the report every N milliseconds, as opposed to

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-09-08 Thread 宋文武
Hi, sorry for my late response. l...@gnu.org (Ludovic Courtès) writes: > Hello 宋文武, > > iyzs...@member.fsf.org (宋文武) skribis: > >> Hello, our progress report of 'guix download' can refresh too fast. For >> example, it blinks much with this script: > > Indeed, that’s always annoyed me. I’m glad

Re: [WIP][PATCH] download: Don't report the progress too fast

2017-08-30 Thread Ludovic Courtès
Hello 宋文武, iyzs...@member.fsf.org (宋文武) skribis: > Hello, our progress report of 'guix download' can refresh too fast. For > example, it blinks much with this script: Indeed, that’s always annoyed me. I’m glad you’re looking at it! > I'd like limiting its rate to render every 300ms. So I wri

[WIP][PATCH] download: Don't report the progress too fast

2017-08-26 Thread 宋文武
Hello, our progress report of 'guix download' can refresh too fast. For example, it blinks much with this script: --8<---cut here---start->8--- (use-modules (guix build download)) (let* ((size (expt 2 20)) (progress (progress-proc "" size))) (let loo