Skip to content

Conversation

@bevzzz
Copy link
Contributor

@bevzzz bevzzz commented Feb 27, 2025

See the last sentence of the paragraph detailing how to estimate the size of a multi-vector embedding:
image

This happens in both Chrome and Safari.

What's being changed:

docusaurus seems to treat * as a meta character for italicizing text regardless of its positioning, so some formulas become mangled.

Escaping them like so \* seems to do the trick.

Type of change:

  • Documentation updates (non-breaking change to fix/update documentation)

How Has This Been Tested?

Tested in a Docusaurus Playground:

  • navigate to tutorial/intro.md for raw markdown
  • open devtools (> icon), run dev task, and Open Preview

(sorry, I still haven't tried running this locally, just wanted to leave this PR as a note about sth I noticed)

  • GitHub action – automated build completed without errors
  • Local build - the site works as expected when running yarn start

docusaurus seems to treat `*` as meta characters for italicizing text and so the formulas become mangled.

Escaping them like so \* seems to do the trick.
Tested in a Docusaurus Playground (navigate to tutorial/intro.md): 
https://codesandbox.io/p/devbox/wild-dust-ltgrg3?workspaceId=ws_7ZwZBX3vvx2Q2XvrF5z9ud
Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@g-despot g-despot merged commit c5dcc4d into main Mar 31, 2025
6 checks passed
@g-despot g-despot deleted the fix/escape-asterisks branch March 31, 2025 09:56
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.

3 participants