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

get rid of annoying warning in vector_space_dimension() #8587

Closed
malb opened this issue Mar 23, 2010 · 7 comments
Closed

get rid of annoying warning in vector_space_dimension() #8587

malb opened this issue Mar 23, 2010 · 7 comments
Assignees
@malb
Labels
c: commutative algebra p: minor / 4 t: enhancement
Milestone
sage-4.4.2

Comments

@malb
Copy link
Member

malb commented Mar 23, 2010

We pass a Groebner basis to vdim() of Singular but forgot to mention it to Singular.

CC: @burcin @sagetrac-PolyBoRi

Component: commutative algebra

Author: Martin Albrecht

Reviewer: Burcin Erocal

Merged: sage-4.4.2.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/8587
@malb malb added this to the sage-4.4.2 milestone Mar 23, 2010
@malb malb self-assigned this Mar 23, 2010
@malb
Copy link
Member Author

malb commented Mar 23, 2010

comment:1

Attachment: vdim_warning.patch.gz

@burcin
Copy link

burcin commented May 4, 2010

comment:2

Looks good to me.

AFAICT, the message is printed to stderr, so there is no easy way to test this.

@burcin
Copy link

burcin commented May 4, 2010

Reviewer: Burcin Erocal

@burcin
Copy link

burcin commented May 4, 2010

apply only this patch

@burcin
Copy link

burcin commented May 4, 2010

comment:3

Attachment: trac_8587-vdim_warning.patch.gz

attachment: trac_8587-vdim_warning.patch adds the ticket number in the log message.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 8, 2010

comment:4

Merged trac_8587-vdim_warning.patch.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 8, 2010

Merged: sage-4.4.2.alpha0

@sagetrac-mvngu sagetrac-mvngu mannequin removed the s: positive review label May 8, 2010
@sagetrac-mvngu sagetrac-mvngu mannequin closed this as completed May 8, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@malb malb

Labels
c: commutative algebra p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-4.4.2
Development

No branches or pull requests
2 participants
@burcin @malb