Project

General

Profile

Support #696

test-OrderedRing: activate or eliminate?

Added by John Abbott almost 9 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Safety
Start date:
08 May 2015
Due date:
% Done:

100%

Estimated time:
3.01 h
Spent time:

Description

There is a file test-OrderedRing1.C which is not actually executed.
Either fix it so that it works (currently fails with InsufficientPrecision) and ensure that it is called, or eliminate it.


Related issues

Related to CoCoALib - Support #695: Remove cruft from test filesNew2015-05-08

Related to CoCoALib - Bug #853: NearestInt can needlessly throw InsufficientPrecisionClosed2016-03-23

Related to CoCoA-5 - Feature #857: floor, ceil, and NearestInt for RingElem in CoCoA-5?New2016-03-25

History

#1 Updated by John Abbott almost 9 years ago

  • Subject changed from test-OrderingRing: activate or eliminate? to test-OrderedRing: activate or eliminate?

#2 Updated by Anna Maria Bigatti about 8 years ago

I don't see it in cvs.

#3 Updated by John Abbott about 8 years ago

  • Assignee set to John Abbott

I'll try to fix it and add it to CVS.

#4 Updated by Anna Maria Bigatti about 8 years ago

  • Target version changed from CoCoALib-0.99540 Feb 2016 to CoCoALib-0.99550 spring 2017

#5 Updated by John Abbott about 8 years ago

  • Related to Bug #853: NearestInt can needlessly throw InsufficientPrecision added

#6 Updated by John Abbott about 8 years ago

Well, this test is worthwhile: it has found a bug in the impl of NearestInt (see #853).

#7 Updated by John Abbott about 8 years ago

I have now checked in 2 new tests for ordered domains: test-OrderedDomain1 and test-OrderedDomain2. The first test is for ZZ and QQ, while the second is for twin-floats. The tests are very similar, but the twin-float version must handle InsuffPrec.

#8 Updated by John Abbott about 8 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 90

The tests pass, of course.

#9 Updated by John Abbott about 8 years ago

  • Related to Feature #857: floor, ceil, and NearestInt for RingElem in CoCoA-5? added

#10 Updated by John Abbott almost 8 years ago

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

#11 Updated by Anna Maria Bigatti almost 7 years ago

  • Estimated time set to 3.01 h

Also available in: Atom PDF