Project

General

Profile

Feature #419

FloatStr

Added by John Abbott about 10 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Incomplete function
Target version:
Start date:
06 Jan 2014
Due date:
% Done:

100%

Estimated time:
2.00 h
Spent time:

Description

FloatStr still uses interpreted C5 code; change it to use the C++ version (which is usefully faster for large numbers).


Related issues

Related to CoCoA-5 - Design #534: Remove float.cpkg5Closed2014-04-10

Related to CoCoALib - Slug #537: FloatStr uses too much memoryNew2014-04-16

History

#1 Updated by John Abbott about 10 years ago

While we're looking at the C++ version of FloatStr (which works via MantExp), try profiling on some larger values (e.g. factorial(10^8)).

#2 Updated by Anna Maria Bigatti almost 10 years ago

  • Category set to Incomplete function
  • Assignee set to John Abbott
  • Target version set to CoCoA-5.1.0 Easter14

#3 Updated by John Abbott almost 10 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 10
  • Estimated time set to 2.00 h

This should be quick & easy!

#4 Updated by Anna Maria Bigatti almost 10 years ago

  • % Done changed from 10 to 30

I tested the new DecimalStr: I'd prefer 3 digits after "." by default (instead of 4).
I find it more natural to group digits in 3.

#5 Updated by John Abbott almost 10 years ago

  • Status changed from In Progress to Feedback
  • % Done changed from 30 to 90

Implementation is in ToString.H/C

DecimalStr now defaults to 3 digits after decimal point.

#6 Updated by John Abbott almost 10 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 90 to 100

Completed doc & testing. Anna's happy too. Closing.

Also available in: Atom PDF