Support #940
Check all fns in packages work even without BackwardCompatible.cpkg5
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
History
#1 Updated by John Abbott over 7 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 over 7 years ago
- Related to Design #868: Der, Bin: why the capital letter? added
#3 Updated by John Abbott over 7 years ago
- Related to Design #939: Rename Fact to factorial? added
#4 Updated by John Abbott over 7 years ago
- Related to Support #941: Improve coverage of CoCoA-5 tests added
#5 Updated by John Abbott over 7 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 over 7 years ago
- Related to Design #491: Cleaning: Cruft in CoCoA-5 packages added
#7 Updated by John Abbott over 6 years 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 over 6 years ago
- Target version changed from CoCoA-5.?.? to CoCoA-5.2.4
#9 Updated by Anna Maria Bigatti over 6 years ago
- Target version changed from CoCoA-5.2.4 to CoCoA-5.2.2