Skip to content

Conversation

@anu3990
Copy link
Contributor

@anu3990 anu3990 commented Aug 15, 2025

No description provided.

Copilot AI review requested due to automatic review settings August 15, 2025 18:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive JSDoc documentation to the vector-util.js module to address missing documentation. The module provides utility functions for encoding and decoding vectors in MarkLogic 12 or higher environments.

  • Added namespace documentation for the vectorUtil module
  • Added JSDoc for the base64Encode function that converts float arrays to encoded strings
  • Added JSDoc for the base64Decode function that converts encoded strings back to float arrays

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

* helps reduce the amount of disk space and memory consumed by vectors.
* @method vectorUtil#base64Encode
* @since 3.7.0
* @param {float[]} vector - an array of float values
Copy link

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

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

The type annotation 'float[]' is not valid JSDoc syntax. Use 'number[]' instead, as JavaScript uses the number type for all numeric values including floats.

Suggested change
* @param {float[]} vector - an array of float values
* @param {number[]} vector - an array of float values

Copilot uses AI. Check for mistakes.
* @method vectorUtil#base64Decode
* @since 3.7.0
* @param {string} encodedVector - an encoded string value.
* @returns {float[]} an array of float values.
Copy link

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

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

The type annotation 'float[]' is not valid JSDoc syntax. Use 'number[]' instead, as JavaScript uses the number type for all numeric values including floats.

Suggested change
* @returns {float[]} an array of float values.
* @returns {number[]} an array of float values.

Copilot uses AI. Check for mistakes.
@anu3990 anu3990 merged commit 87c41d7 into marklogic:develop Aug 15, 2025
1 of 2 checks passed
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