Release Notes Update#205
Conversation
jrobble
left a comment
There was a problem hiding this comment.
@jrobble made 12 comments.
Reviewable status: 0 of 7 files reviewed, 12 unresolved discussions (waiting on @brosenberg42 and @clnowacki).
docs/docs/Release-Notes.md line 194 at r2 (raw file):
Currently, the component can't generate an overall summary for the video unless the video is under 3 minutes. For clarity, let's say:
This component uses the VideoLLaMA3 model to generate a description of the activity in a video. It summarizes each 3-minute video segment and generates a timeline of events for each segment.
docs/site/Release-Notes/index.html line 315 at r2 (raw file):
This renders a bit strange:
By default, this component is configured to use the
facebook/nllb-200-3.3Bmodel No Language Left Behind (NLLB).
Not sure what you were going for. facebook/nllb-200-3.3B is not a link. If you just want to make it look like code you could use tick marks. Was the "model" part supposed to be the link? The "No Language Left Behind (NLLB)" part at the end seems extraneous.
The "nllb-200-distilled-1.3B" and "nllb-200-distilled-600M" links at the bottom look and work fine. Should we just link to "nllb-200-3.3B" in the same way?
docs/site/Release-Notes/index.html line 306 at r2 (raw file):
Rights in Data-General Clause 52.227-14, Alt. IV (DEC 2007). Copyright 2026 The MITRE Corporation. All Rights Reserved.</p> <h1 id="openmpf-100x">OpenMPF 10.0.x</h1> <h2>10.0.0: January 2026</h2>
Add a section for Media Selectors. Provide motivation. Mention JSON and CSV support. Link to the guide: https://github.com/openmpf/openmpf.github.io/blob/feat/release-notes-10/docs/docs/Media-Selectors-Guide.md
docs/site/Release-Notes/index.html line 329 at r2 (raw file):
</li> <li> <p>The component source can be found <a href="https://github.com/openmpf-components/blob/master/python/OrToolsSubjectComponent">here</a>.</p>
You're missing the 'openmpf" part in this link. Use this link: https://github.com/openmpf/openmpf-components/tree/master/python/OrToolsSubjectComponent
docs/site/Release-Notes/index.html line 402 at r2 (raw file):
</li> <li> <p>Refer to the <a href="https://github.com/openmpf-python-sdk/blob/master/detection/api">python component SDK</a> for details.</p>
You're missing "component" in this link. Use https://github.com/openmpf/openmpf-python-component-sdk/tree/master/detection/api
docs/site/Release-Notes/index.html line 405 at r2 (raw file):
</li> <li> <p>An example illustrating the use of this feature can be found in the python SDK test component <a href="https://github.com/openmpf-python-component-sdk/blob/master/detection/examples/PythonTestComponent/test_component.test_component.py">test_component.py</a>.</p>
You're missing "openmpf" in this link and the final test_component.test_component.py part is odd. Use this link: https://github.com/openmpf/openmpf-python-component-sdk/blob/master/detection/examples/PythonTestComponent/test_component/test_component.py
docs/docs/Release-Notes.md line 14 at r2 (raw file):
<h3>No Language Left Behind (NLLB) Translation Component</h3> The No Language Left Behind component is based on [Meta's No Language Left Behind Project](https://ai.meta.com/research/no-language-left-behind/). The component translates input text from a given source language to English. The source language can be provided as a job property, or be indicated in the detection properties from a feed-forward track. By default, this component is configured to use the **`facebook/nllb-200-3.3B` model** [No Language Left Behind (NLLB)](https://huggingface.co/models?search=facebook/nllb). This provides the high translation quality, but also requires significant hardware resources. To accommodate smaller deployment enviroments, this component can use smaller NLLB models, such as [nllb-200-distilled-1.3B](https://huggingface.co/facebook/nllb-200-distilled-1.3B) or [nllb-200-distilled-600M](https://huggingface.co/facebook/nllb-200-distilled-600M).
This section and the FastText section don't use bullets. The other ones do. Use them in all sections for consistency.
Audit logging has bullets, but only for lists. To be consistent, all lines/sections should be bulleted.
docs/docs/Release-Notes.md line 94 at r2 (raw file):
and a new method: `get_detections_from_all_video_tracks()`. - This functionality is experimental and currently supported only with the python component API.
Capitalize "Python".
docs/docs/Release-Notes.md line 96 at r2 (raw file):
- This functionality is experimental and currently supported only with the python component API. - Refer to the [python component SDK](https://github.com/openmpf-python-sdk/blob/master/detection/api) for details.
Please capitalize to "Python Component SDK" for consistency with other docs.
docs/docs/Release-Notes.md line 98 at r2 (raw file):
- Refer to the [python component SDK](https://github.com/openmpf-python-sdk/blob/master/detection/api) for details. - An example illustrating the use of this feature can be found in the python SDK test component [test_component.py](https://github.com/openmpf-python-component-sdk/blob/master/detection/examples/PythonTestComponent/test_component.test_component.py).
Capitalize "Python".
docs/docs/Release-Notes.md line 139 at r2 (raw file):
- [[#1921](https://github.com/openmpf/openmpf/issues/1921)] Add audit logging for TiesDb - [[#1979](https://github.com/openmpf/openmpf/issues/1979)] Add event ids, bucket keys, and URIs to audit logging - [[#1933](https://github.com/openmpf/openmpf/issues/1933)] Add Custom SSO support
This issue is out of order. The list should be increasing top to bottom.
docs/docs/Release-Notes.md line 166 at r2 (raw file):
<h3>Updates</h3> - [[#1911](https://github.com/openmpf/openmpf/issues/1911)] Update LlamaVideoSummarization to use TIMELINE_CHECK_ACCEPTABLE_THRESHOLD
Use tick marks around TIMELINE_CHECK_ACCEPTABLE_THRESHOLD.
|
Replace "medaURI" with |
jrobble
left a comment
There was a problem hiding this comment.
@jrobble reviewed 7 files and all commit messages.
Reviewable status: all files reviewed, 13 unresolved discussions (waiting on @brosenberg42 and @clnowacki).
jrobble
left a comment
There was a problem hiding this comment.
@jrobble reviewed 5 files and all commit messages, made 1 comment, and resolved 13 discussions.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @brosenberg42).
docs/site/Release-Notes/index.html line 306 at r2 (raw file):
Previously, jrobble (Jeff Robble) wrote…
Add a section for Media Selectors. Provide motivation. Mention JSON and CSV support. Link to the guide: https://github.com/openmpf/openmpf.github.io/blob/feat/release-notes-10/docs/docs/Media-Selectors-Guide.md
I updated this and pushed a commit.
Issues:
This change is