Opened 4 years ago

Closed 4 years ago

#15805 closed defect (fixed)

the q-integer 0 has no parent.

Reported by: chapoton Owned by:
Priority: minor Milestone: sage-6.2
Component: combinatorics Keywords: q-analogs
Cc: sage-combinat Merged in:
Authors: Travis Scrimshaw Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 80450c0 (Commits) Commit: 80450c0b9ab464dc06c38568f91f70be744f13df
Dependencies: Stopgaps:


For uniformity, all q-integers should belong to the polynomial ring in q over ZZ.

sage: from sage.combinat.q_analogues import q_int
sage: q_int(1).parent()
Univariate Polynomial Ring in q over Integer Ring
sage: q_int(0).parent()
AttributeError                            Traceback (most recent call last)
<ipython-input-3-f312dad40f56> in <module>()
----> 1 q_int(Integer(0)).parent()

AttributeError: 'int' object has no attribute 'parent'

Change History (5)

comment:1 Changed 4 years ago by tscrim

  • Authors set to Travis Scrimshaw
  • Branch set to public/combinat/q_int_parent-15805
  • Cc sage-combinat added
  • Commit set to d1ea9d8bfbba01fbeef4f4ae50a19127bfeea2a2
  • Status changed from new to needs_review

We should be consistent about this.

New commits:

d1ea9d8Added special case of q_int(0).

comment:2 Changed 4 years ago by git

  • Commit changed from d1ea9d8bfbba01fbeef4f4ae50a19127bfeea2a2 to 80450c0b9ab464dc06c38568f91f70be744f13df

Branch pushed to git repo; I updated commit sha1. New commits:

80450c0trac #15805 : small enhancement

comment:3 Changed 4 years ago by chapoton

Thanks, Travis.

This looks good to me. If you approve my small change, you can set a positive review

comment:4 Changed 4 years ago by tscrim

  • Reviewers set to Frédéric Chapoton
  • Status changed from needs_review to positive_review

Then positive review. Thanks Frederic.

comment:5 Changed 4 years ago by vbraun

  • Branch changed from public/combinat/q_int_parent-15805 to 80450c0b9ab464dc06c38568f91f70be744f13df
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.