Opened 5 years ago

Closed 3 years ago

Last modified 3 years ago

#23056 closed defect (invalid)

givaro does not pass self-tests on 32bit systems

Reported by: tmonteil Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: packages: standard Keywords: sdl
Cc: cpernet Merged in:
Authors: Reviewers:
Report Upstream: Reported upstream. Developers acknowledge bug. Work issues:
Branch: Commit:
Dependencies: #24214 Stopgaps:

Status badges

Description

See the attached build log givaro-4.0.2.p0.log and the corresponding failing test suite test-suite.log.

Attachments (2)

givaro-4.0.2.p0.log (210.3 KB) - added by tmonteil 5 years ago.
test-suite.log (351 bytes) - added by tmonteil 5 years ago.

Download all attachments as: .zip

Change History (14)

Changed 5 years ago by tmonteil

Changed 5 years ago by tmonteil

comment:1 Changed 5 years ago by tmonteil

  • Report Upstream changed from N/A to Reported upstream. No feedback yet.

comment:2 Changed 5 years ago by tmonteil

In case, the difference between 94499109968480 and 94499109902944 is 2^16.

comment:3 Changed 5 years ago by cpernet

This was https://github.com/linbox-team/givaro/issues/39 which I had erroneously closed, as it did not show up for while. I reopened the issue. Note that on our CI build, the difference is 2^13 x 13, 2^15 or 5x2^14.

Last edited 5 years ago by cpernet (previous) (diff)

comment:4 follow-up: Changed 5 years ago by cpernet

  • Cc cpernet added

Note that the field that causes this bug ModularExtended<double> is not being used in sage.

Last edited 5 years ago by cpernet (previous) (diff)

comment:5 Changed 5 years ago by tmonteil

  • Report Upstream changed from Reported upstream. No feedback yet. to Reported upstream. Developers acknowledge bug.

comment:6 in reply to: ↑ 4 Changed 5 years ago by tmonteil

  • Priority changed from blocker to major

Replying to cpernet:

Note that the field that causes this bug ModularExtended<double> is not being used in sage.

Thanks for the hint, let me remove the blocker priority. But i guess it is still healthy to provide upstream packages well-tested and with their whole capabilities (also, we might use that field later in Sage).

comment:7 Changed 5 years ago by cpernet

  • Dependencies set to #24214

fixed in #24214

comment:8 Changed 4 years ago by chapoton

  • Milestone changed from sage-8.0 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

is this fixed now ? should we close this ticket ?

comment:9 Changed 4 years ago by cpernet

  • Status changed from needs_review to positive_review

I confirm that this ticket can be closed. Sorry for having forgotten to close when #24214 was fixed.

comment:10 Changed 4 years ago by tmonteil

I confirm that givaro 4.0.4 builds and pass self-tests on my 23bit VM.

comment:11 Changed 3 years ago by embray

  • Resolution set to invalid
  • Status changed from positive_review to closed

Presuming these are all correctly reviewed as either duplicate, invalid, or wontfix.

comment:12 Changed 3 years ago by tmonteil

  • Keywords sdl added
Note: See TracTickets for help on using tickets.