Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add get_bandwidth to matrix/matrix2.pyx #13565

Open
sagetrac-r-gaia-cs mannequin opened this issue Oct 3, 2012 · 6 comments
Open

Add get_bandwidth to matrix/matrix2.pyx #13565

sagetrac-r-gaia-cs mannequin opened this issue Oct 3, 2012 · 6 comments
Assignees
@sagetrac-r-gaia-cs
Labels
c: linear algebra p: major / 3 s: needs work t: enhancement

Comments

@sagetrac-r-gaia-cs
Copy link
Mannequin

sagetrac-r-gaia-cs mannequin commented Oct 3, 2012

It will be great one method to return the bandwidth of a matrix.

Apply attachment: trac_13565-bandwidth.patch

Depends on #13564

Component: linear algebra

Keywords: matrix, bandwidth

Author: r.gaia.cs

Issue created by migration from https://trac.sagemath.org/ticket/13565
@sagetrac-r-gaia-cs sagetrac-r-gaia-cs mannequin added this to the sage-5.11 milestone Oct 3, 2012
@sagetrac-r-gaia-cs sagetrac-r-gaia-cs mannequin self-assigned this Oct 3, 2012
@sagetrac-r-gaia-cs
Copy link
Mannequin Author

sagetrac-r-gaia-cs mannequin commented Oct 3, 2012

Attachment: bandwidth.patch.gz

@ppurka
Copy link
Member

ppurka commented Oct 8, 2012

comment:2

Hello,

  1. you need to provide a good commit message. You can do that by using hg. This will then appear in your patch instead of the [mq]: bandwidth at the top of the patch.
sage -hg qref -m "a good commit message"
  1. Also, name your patch as per this ticket number. Such as trac_13565-bandwidth.patch, instead of simply bandwidth.patch.

Both these modifications help the release manager when the patch is merged into Sage proper. Same comments apply to your dependency ticket #13564.

@sagetrac-r-gaia-cs
Copy link
Mannequin Author

sagetrac-r-gaia-cs mannequin commented Oct 8, 2012

Attachment: trac_13565-bandwidth.patch.gz

Make correction sugested by ppurka.

@ppurka

This comment has been minimized.

Sign in to view
@ppurka
Copy link
Member

ppurka commented Oct 12, 2012

comment:4

Force patchbot.

Patchbot apply trac_13565-bandwidth.patch

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@fchapoton
Copy link
Contributor

comment:8

needs a git branch

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
@dcoudert dcoudert mentioned this issue Jan 4, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-r-gaia-cs sagetrac-r-gaia-cs

Labels
c: linear algebra p: major / 3 s: needs work t: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ppurka @jdemeyer @fchapoton @mkoeppe