Opened 6 months ago
Last modified 5 months ago
#30287 closed enhancement
sage.tensor.modules.free_module_basis: Add testsuite — at Version 6
Reported by: | mkoeppe | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | sage-9.2 |
Component: | linear algebra | Keywords: | |
Cc: | gh-mjungmath, egourgoulhon, tscrim | Merged in: | |
Authors: | Matthias Koeppe | Reviewers: | |
Report Upstream: | N/A | Work issues: | |
Branch: | u/mkoeppe/sage_tensor_modules_free_module_basis__add_testsuite (Commits) | Commit: | f1f67d3ddf2639d20ea4d8cd7bac4bd59e90a155 |
Dependencies: | Stopgaps: |
Description (last modified by )
We add a _test...
method to Basis_abstract
and run the test suite for some instances of FreeModuleBasis
, FreeModuleCoBasis
.
Change History (6)
comment:1 Changed 6 months ago by
- Branch set to u/mkoeppe/sage_tensor_modules_free_module_basis__add_testsuite
comment:2 Changed 6 months ago by
- Commit set to 1e69c0eba014ac4d5a1e5ecd664f1bf953242507
comment:3 Changed 6 months ago by
Should the method FreeModuleCoBasis.free_module
really return the (primal) base module, as it currently does? Or rather the dual of it?
comment:4 Changed 6 months ago by
- Status changed from new to needs_info
comment:5 Changed 6 months ago by
- Commit changed from 1e69c0eba014ac4d5a1e5ecd664f1bf953242507 to f1f67d3ddf2639d20ea4d8cd7bac4bd59e90a155
Branch pushed to git repo; I updated commit sha1. New commits:
f1f67d3 | FreeModuleCoBasis: Add _test_iter_len
|
comment:6 Changed 6 months ago by
- Description modified (diff)
- Priority changed from major to minor
- Status changed from needs_info to needs_review
- Type changed from defect to enhancement
Note: See
TracTickets for help on using
tickets.
The tests currently fail:
New commits:
sage.tensor.modules.free_module_basis.Basis_abstract: Add _test_iter_len