I disagree - then you end up with to_years, to_months, to_weeks, to_days, to_hours, to_minutes, to_seconds, to_miliseconds...
-
-
Replying to @erincandescent @EmilyGorcenski and
It's redundant, and dividing by an hour is exactly how you'd do things mathematically
2 replies 0 retweets 0 likes -
Replying to @erincandescent @oshepherd and
Who cares about redundancy? Readability is better than "how you'd do things mathematically."
2 replies 0 retweets 0 likes -
As a maintainer I care about redundancy enormously. I work on CPUs which involve lots of redundancy by necessity and it fucking sucks.
1 reply 0 retweets 1 like -
Replying to @erincandescent @EmilyGorcenski and
Copy & pasted & slightly modified code is a a complete and utter pox and a continual source of bugs
2 replies 0 retweets 0 likes -
-
Replying to @EmilyGorcenski @oshepherd and
And notwithstanding, this very simple script has *three* separate libraries for handling time. stdlib, numpy, and pandas. So... /shrug
1 reply 0 retweets 0 likes -
I mean this seems to say more about numpy and pandas than anything else :(
1 reply 0 retweets 0 likes -
Replying to @erincandescent @oshepherd and
Did you not see the opening tweet of the thread?
1 reply 0 retweets 0 likes -
I did, but I still see no issue with using division to do it
2 replies 0 retweets 0 likes
I can name three other practices that this violates, but also the general point should be "why are we reimplementing existing stuff?"
-
-
Replying to @EmilyGorcenski @oshepherd and
And I know what the answer is and why it is. But explain to me why I'm not allowed to be mad about that?
1 reply 0 retweets 0 likes -
I have 0 issues with the complaint about subclasses of timedelta
0 replies 0 retweets 0 likes
End of conversation
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.