-
Notifications
You must be signed in to change notification settings - Fork 704
Main to develop 20250324 #5641
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
Main to develop 20250324 #5641
Conversation
WalkthroughThe changes update the release notes by adding a new section for FiftyOne Enterprise version 2.7.1, dated March 24, 2025. This section documents a fix for vulnerability CVE-2025-29927 related to next.js. No modifications were made to the declarations of exported or public entities. Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
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.
Actionable comments posted: 1
🧹 Nitpick comments (1)
docs/source/user_guide/app.rst (1)
361-364: Clarify and refine default sidebar field visibility text
The updated text now explicitly states that by default only label fields (excluding heatmaps and semantic segmentations) are visible, which improves clarity for users configuring the sidebar. There is a minor grammatical issue in line 363 ("define a dataset's a default configuration") that should be corrected (remove the extra "a").
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
docs/source/brain.rst(1 hunks)docs/source/enterprise/data_lens.rst(1 hunks)docs/source/enterprise/data_quality.rst(1 hunks)docs/source/enterprise/query_performance.rst(1 hunks)docs/source/integrations/mosaic.rst(2 hunks)docs/source/integrations/pgvector.rst(1 hunks)docs/source/release-notes.rst(3 hunks)docs/source/user_guide/app.rst(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (8)
docs/source/brain.rst (1)
395-395: New Reference Label Added
The addition of the reference label.. _brain-optimizing-lassoing-performance:enhances in-document cross-referencing and navigability. Please verify that this label is unique and that all internal links referring to this section are correct.docs/source/integrations/pgvector.rst (2)
3-4: Improved Header Formatting
The header "Pgvector Vector Search Integration" now uses an underline with a consistent length ("==================================") that aligns with style guidelines and mirrors similar changes in related documents.
8-11: Enhanced Readability in Description
The description has been refactored by inserting a line break, which breaks up a long sentence and improves readability. This change makes it easier for users to digest the key points about the Pgvector extension.docs/source/enterprise/data_quality.rst (1)
1-7: Streamlined Content by Removing Version Note
The explicit version availability note has been removed, resulting in cleaner documentation. This is consistent with similar updates in other enterprise pages and helps focus on core functionality rather than version-specific details.docs/source/enterprise/query_performance.rst (1)
1-9: Concise Presentation of Query Performance
The removal of the explicit version availability note and subsequent minor reformatting results in a more concise presentation of the Query Performance feature. This update improves clarity and maintains consistency with related enterprise documentation.docs/source/enterprise/data_lens.rst (1)
8-10: Improved clarity in the feature description.
The revised phrasing using "that allows you to use FiftyOne" enhances the readability and maintains consistency with the documentation style. The removal of the explicit version availability note streamlines the content without losing essential information.docs/source/integrations/mosaic.rst (2)
4-4: Section header formatting updated.
The adjustment of the underline from a longer series of equals signs to a shorter version now better aligns with the title length, improving visual consistency in the documentation.
707-708: Minor formatting improvement in the code block.
Removing the extra newline after themosaic_index.add_to_index(embeddings, sample_ids)call improves the code block’s consistency without affecting functionality.
583976b to
308c372
Compare
What changes are proposed in this pull request?
Merge
maintodevelopafter v2.7.1 release (mostly release notes, some trailing docs)Summary by CodeRabbit
Documentation
Bug Fixes