Opened 7 years ago
Closed 5 years ago
#19996 closed enhancement (fixed)
Implement extracting symbolic coefficient of rational exponent
Reported by:  dkrenn  Owned by:  

Priority:  major  Milestone:  sage8.1 
Component:  symbolics  Keywords:  
Cc:  Merged in:  
Authors:  Ralf Stephan  Reviewers:  Travis Scrimshaw 
Report Upstream:  N/A  Work issues:  
Branch:  6d8d994 (Commits, GitHub, GitLab)  Commit:  6d8d9942ebfc4f9d8b83d52e0daca2146476cecf 
Dependencies:  #23820  Stopgaps: 
Description (last modified by )
sage: (x^(1/2)).coefficient(x, 1/3) sqrt(x)
It seems that 1/3
gets 0
(floor division, whyever).
EDIT: because the function takes an integer:
INPUT:  ``s``  expression  ``n``  integer, default 1
so it's rather an enhancement.
Reported as https://github.com/pynac/pynac/issues/216
Change History (9)
comment:1 Changed 6 years ago by
 Description modified (diff)
 Report Upstream changed from N/A to Reported upstream. Developers acknowledge bug.
comment:2 Changed 6 years ago by
 Description modified (diff)
 Report Upstream changed from Reported upstream. Developers acknowledge bug. to Reported upstream. Developers deny it's a bug.
 Summary changed from wrong result extracting symbolic coefficient (rational exponent) to Implement extracting symbolic coefficient of rational exponent
 Type changed from defect to enhancement
comment:3 Changed 5 years ago by
 Branch set to u/rws/implement_extracting_symbolic_coefficient_of_rational_exponent
comment:4 Changed 5 years ago by
 Commit set to 484142a16b5efe58366424eb5e9dc9247b7fde8e
 Dependencies set to pynac0.7.11
 Milestone changed from sage7.1 to sage8.1
 Report Upstream changed from Reported upstream. Developers deny it's a bug. to Fixed upstream, in a later stable release.
comment:5 Changed 5 years ago by
 Dependencies changed from pynac0.7.11 to #23820
 Report Upstream changed from Fixed upstream, in a later stable release. to N/A
comment:6 Changed 5 years ago by
 Commit changed from 484142a16b5efe58366424eb5e9dc9247b7fde8e to 6d8d9942ebfc4f9d8b83d52e0daca2146476cecf
Branch pushed to git repo; I updated commit sha1. New commits:
6d8d994  Merge branch 'develop' into t/19996/implement_extracting_symbolic_coefficient_of_rational_exponent

comment:7 Changed 5 years ago by
 Status changed from new to needs_review
comment:8 Changed 5 years ago by
 Reviewers set to Travis Scrimshaw
 Status changed from needs_review to positive_review
LGTM.
comment:9 Changed 5 years ago by
 Branch changed from u/rws/implement_extracting_symbolic_coefficient_of_rational_exponent to 6d8d9942ebfc4f9d8b83d52e0daca2146476cecf
 Resolution set to fixed
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
This branch needs pynac0.7.11+ and allows any coefficient. There is different behaviour in this doctest:
New commits:
19996: allow any n in ex.coefficient(x,n)