Skip to content
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

Adapt the printed remaining time granularity when it is very high #63

Open
nherment opened this issue May 9, 2014 · 9 comments
Open

Comments

@nherment
Copy link

nherment commented May 9, 2014

We have a data importer and for big files, the remaining time reaches a high amount of seconds (>100'000). That makes the timing tough to estimate.

When the timing reaches say more than 5 minutes, would it be possible to display the time in minutes ? Same for minutes to hours and hours to days ?

reference: senecajs-labs/seneca-import-csv#1

@hallas
Copy link
Contributor

hallas commented May 9, 2014

yeah, good idea

@jdan
Copy link
Collaborator

jdan commented Jun 11, 2014

Could through the ms module in: link.

@marceloboeira
Copy link

@jdan I use ms module to implement :)

Can someone test the code ?

On my tests all its working fine.

@marceloboeira
Copy link

@hallas can you check the pull request?

@hallas
Copy link
Contributor

hallas commented Sep 6, 2014

Hey @marceloboeira

Sorry .. I will.. Been very busy lately

@marceloboeira
Copy link

@hallas no worries man 👍

myndzi pushed a commit to myndzi/node-progress that referenced this issue May 18, 2015
…, e.g. { 's': 1, 'm': 60, 'h': 3600 }

Modify ETA estimation to return the least granular unit for which the eta is >= 1, based on supplied values
Add replacement token ':unit', returns the label associated with the unit value
ProgressBar.defaultUnits holds the h/m/s arrangement; it is not actually used by default to preserve compatibility

Example usage: new ProgressBar(':eta:unit', { units: ProgressBar.defaultUnits });
Example output: '12.3m' -> '38.5s' (changes from minutes to seconds over time)

Addresses visionmedia#63
myndzi pushed a commit to myndzi/node-progress that referenced this issue May 18, 2015
…, e.g. { 's': 1, 'm': 60, 'h': 3600 }

Modify ETA estimation to return the least granular unit for which the eta is >= 1, based on supplied values
Add replacement token ':unit', returns the label associated with the unit value
ProgressBar.defaultUnits holds the h/m/s arrangement; it is not actually used by default to preserve compatibility

Example usage: new ProgressBar(':eta:unit', { units: ProgressBar.defaultUnits });
Example output: '12.3m' -> '38.5s' (changes from minutes to seconds over time)

Addresses visionmedia#63
@hallas
Copy link
Contributor

hallas commented Aug 16, 2015

Reviews are coming very soon! :-)

@marceloboeira
Copy link

0/

@sanderkoenders
Copy link

sanderkoenders commented Oct 4, 2018

Why exactly was this never implemented, I am seeing three pull-requests but none got accepted. There is still a need for this feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants