Opened 3 years ago
Last modified 12 months ago
#12482 new defect
isinstance failure on lazy imported class
Reported by: | nthiery | Owned by: | jason |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | misc | Keywords: | Cernay2012 |
Cc: | rishi, jason, rlm, rbradshaw | Merged in: | |
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by nthiery)
isinstance may fail on a lazy imported class, which can cause hard to track history-dependent failures (just been hit: #12483)::
sage: lazy_import('sage.rings.rational_field', 'RationalField') sage: isinstance(QQ, RationalField) False sage: isinstance(QQ, RationalField) True
Would there be a way to ensure that isinstance triggers the importing of the class *before* its use?
Change History (6)
comment:1 Changed 3 years ago by nthiery
- Description modified (diff)
comment:2 Changed 3 years ago by nthiery
- Keywords Cernay2012 added
comment:3 Changed 2 years ago by jdemeyer
- Milestone changed from sage-5.11 to sage-5.12
comment:4 Changed 18 months ago by vbraun_spam
- Milestone changed from sage-6.1 to sage-6.2
comment:5 Changed 15 months ago by vbraun_spam
- Milestone changed from sage-6.2 to sage-6.3
comment:6 Changed 12 months ago by vbraun_spam
- Milestone changed from sage-6.3 to sage-6.4
Note: See
TracTickets for help on using
tickets.