Dan King

Results 218 comments of Dan King

cc: @daniel-goldstein is it possible to make the error more clear? The CI test job is a bit hard to read. Maybe the last line of the output should be...

@daniel-goldstein back to you

FWIW, I was encountering this issue when initializing asyncio *after* a loop was already running but before I created any tasks. Since 1.5.2, you can `nest_asnycio.apply()` outside of an event...

5.2.1 was never yanked but there’s enough subsequent releases that this issue now feels irrelevant. Closing. Thanks all

We're replacing this issue with [#13563](https://github.com/hail-is/hail/issues/13563) which contains a little less discussion but has some directions forward.

Waiting on google to explain why the latest version of the sheets library gives me 403s.

No explanation from Google, but I figured out how to make it work. TLDR: don't use the thing that seems like the obviously correct choice, use some obscure other approach.

@tpoterba I heavily modified this so best to get a second review.

I keep losing track of this because it's not on my CI queue. I've cloned the branch and PR'ed under my user so it shows up in my queue: https://github.com/hail-is/hail/pull/13586

Unearthing this old issue. I agree with John's assessment. We should: 1. Fix the docs for `n` to indicate it is the maximum number of returned strings, not the maximum...