Skip to content

Wip 0.4.3#106

Merged
dpieski merged 9 commits intomainfrom
wip-0.4.3
Mar 5, 2026
Merged

Wip 0.4.3#106
dpieski merged 9 commits intomainfrom
wip-0.4.3

Conversation

@dpieski
Copy link
Copy Markdown
Contributor

@dpieski dpieski commented Mar 4, 2026

Description

Implements get_IFW (#100) — a new method that retrieves the full Image File Wrapper
and bulk-downloads all prosecution history documents (PDF preferred, DOCX fallback) into
either a ZIP archive or a flat directory. Returns a new IFWResult dataclass with the
PatentFileWrapper, the output path, and a downloaded_documents mapping of
document_identifier → filename so each downloaded file can be linked back to its
Document in the bag.

Also includes several related improvements made during development:

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this manually (if applicable)

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

Related Issues

Closes #100
Closes #99
Closes #101
Closes #102

Additional Notes

get_IFW supports two output modes controlled by as_zip: bool = True:

  • as_zip=True (default): all documents packaged into {app_no}_ifw.zip
  • as_zip=False: documents downloaded into {app_no}_ifw/ directory

Documents with no PDF/DOCX download URL (e.g. NPL references) are silently skipped.
A warning is issued only if a document has a URL but the download itself fails.

@dpieski dpieski merged commit ab8da2c into main Mar 5, 2026
7 checks passed
@dpieski dpieski deleted the wip-0.4.3 branch March 12, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant