Ticket #4464 (closed task: fixed)

Opened 5 years ago

Last modified 4 years ago

devel/sage/sage/combinat/root_system/weyl_characters.py test (too) long

Reported by: jsp Owned by: mabshoff
Priority: minor Milestone: sage-3.4.1
Component: doctest coverage Keywords: test, long
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Jaap,

can you please open a ticket for that one. I suspect that we don't have anything tested via long or that the tests aren't properly marked "#long time". This one has popped up so often that we really ought to fix it once and for all since you hit it every time.

[...]

Cheers,

Michael

Change History

comment:1 follow-up: ↓ 2 Changed 5 years ago by bump

I think the file weyl_characters.py should be converted to Cython. The single most computationally intensive thing is the Freudenthal multiplicity formula (which is invoked a lot).

comment:2 in reply to: ↑ 1 Changed 5 years ago by mabshoff

  • Milestone changed from sage-3.2.1 to sage-3.2

Replying to bump:

I think the file weyl_characters.py should be converted to Cython. The single most computationally intensive thing is the Freudenthal multiplicity formula (which is invoked a lot).

Hi Dan,

that sounds fine to me, but we should do that on another ticket post Sage 3.2.

Cheers,

Michael

comment:3 Changed 5 years ago by was

  • Milestone changed from sage-3.2 to sage-3.2.1

comment:4 Changed 4 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from sage-3.4.2 to sage-3.4.1

#5721 fixed this.

Cheers,

Michael

Note: See TracTickets for help on using tickets.