Update xgboost_direct_marketing_sagemaker.ipynb

[roachmd]

I was getting an error with this line in conda_python3 :: hist = data.hist(bins=30, sharey=True, figsize=(10, 10))
I updated it to read hist = data.hist(30, True, figsize(10, 10))

[djarpin]

Thanks for your interest in this Amazon SageMaker example notebook!

Were you running this code from a SageMaker Notebook Instance or your own laptop/desktop/EC2 instance? The existing code runs successfully from a fresh SageMaker Notebook Instance. I'm wondering if this is an issue caused by running a different version of Pandas (SageMaker Notebook's version is 0.20.3). Can you confirm which version of Pandas you're running?

The code you've updated to doesn't appear to run for me, giving an error of:

name 'figsize' is not defined

Even when adjusted to:

hist = data.hist(30, True, figsize=(10, 10))

I still receive a KeyError as according to the documentation Pandas appears to be looking for "column" as the second positional argument, which doesn't accept a boolean as input.

[roachmd]

Running this in a new sagemaker notebook.
pd.version
0.20.3

That line I get "SyntaxError: positional argument follow keyword argument"

Now I just remove and retyped the line. Seems fine now. So, disreguard ;-)