-
Notifications
You must be signed in to change notification settings - Fork 1
Adding a new Signals tutorial with a demo site #1450
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.
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
base: main
Are you sure you want to change the base?
Conversation
❌ Deploy Preview for snowplow-docs failed.
|
There was a problem hiding this 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. |
Copilot
AI
Oct 10, 2025
There was a problem hiding this comment.
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'.
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. |
Copilot
AI
Oct 10, 2025
There was a problem hiding this comment.
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.
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.
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). |
Copilot
AI
Oct 10, 2025
There was a problem hiding this comment.
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'.
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.
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.