#11772: improving error reporting of random_matrix, and bug fixing
------------------------------+---------------------------------------------
Reporter: dimpase | Owner: jason, was
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: linear algebra | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Rob Beezer | Author:
Merged: | Dependencies:
------------------------------+---------------------------------------------
Comment(by dimpase):
Replying to [comment:9 rbeezer]:
> Replying to [comment:7 dimpase]:
>
> I agree with your reservations - this routine should have some
safeguards and could probably be cleaned-up quite a bit. I also find it
very useful, even as is. It can sometimes take a second or two to finish,
but with reasonable inputs, it seems to always finish, and produce useful
output.
>
> If you ask for {{{upper_bound=1}}}, then I think you get what you
deserve. ;-)
our servers don't deserve being trashed by 120 undergrads, running never
ending loops, executing seemingly innocent Sage commands. :)
>
> > I'd say this has either to be re-done, or scrapped...
>
> So I would definitely like to see this feature kept - in other words, I
would like to see the interface stay stable, with changes to the code.
I'd be happy with throwing an exception after some number of attempts.
>
> I'm taking a break from most Sage work for a few weeks to get caught up
on other things, so won't be able to address this immediately.
>
I've bitten the bullet and hopefully fixed all the shortcomings we
discussed, added throwing exceptions, etc. Please review soon. :-)
> Rob
>
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11772#comment:11>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
