Adam Weeden

Results 97 comments of Adam Weeden

> This line is 103 chars. I think it is a good example of being clean as-is > > ```python > mlb_only_cols = ['key_retro', 'key_bbref', 'key_fangraphs', 'mlb_played_first', 'mlb_played_last'] > ```...

Strange. Does yapf from the command line work? If so, then in your "Output" console on the bottom change the dropdown to Python, and see what it may be showing....

> Just an idea: [`black`](https://black.readthedocs.io/en/stable/index.html) --- you get zero customization, but it's stupid simple and its style is inoffensive to basically everyone. Integrates with all editors, and can be run...

The only reason I'd prefer numpydoc over Google is the addition of data types.

> I've seen variations on the Google format that specify that, e.g. for a toy example: > > > > ``` > > Args: > > df: DataFrame. A Pandas...

@schorrm @bdilday Do you have a preference?

That's a good point. I'm less concerned with using this as a weapon to use at PR time, and was more looking to fix what we have now (which I...

I was also looking at it less from a pybaseball developer's perspective, and more from a consumer of the package's perspective Standardized docstrings equate to an easier understanding and a...

Try `--file-download-options '{"saveAs": true}'`

It doesn't appear as if this is the intention of the nptyping library https://github.com/ramonhagenaars/nptyping/issues/34