Skip to content
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

fix anyscale template utm tracking #50236

Conversation

chris-ray-zhang
Copy link
Contributor

@chris-ray-zhang chris-ray-zhang commented Feb 5, 2025

Verified fixes locally:

Screenshot 2025-02-04 at 5 35 20 PM Screenshot 2025-02-04 at 5 35 10 PM Screenshot 2025-02-04 at 5 32 22 PM Screenshot 2025-02-04 at 5 32 15 PM Screenshot 2025-02-04 at 5 31 56 PM Screenshot 2025-02-04 at 5 31 49 PM

Why are these changes needed?

Markdown URLs with "&" incorrectly rendered to "&amp", which is breaking utm tracking

e.g. Clicking on "Run quickstart" on https://docs.ray.io/en/latest/data/examples/pytorch_resnet_batch_prediction.html renders "https://www.anyscale.com/ray-on-anyscale?utm_source=ray_docs&utm_medium=docs&utm_campaign=image-classification-batch-inference-with-pytorch&redirectTo=/v2/template-preview/ray-data-batch-image-classification"

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

@chris-ray-zhang chris-ray-zhang force-pushed the chris/fixanyscaletemplateutmtracking branch from 84f8aa5 to 1a3e6f5 Compare February 5, 2025 01:39
Copy link
Contributor Author

@chris-ray-zhang chris-ray-zhang left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@GeneDer GeneDer left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -9,7 +9,10 @@
"source": [
"# Image Classification Batch Inference with PyTorch\n",
"\n",
"[![try-anyscale-quickstart-image-classification-batch-inference-with-pytorch](../../_static/img/run-quickstart-anyscale.svg)](https://www.anyscale.com/ray-on-anyscale?utm_source=ray_docs&utm_medium=docs&utm_campaign=image-classification-batch-inference-with-pytorch&redirectTo=/v2/template-preview/ray-data-batch-image-classification)\n",
"<a href=\"https://www.anyscale.com/ray-on-anyscale?utm_source=ray_docs&utm_source=ray_docs&utm_medium=docs&utm_campaign=image-classification-batch-inference-with-pytorch&redirectTo=/v2/template-preview/ray-data-batch-image-classification\">\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

utm_source=ray_docs is repeated twice 🙃, but I guess it doesn't affect the functionality

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good call out. thanks. fixed

@chris-ray-zhang chris-ray-zhang force-pushed the chris/fixanyscaletemplateutmtracking branch from 42b5d2d to 80ab392 Compare February 5, 2025 07:03
@chris-ray-zhang chris-ray-zhang added the go add ONLY when ready to merge, run all tests label Feb 5, 2025
Copy link
Contributor

@angelinalg angelinalg left a comment

Choose a reason for hiding this comment

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

stamp

@pcmoritz pcmoritz enabled auto-merge (squash) February 5, 2025 22:42
@pcmoritz pcmoritz disabled auto-merge February 5, 2025 22:42
@pcmoritz pcmoritz merged commit a3d6bf8 into ray-project:master Feb 5, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants