Opened 5 years ago

Closed 5 years ago

#22518 closed defect (fixed)

Fix bug in list functionality of free module bases

Reported by: egourgoulhon Owned by:
Priority: major Milestone: sage-7.6
Component: linear algebra Keywords: free module, basis
Cc: tscrim Merged in:
Authors: Eric Gourgoulhon Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: ec8fc98 (Commits, GitHub, GitLab) Commit: ec8fc985db5bd75d4bc4b18e98aeb6b4b81413bd
Dependencies: Stopgaps:

Status badges

Description

The following is a bug: 

sage: M = FiniteRankFreeModule(ZZ, 3, name='M', start_index=1)
sage: e = M.basis('e')
sage: list(e)
...
ValueError: Index out of range: 0 not in [1,3]

This ticket fixes it.

Change History (5)

comment:1 Changed 5 years ago by egourgoulhon

  • Branch set to public/manifolds/bug-22518
  • Cc tscrim added
  • Commit set to ec8fc985db5bd75d4bc4b18e98aeb6b4b81413bd
  • Status changed from new to needs_review

New commits:

​ec8fc98Fix bug in list functionality of free module bases (#22518)

comment:2 follow-up: Changed 5 years ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

LGTM.

comment:3 in reply to: ↑ 2 Changed 5 years ago by egourgoulhon

Thanks Travis for the review!

comment:4 Changed 5 years ago by tscrim

It was nice and simple.

comment:5 Changed 5 years ago by vbraun

  • Branch changed from public/manifolds/bug-22518 to ec8fc985db5bd75d4bc4b18e98aeb6b4b81413bd
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.