Opened 9 years ago

Closed 9 years ago

#10964 closed defect (fixed)

bug with conjugate in symbolic ring

Reported by: pierre Owned by: burcin
Priority: critical Milestone: sage-4.7.1
Component: symbolics Keywords:
Cc: kcrisman, burcin, jpflori, rbk Merged in: sage-4.7.1.alpha4
Authors: Richard Kreckel, Burcin Erocal Reviewers: Karl-Dieter Crisman
Report Upstream: Fixed upstream, in a later stable release. Work issues:
Branch: Commit:
Dependencies: #11415 Stopgaps:

Description (last modified by kcrisman)

this is wrong: 

sage: z= I*sqrt(-3)
sage: z.conjugate()
-I*sqrt(-3)

Apply trac_10964-conjugate_doctest.patch​

Attachments (1)

trac_10964-conjugate_doctest.patch​ (915 bytes) - added by burcin 9 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 follow-up: Changed 9 years ago by kcrisman

  • Description modified (diff)
  • Report Upstream changed from N/A to Not yet reported upstream; Will do shortly.

This is probably followup to #8775. Certainly it is a Ginac issue, isn't it? (We'll eventually want to confirm this and report upstream.)

comment:2 in reply to: ↑ 1 Changed 9 years ago by kcrisman

This is probably followup to #8775.

Meaning that #8775 fixed the issue for basic functions, but apparently it doesn't propagate somehow to products.

comment:3 Changed 9 years ago by burcin

  • Cc rbk added
  • Milestone set to sage-4.7.1
  • Priority changed from major to critical
  • Report Upstream changed from Not yet reported upstream; Will do shortly. to None of the above - read trac for reasoning.

I sent an email about this to ginac-devel. It didn't show up yet, so I can't link to the thread. I can include a fix in the next pynac release if there is a fix in the next couple of days.

comment:4 Changed 9 years ago by kcrisman

  • Report Upstream changed from None of the above - read trac for reasoning. to Reported upstream. Little or no feedback.

Here is the ​beginning of the thread. No feedback so far.

comment:5 Changed 9 years ago by rbk

  • Report Upstream changed from Reported upstream. Little or no feedback. to Fixed upstream, in a later stable release.

Changed 9 years ago by burcin

comment:6 Changed 9 years ago by burcin

  • Authors set to Richard Kreckel, Burcin Erocal
  • Dependencies set to #11415
  • Description modified (diff)
  • Status changed from new to needs_review

There is a pynac package which fixes this at #11415. attachment:trac_10964-conjugate_doctest.patch​ adds doctests to the library.

comment:7 Changed 9 years ago by kcrisman

  • Description modified (diff)
  • Reviewers set to Karl-Dieter Crisman
  • Status changed from needs_review to positive_review

Positive review.

Apply trac_10964-conjugate_doctest.patch​

comment:8 Changed 9 years ago by kcrisman

  • Description modified (diff)

comment:9 Changed 9 years ago by jdemeyer

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