Type checking in Python

At several points in pyeikon 1.0.0 it appears the API is not using Python best practices for type checking. For example, in time_series.to_datetime:
if type(date_value) is timedelta:
...
if type(date_value) in (datetime, date):
These should really be:
if isinstance(date_value, timedelta):
...
if isinstance(date_value, (datetime, date)):
As is, for example, use of a pandas Timestamp as the start_date or end_date parameter fails, even though pandas.Timestamp is a subclass of datetime. This is especially frustrating as it means that the start_date parameter can't be set to, say, df.index[0] from a df that was itself returned by the get_timeseries API.
Similarly, get_data and get_timeseries currently require the rics parameter to be a list of strings. These should really take any iterable. If you need a list (for example to guarantee order), you should cast it internally. As-is you cannot, for example, pass a generator, a pandas Index, or numpy array to get_data or get_timeseries which is similarly frustrating (you cannot, for example, directly take the values of a query for index members and pass them into get_data without casting).
Answers
-
@davidk that is a fair observation, thank you very much for the feedback. We will look into this.
0
Categories
- All Categories
- 6 AHS
- 36 Alpha
- 166 App Studio
- 6 Block Chain
- 4 Bot Platform
- 18 Connected Risk APIs
- 47 Data Fusion
- 33 Data Model Discovery
- 682 Datastream
- 1.4K DSS
- 613 Eikon COM
- 5.2K Eikon Data APIs
- 10 Electronic Trading
- Generic FIX
- 7 Local Bank Node API
- 3 Trading API
- 2.9K Elektron
- 1.4K EMA
- 248 ETA
- 552 WebSocket API
- 37 FX Venues
- 14 FX Market Data
- 1 FX Post Trade
- 1 FX Trading - Matching
- 12 FX Trading – RFQ Maker
- 5 Intelligent Tagging
- 2 Legal One
- 23 Messenger Bot
- 3 Messenger Side by Side
- 9 ONESOURCE
- 7 Indirect Tax
- 60 Open Calais
- 275 Open PermID
- 44 Entity Search
- 2 Org ID
- 1 PAM
- PAM - Logging
- 6 Product Insight
- Project Tracking
- ProView
- ProView Internal
- 22 RDMS
- 1.8K Refinitiv Data Platform
- 622 Refinitiv Data Platform Libraries
- 4 LSEG Due Diligence
- LSEG Due Diligence Portal API
- 4 Refinitiv Due Dilligence Centre
- Rose's Space
- 1.2K Screening
- 18 Qual-ID API
- 13 Screening Deployed
- 23 Screening Online
- 12 World-Check Customer Risk Screener
- 1K World-Check One
- 46 World-Check One Zero Footprint
- 45 Side by Side Integration API
- 2 Test Space
- 3 Thomson One Smart
- 10 TR Knowledge Graph
- 151 Transactions
- 143 REDI API
- 1.8K TREP APIs
- 4 CAT
- 26 DACS Station
- 121 Open DACS
- 1.1K RFA
- 104 UPA
- 191 TREP Infrastructure
- 228 TRKD
- 915 TRTH
- 5 Velocity Analytics
- 9 Wealth Management Web Services
- 84 Workspace SDK
- 11 Element Framework
- 5 Grid
- 18 World-Check Data File
- 1 Yield Book Analytics
- 46 中文论坛