-
Couldn't load subscription status.
- Fork 5
Add new articles in Python via .NET sections #498
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
Conversation
fix: update installation guide for Aspose.PDF for Rust via C++
…/export-import-json/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
…/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
feat: Add articles in Python/Acroform section
…/modifing-form/_index.md Co-authored-by: Copilot <[email protected]>
…/_index.md Co-authored-by: Copilot <[email protected]>
Renamed 'export-import-json' documentation to 'import-export-form-data' for clarity. Updated the sitemap change frequency for the modifying form documentation from weekly to monthly.
…/modifying-form/_index.md Co-authored-by: Copilot <[email protected]>
…/import-export-form-data/_index.md Co-authored-by: Copilot <[email protected]>
…/fill-form/_index.md Co-authored-by: Copilot <[email protected]>
Updated code examples to use explicit 'data_dir' and 'work_dir' variables for constructing input and output PDF file paths, replacing previous usage of 'self.dataDir'. This improves clarity and consistency in file path management across documentation for modifying and removing PDF forms.
Removed trailing whitespace and corrected minor formatting issues in the extract-form and import-export-form-data documentation. Updated section title to 'Extract Form Fields to JSON Document' for clarity.
…questions and answers
feat: Add FAQ section for Aspose.PDF for Python via .NET with common …
Co-authored-by: Andriy Andruhovski <[email protected]>
Co-authored-by: Andriy Andruhovski <[email protected]>
Co-authored-by: Andriy Andruhovski <[email protected]>
…mages-from-pdf-file/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md Co-authored-by: Andriy Andruhovski <[email protected]>
feat: Added articles to Working with Images (Python)
fix: updated articles in Conversion section (Python)
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.
Pull Request Overview
This pull request adds new articles in Python via .NET sections, enhancing the documentation with comprehensive code examples and updated content. The changes include adding new release notes for version 25.5, creating several new conversion articles, expanding existing documentation, and adding new functionality examples.
- Significant content addition to the "What's new" section with detailed code examples for PDF functionality
- Addition of new file format conversion documentation (PDF/x formats, OFD, LaTeX)
- Enhancement of existing conversion articles with more detailed examples and improved code structure
- Addition of new image manipulation and PDF layer functionality documentation
Reviewed Changes
Copilot reviewed 24 out of 26 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| en/python-net/whatsnew/_index.md | Comprehensive update with new version release notes and detailed Python code examples |
| en/python-net/faq/_index.md | New FAQ page with structured questions and answers about Aspose.PDF capabilities |
| en/python-net/converting/convert-pdfx-to-pdf/_index.md | New article for converting PDF/x formats to standard PDF |
| en/python-net/converting/convert-pdf-to-pdfx/_index.md | New comprehensive article for converting PDF to various PDF/x formats |
| en/python-net/converting/convert-pdf-to-word/_index.md | Updated with improved code examples and structure |
| en/python-net/converting/convert-pdf-to-powerpoint/_index.md | Enhanced with additional conversion options and examples |
| en/python-net/converting/convert-pdf-to-other-files/_index.md | Expanded with new format support including Markdown and MobiXML |
| en/python-net/converting/convert-other-files-to-pdf/_index.md | Updated with new OFD and LaTeX conversion examples |
| en/python-net/converting/convert-images-format-to-pdf/_index.md | Enhanced with additional image format support and improved examples |
| en/python-net/converting/convert-html-to-pdf/_index.md | Significantly expanded with multiple HTML conversion scenarios |
| en/python-net/converting/convert-pdf-to-excel/_index.md | Updated with improved code structure and additional format support |
| en/python-net/converting/convert-pdf-to-images-format/_index.md | Enhanced with better code organization and additional examples |
| en/python-net/converting/_index.md | Updated index with new conversion article references |
| en/python-net/advanced-operations/working-with-pages/adding-headers-and-footers-of-pdf-file/_index.md | Enhanced with new header/footer examples including tables and LaTeX |
| en/python-net/advanced-operations/working-with-images/ | Multiple new articles for image manipulation including search, replace, and extract functionality |
| en/python-net/advanced-operations/working-with-documents/working-with-layers/_index.md | New comprehensive article for PDF layer manipulation |
Comments suppressed due to low confidence (1)
en/python-net/converting/convert-images-format-to-pdf/_index.md:229
- Missing page assignment after document.pages.add(). The code creates a page but doesn't assign it to a variable, then tries to use methods on an undefined 'page' variable in the next lines.
page = document.pages.add()
rectangle = apdf.Rectangle(0, 0, 595, 842, True) # A4 size in points
page.add_image(path_infile, rectangle)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
en/python-net/advanced-operations/working-with-documents/working-with-layers/_index.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
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.
Ok
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.
Pull Request Overview
Copilot reviewed 20 out of 22 changed files in this pull request and generated 6 comments.
Comments suppressed due to low confidence (1)
en/python-net/converting/convert-pdf-to-other-files/_index.md:1
- The function uses
data_dirbut this variable is not defined. It should beself.data_dirto match the pattern used elsewhere in the codebase.
---
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| path_infile = path.join(data_dir, infile) | ||
| path_outfile = path.join(data_dir, "python", outfile) | ||
| path_template = path.join(data_dir, template) | ||
| path_temp_file = path.join(data_dir, "temp.html") |
Copilot
AI
Oct 1, 2025
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.
The function uses data_dir but this variable is not defined. It should be self.data_dir to match the pattern used elsewhere in the codebase.
| path_infile = path.join(data_dir, infile) | |
| path_outfile = path.join(data_dir, "python", outfile) | |
| path_template = path.join(data_dir, template) | |
| path_temp_file = path.join(data_dir, "temp.html") | |
| path_infile = path.join(self.data_dir, infile) | |
| path_outfile = path.join(self.data_dir, "python", outfile) | |
| path_template = path.join(self.data_dir, template) | |
| path_temp_file = path.join(self.data_dir, "temp.html") |
| image_names = list(document.pages[1].resources.images.names) | ||
|
|
||
| # Push identity matrix to stack | ||
| graphics_state.append(Matrix23(1, 0, 0, 1, 0, 0)) |
Copilot
AI
Oct 1, 2025
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.
The Matrix23 class is used but not imported or defined. This should likely be ap.Matrix or the appropriate matrix class from the aspose.pdf module.
No description provided.