Modify ↓

Ticket #214 (new enhancement)

Opened 13 months ago

Last modified 13 months ago

allow Strings for all UTCDateTime args/kwargs

Reported by: megies Owned by:
Priority: minor Component: ObsPy library
Keywords: Cc:

Description

I vote for making all starttime/endtime/min_datetime/.. type args and kwargs accept Strings in addition to UTCDateTime objects (and of course note this in the docstrings). Sometimes we already have this and its pretty annoying in a quick interactive session to run into errors and then have to import UTCDateTime and make such an object manually...

Attachments

Change History

comment:1 Changed 13 months ago by barsch

-1 - mainly for performance reasons and it does not follow the KISS principle

we have to introduce in each method using UTCDateTime as arguments additional logic for type checking just for the lazy user - and in this case even worse for the lazy developer :) This will blow up the code and hurts the readability of the source code.

To much "auto-magic" and IMHO not a part of a basic library.

comment:2 Changed 13 months ago by krischer

Using Traits would make this possible and (I think) very easy to implement across all modules.

View ↑

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will be changed from (none). Next status will be 'new'
The owner will be changed from (none) to anonymous. Next status will be 'assigned'
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.