search

aws / sagemaker-core

Apache License 2.0
5 stars 3 forks source link

Incorrect snake case names for AutoMLJob and AutoMLJobV2 #25

Closed pintaoz-aws closed 3 months ago

pintaoz-aws commented 4 months ago

For the API and parameter names of AutoMLJob and AutoMLJobV2, their snake case should be auto_ml_job, now we are using auto_m_l_job. Need to change the rule in convert_to_snake_case().

Boto3 doc: https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/sagemaker/client/create_auto_ml_job.html

The generated code:

# create the resource
response = client.create_auto_m_l_job(**operation_input_args)
logger.debug(f"Response: {response}")

return cls.get(auto_m_l_job_name=auto_m_l_job_name, session=session, region=region)
pintaoz-aws commented 3 months ago

Fixed in https://github.com/aws/sagemaker-core/pull/38