Project

General

Profile

Support #940

Check all fns in packages work even without BackwardCompatible.cpkg5

Added by John Abbott almost 2 years ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
enhancing/improving
Target version:
Start date:
10 Oct 2016
Due date:
% Done:

100%

Estimated time:
Spent time:

Description

Even though (we hope that) all the packages are destined to migrate to CoCoALib, it is probably worth keeping them reasonably "clean". In this case I mean that they should use the "modern" CoCoA-5 function names rather than the ones from BackwardCompatible.cpkg5. This should also help make the migration quicker and less painful.


Related issues

Related to CoCoA-5 - Design #868: Der, Bin: why the capital letter?Closed2016-04-17

Related to CoCoA-5 - Design #939: Rename Fact to factorial?Closed2016-10-10

Related to CoCoA-5 - Support #941: Improve coverage of CoCoA-5 testsNew2016-10-10

Related to CoCoA-5 - Design #491: Cleaning: Cruft in CoCoA-5 packagesIn Progress2014-03-22

History

#1 Updated by John Abbott almost 2 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 10

I have already make a first pass at cleaning up the packages, but there is still more to do.

The CoCoA-5 tests are very far from ensuring 100% code coverage -- I'll add this as a new issue.

In some cases I was not sure what the correct modernization would be (e.g. several packages call a function called Poly; there were other examples too, I believe).

Some packages define "local" functions whose names are the same as those of "global" functions (e.g. in the latex package there are functions called Poly, Mat, List and Ideal). This is rather confusing; some other functions have names like Latex_term and Latex_indet which makes it clearer that they are local. For matrices the fn seems to call Latex but I do not know where that is defined (shouldn't it be either latex or LaTeX?)

#2 Updated by John Abbott almost 2 years ago

  • Related to Design #868: Der, Bin: why the capital letter? added

#3 Updated by John Abbott almost 2 years ago

  • Related to Design #939: Rename Fact to factorial? added

#4 Updated by John Abbott almost 2 years ago

  • Related to Support #941: Improve coverage of CoCoA-5 tests added

#5 Updated by John Abbott almost 2 years ago

There is quite a lot of "cruft" in the packages: either commented out code or code which I suspect can never be called. I have been inconsistent in "modernizing" the cruft (since I hope practically all of it will soon be eliminated).

#6 Updated by John Abbott almost 2 years ago

  • Related to Design #491: Cleaning: Cruft in CoCoA-5 packages added

#7 Updated by John Abbott 8 months ago

  • Status changed from In Progress to Closed
  • Assignee set to John Abbott
  • % Done changed from 10 to 100

I have just run the CoCoA-5 tests with BackwardCompatible.cpkg5 moved to different place. All tests passed.

Yes, 100% coverage would be nice, but we cannot afford to do it...

Closing.

#8 Updated by Anna Maria Bigatti 8 months ago

  • Target version changed from CoCoA-5.?.? to CoCoA-5.2.4

#9 Updated by Anna Maria Bigatti 7 months ago

  • Target version changed from CoCoA-5.2.4 to CoCoA-5.2.2

Also available in: Atom PDF