Skip to content

Conversation

@PierreDillard
Copy link
Collaborator

Added a minimal Node.js & Python examples to create and run a FilterSession.

@Copilot Copilot AI review requested due to automatic review settings September 29, 2025 09:25
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 Node.js and Python code examples to the existing C-only filter session tutorial documentation. The changes create multi-language tabs showing equivalent functionality across all three programming languages for creating and using GPAC FilterSession objects.

  • Adds tabbed code examples for Node.js and Python alongside existing C examples
  • Demonstrates creating FilterSession objects, loading filters, and running sessions in all three languages
  • Includes complete sample code implementations showing a full filter pipeline workflow

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
mkdocs.yml Enables tabbed code blocks extension for multi-language documentation
docs/Developers/tutorials/filter-session-intro.md Adds Node.js and Python examples throughout the FilterSession tutorial

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

=== "NodeJS"

```javascript
const gpac = require(path/to/gpac/share/nodejs);
Copy link

Copilot AI Sep 29, 2025

Choose a reason for hiding this comment

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

Missing quotes around the path in the require statement. Should be require('path/to/gpac/share/nodejs')

Suggested change
const gpac = require(path/to/gpac/share/nodejs);
const gpac = require('path/to/gpac/share/nodejs');

Copilot uses AI. Check for mistakes.

```javascript

const gpac = require(path/to/gpac/share/nodejs);
Copy link

Copilot AI Sep 29, 2025

Choose a reason for hiding this comment

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

Missing quotes around the path in the require statement. Should be require('path/to/gpac/share/nodejs')

Suggested change
const gpac = require(path/to/gpac/share/nodejs);
const gpac = require('path/to/gpac/share/nodejs');

Copilot uses AI. Check for mistakes.
dst_filter.set_source(writegen_filter);


session.run();
Copy link

Copilot AI Sep 29, 2025

Choose a reason for hiding this comment

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

The variable err is referenced but not assigned from the session.run() call. Should be const err = session.run();

Suggested change
session.run();
const err = session.run();

Copilot uses AI. Check for mistakes.
@rbouqueau
Copy link
Member

Loving this, great addition !

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