Skip to content

serverless: add support for pptx export #353

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

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Conversation

viseshrp
Copy link
Collaborator

@viseshrp viseshrp commented Jul 16, 2025

Closes #326

@viseshrp viseshrp requested a review from Copilot July 16, 2025 16:03
Copilot

This comment was marked as outdated.

@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2025

Codecov Report

Attention: Patch coverage is 53.33333% with 14 lines in your changes missing coverage. Please review.

Project coverage is 80.43%. Comparing base (f71c77e) to head (73dd979).

Files with missing lines Patch % Lines
...ansys/dynamicreporting/core/serverless/template.py 25.00% 12 Missing ⚠️
src/ansys/dynamicreporting/core/serverless/adr.py 85.71% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #353      +/-   ##
==========================================
- Coverage   80.53%   80.43%   -0.11%     
==========================================
  Files          25       25              
  Lines        7163     7190      +27     
  Branches     1347     1351       +4     
==========================================
+ Hits         5769     5783      +14     
- Misses        976      989      +13     
  Partials      418      418              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@viseshrp viseshrp requested review from phchen95 and Copilot July 16, 2025 21:00
Copy link

@Copilot 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 support for exporting reports to PowerPoint (PPTX) format in the serverless dynamic reporting system. The implementation includes both template-level and ADR-level rendering methods for PPTX generation, along with comprehensive test coverage and documentation updates.

Key changes:

  • Adds render_pptx() method to PPTXLayout template class for direct PPTX rendering
  • Introduces render_report_as_pptx() method to the ADR singleton for convenient PPTX export
  • Updates dependency specification for python-pptx to ensure compatibility

Reviewed Changes

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

Show a summary per file
File Description
src/ansys/dynamicreporting/core/serverless/template.py Adds render_pptx() method to PPTXLayout class and improves HTML rendering logic
src/ansys/dynamicreporting/core/serverless/adr.py Implements render_report_as_pptx() method and adds validation for template loading
tests/serverless/test_adr.py Adds comprehensive test coverage for PPTX rendering functionality
tests/serverless/test_template.py Updates test variable names and enables previously disabled test
pyproject.toml Pins python-pptx dependency to specific version for compatibility
doc/source/serverless/templates.rst Documents PPTX rendering capabilities and usage examples
doc/source/serverless/instantiation.rst Adds note about resource cleanup best practices
Comments suppressed due to low confidence (1)

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.

serverless: enable pptx export
2 participants