Skip to content

Conversation

miike
Copy link
Contributor

@miike miike commented Oct 10, 2025

Adds a new Signals tutorial involving a demo travel website by creating attributes, groups and an intervention.

Draft as I may have to remove the Jupyter notebook unless there is a way we can easily include this in the docs site.

@miike miike requested review from Copilot and pif October 10, 2025 05:05
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for snowplow-docs failed.

Name Link
🔨 Latest commit 9918b37
🔍 Latest deploy log https://app.netlify.com/projects/snowplow-docs/deploys/68e8943592f90800088219e8

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive Signals tutorial involving a demo travel website, including documentation for creating attributes, groups, and interventions to personalize user experiences.

  • Creates a complete hands-on tutorial for implementing Snowplow Signals in a travel demo site
  • Provides step-by-step instructions for setting up attributes, groups, and services
  • Includes optional sections for personalizing agent responses and interventions

Reviewed Changes

Copilot reviewed 10 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tutorials/signals-bdp/meta.json Defines the tutorial metadata and structure
tutorials/signals-bdp/intro.md Introduction explaining prerequisites and tutorial objectives
tutorials/signals-bdp/setting-up.md Setup instructions for running the demo travel website
tutorials/signals-bdp/defining-attributes.md Guide for creating Signals attributes using Python SDK
tutorials/signals-bdp/signals.ipynb Jupyter notebook with complete attribute definitions and code
tutorials/signals-bdp/testing-attributes.md Instructions for verifying attribute functionality
tutorials/signals-bdp/personalizing-site.md Guide for implementing site personalization using Signals
tutorials/signals-bdp/personalizing-agent.md Optional section for personalizing chatbot responses
tutorials/signals-bdp/interventions.md Optional section for implementing triggered interventions
tutorials/signals-bdp/conclusion.md Tutorial wrap-up and next steps
Comments suppressed due to low confidence (2)

tutorials/signals-bdp/signals.ipynb:1

  • EntityProperty is used but not imported. The import statement at line 37 includes AtomicProperty, Event, EventProperty but not EntityProperty.
{

tutorials/signals-bdp/personalizing-site.md:1

  • Missing 'not' before 'take into account' - should be 'does not take into account'.
---

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


## Performing some events

In our notebook we defined a number of attributes that we can now test. Let's test two straighforward ones now.
Copy link

Copilot AI Oct 10, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'straighforward' to 'straightforward'.

Suggested change
In our notebook we defined a number of attributes that we can now test. Let's test two straighforward ones now.
In our notebook we defined a number of attributes that we can now test. Let's test two straightforward ones now.

Copilot uses AI. Check for mistakes.


## Next steps

Now that we can use behavior to define attributes and retrieve these using a service we'll being to use these attributes to personalize how content on the site is displayed.
Copy link

Copilot AI Oct 10, 2025

Choose a reason for hiding this comment

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

Corrected 'being' to 'begin' in the sentence about using attributes for personalization.

Suggested change
Now that we can use behavior to define attributes and retrieve these using a service we'll being to use these attributes to personalize how content on the site is displayed.
Now that we can use behavior to define attributes and retrieve these using a service we'll begin to use these attributes to personalize how content on the site is displayed.

Copilot uses AI. Check for mistakes.

Comment on lines +56 to +58
4. If you are using an agent setup set your

In order to use the agent you will need either a OpenAI API key (OPENAI_API_KEY) or an AWS Bearer token for Bedrock (AWS_BEARER_TOKEN_BEDROCK).
Copy link

Copilot AI Oct 10, 2025

Choose a reason for hiding this comment

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

Incomplete sentence - missing continuation after 'set your'.

Suggested change
4. If you are using an agent setup set your
In order to use the agent you will need either a OpenAI API key (OPENAI_API_KEY) or an AWS Bearer token for Bedrock (AWS_BEARER_TOKEN_BEDROCK).
4. If you are using an agent setup, set your agent credentials in the `.env` file:
In order to use the agent you will need either an OpenAI API key (`OPENAI_API_KEY`) or an AWS Bearer token for Bedrock (`AWS_BEARER_TOKEN_BEDROCK`).

Copilot uses AI. Check for mistakes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant