Michael Chow

Results 251 comments of Michael Chow
trafficstars

Thanks for reporting. Digging a bit into dplyr, it seems like some it has careful handling of this case: * it runs the given operation on the empty data *...

Note also that the experimental behavior of summarize being able to return 0 or > 1 rows [is deprecated](https://github.com/tidyverse/dplyr/pull/6557) (and a new function tentatively called reframe will handle that behavior!)....

Thanks for reporting. Let me try adding some overloads quickly to the decorator being used, to see it can silence the warnings. siuba uses generic function dispatch, which python has...

I added a quick draft in this PR, but am still seeing the error, so need to dig a bit deeper..! If you have any sense for why it'd still...

@nathanjmcdougall it seems to be working now on this PR: https://github.com/machow/siuba/pull/466 , any chance you can double check? Should be able to install the PR using... ```python pip install git+https://github.com/machow/siuba.git@ux-type-overloads...

Thanks for digging into this! This is a tough one, since it seems like pylint is doing a poor job with this specific type check, and doesn't make full use...

@wch pointed out pylint seems to allow [plugins](https://pylint.pycqa.org/en/latest/development_guide/how_tos/transform_plugins.html). Not sure how they work, but could be another option..

I wonder if this could be related to quarto needing a latex install, or something similar? We regenerate the docs every CI run, and it seems to work okay, so...

Shoot, it looks like the issue is that we need to resize the browser to capture the table, but resizing sets the browser outerHeight (the full browser, including topbar etc..)....

Shoot -- do you mind posting your chrome version? I can look a bit more, since chrome is a fairly popular choice 😬