Ticket #12594 (closed defect: fixed)

Opened 15 months ago

Last modified 15 months ago

fix some trivial noise issues on some architecture with hidden markov models

Reported by: was Owned by: amhou
Priority: trivial Milestone: sage-5.0
Component: statistics Keywords:
Cc: Work issues:
Report Upstream: N/A Reviewers: John Cremona
Authors: William Stein Merged in: sage-5.0.beta7
Dependencies: Stopgaps:

Description (last modified by jdemeyer) (diff)

From John Cremona:

Here's a test failure with sage-5.0.beta5 which I don't think I have seen before. Build from scratch on 32-bit ubuntu, gcc 4.4.3.

john@john-laptop%./sage -t  --long -force_lib
"devel/sage/sage/stats/hmm/chmm.pyx"
sage -t --long -force_lib "devel/sage/sage/stats/hmm/chmm.pyx"
**********************************************************************
File "/home/john/sage-5.0.beta5/devel/sage/sage/stats/hmm/chmm.pyx", line 865:
   sage: m.baum_welch([-2,-1,.1,0.1])
Expected:
   (4.534646052182663, 7)
Got:
   (4.534646052182662, 7)
**********************************************************************
File "/home/john/sage-5.0.beta5/devel/sage/sage/stats/hmm/chmm.pyx", line 867:
   sage: m.log_likelihood([-2,-1,.1,0.1])
Expected:
   4.534646052182663
Got:
   4.534646052182662
**********************************************************************
1 items had failures:
  2 of  21 in __main__.example_19
***Test Failed*** 2 failures.
For whitespace errors, see the file /home/john/.sage//tmp/chmm_19036.py
        [2.7 s]

Attachments

trac_12594.patch Download (808 bytes) - added by was 15 months ago.

Change History

Changed 15 months ago by was

comment:1 Changed 15 months ago by was

  • Status changed from new to needs_review

comment:2 Changed 15 months ago by cremona

  • Status changed from needs_review to positive_review
  • Reviewers set to John Cremona

Applied the the previously offending build of 5.0.beta5, this works fine.

comment:3 Changed 15 months ago by jdemeyer

  • Description modified (diff)
  • Authors set to William Stein

comment:4 Changed 15 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-5.0.beta7
Note: See TracTickets for help on using tickets.