Skip to content

Update MemryX section documentation#22711

Closed
abinila4 wants to merge 2 commits intoblakeblackshear:devfrom
memryx:memryx_docs_update
Closed

Update MemryX section documentation#22711
abinila4 wants to merge 2 commits intoblakeblackshear:devfrom
memryx:memryx_docs_update

Conversation

@abinila4
Copy link
Copy Markdown
Contributor

@abinila4 abinila4 commented Mar 31, 2026

Proposed change

This PR updates the MemryX section of the documentation to point users to SDK 2.1 resources and adds guidance for compiling custom models, as the current MemryX setup requires SDK 2.1, even if newer SDK releases become available in the future.

Changes included:

  • update MemryX SDK links to the 2.1 documentation
  • add a custom model compilation section, including a compiler example command

Will merging this PR into dev also update the public docs at https://docs.frigate.video/, or is a separate docs PR needed for a different branch or repo?

These changes are documentation-only and do not modify Frigate runtime behavior.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code
  • Documentation Update

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to discussion with maintainers (required for large/pinned features):

AI disclosure

  • No AI tools were used in this PR.
  • AI tools were used in this PR. Details below:

AI tool(s) used (e.g., Claude, Copilot, ChatGPT, Cursor):

How AI was used (e.g., code generation, code review, debugging, documentation):

Extent of AI involvement (e.g., generated entire implementation, assisted with specific functions, suggested fixes):

Human oversight: Describe what manual review, testing, and validation you performed on the AI-generated portions.

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I can explain every line of code in this PR if asked.
  • UI changes including text have used i18n keys and have been added to the en locale.
  • The code has been formatted using Ruff (ruff format frigate)

@github-actions
Copy link
Copy Markdown

PR template validation failed

This PR was automatically closed because the description does not follow the pull request template.

Issues found:

  • Missing section: ## AI disclosure

Please update your PR description to include all required sections from the template, then reopen this PR.

If you used an AI tool to generate this PR, please see our contributing guidelines for details.

@github-actions github-actions bot closed this Mar 31, 2026
@NickM-27
Copy link
Copy Markdown
Collaborator

You should just make a PR to master anyway for docs updates, we will sync to dev

@abinila4
Copy link
Copy Markdown
Contributor Author

@NickM-27, Thank you so much.

@abinila4
Copy link
Copy Markdown
Contributor Author

abinila4 commented Mar 31, 2026

You should just make a PR to master anyway for docs updates, we will sync to dev

@NickM-27, thank you so much. As this PR is closed, I created a new PR for the master branch. Thank you again.

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.

2 participants