Skip to content

Asset optimization experiment updates #1079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
laurachevalier4 wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Asset optimization experiment updates #1079

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
fd987ae
Update experiments examples for clarity and accuracy.
laurachevalier4 Jun 4, 2026
fdbcb90
Additional comment updates
laurachevalier4 Jun 4, 2026
96690af
Merge main into this branch.
laurachevalier4 Jun 4, 2026
068df37
Update graduation-related comments and print statements.
laurachevalier4 Jun 4, 2026
c1af3a4
Update casing.
laurachevalier4 Jun 4, 2026
a289f5a
Address comments
laurachevalier4 Jun 4, 2026
3e84930
Reorder the operations in the asset optimization experiment example
laurachevalier4 Jun 10, 2026
cc44a8f
Merge branch 'main' of https://github.com/googleads/google-ads-python…
laurachevalier4 Jun 10, 2026
417249c
Refactor asset operation creation to use helper function
laurachevalier4 Aug 4, 2026
ac1a28e
Merge branch 'main' into laura/asset-optimization-update
bobhancockg Aug 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 22 additions & 17 deletions examples/experiments/create_asset_optimization_experiment.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -135,9 +135,9 @@ def main(
asset_operation_1,
asset_operation_2,
experiment_operation,
*arm_operations,
asset_group_asset_operation_1,
asset_group_asset_operation_2,
*arm_operations,
]

response = googleads_service.mutate(
Expand All @@ -160,31 +160,28 @@ def main(
f" {response.mutate_operation_responses[2].experiment_result.resource_name}"
)
print(
"Created control arm:"
f" {response.mutate_operation_responses[3].experiment_arm_result.resource_name}"
"Created asset group asset for headline:"
f" {response.mutate_operation_responses[3].asset_group_asset_result.resource_name}"
)
print(
"Created treatment arm:"
f" {response.mutate_operation_responses[4].experiment_arm_result.resource_name}"
"Created asset group asset for image:"
f" {response.mutate_operation_responses[4].asset_group_asset_result.resource_name}"
)
print(
"Created asset group asset for headline:"
f" {response.mutate_operation_responses[5].asset_group_asset_result.resource_name}"
"Created control arm:"
f" {response.mutate_operation_responses[5].experiment_arm_result.resource_name}"
)
print(
"Created asset group asset for image:"
f" {response.mutate_operation_responses[6].asset_group_asset_result.resource_name}"
"Created treatment arm:"
f" {response.mutate_operation_responses[6].experiment_arm_result.resource_name}"
)


def create_text_asset_operation(
client: GoogleAdsClient, customer_id: str, temp_id: str, text: str
) -> MutateOperation:
"""Creates a mutate operation for a text asset."""
googleads_service = client.get_service("GoogleAdsService")
operation = client.get_type("MutateOperation")
asset = operation.asset_operation.create
asset.resource_name = googleads_service.asset_path(customer_id, temp_id)
operation, asset = _create_base_asset_operation(client, customer_id, temp_id)
asset.text_asset.text = text
return operation

Expand All @@ -197,10 +194,7 @@ def create_image_asset_operation(
name: str,
) -> MutateOperation:
"""Creates a mutate operation for an image asset."""
googleads_service = client.get_service("GoogleAdsService")
operation = client.get_type("MutateOperation")
asset = operation.asset_operation.create
asset.resource_name = googleads_service.asset_path(customer_id, temp_id)
operation, asset = _create_base_asset_operation(client, customer_id, temp_id)
asset.name = name
asset.type_ = client.enums.AssetTypeEnum.IMAGE
asset.image_asset.data = get_image_bytes_from_url(url)
Expand Down Expand Up @@ -289,6 +283,17 @@ def create_asset_group_asset_operation(
return operation


def _create_base_asset_operation(

@bobhancockg bobhancockg Aug 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion (Type Hinting): Tighten the return type annotation from Tuple[Any, Any] to Tuple[MutateOperation, Any] for improved type checking and IDE autocompletion.

def _create_base_asset_operation(
client: GoogleAdsClient, customer_id: str, temp_id: str
) -> Tuple[MutateOperation, Any]:

client: GoogleAdsClient, customer_id: str, temp_id: str
) -> Tuple[Any, Any]:
"""Helper to create a base asset mutate operation with a temp ID."""
googleads_service = client.get_service("GoogleAdsService")
operation = client.get_type("MutateOperation")
asset = operation.asset_operation.create
asset.resource_name = googleads_service.asset_path(customer_id, temp_id)
return operation, asset


if __name__ == "__main__":
parser = argparse.ArgumentParser(
description="Creates an OPTIMIZE_ASSETS experiment."
Expand Down