Limit PyPI deployments

[d33bs]

Description

This PR helps address package deployment issues by limiting the amount of data transfered.

What is the nature of your change?

• Bug fix (fixes an issue).
• Enhancement (adds functionality).
• Breaking change (fix or feature that would cause existing functionality to not work as expected).
• This change requires a documentation update.

Checklist

Please ensure that all boxes are checked before indicating that a pull request is ready for review.

• I have read the CONTRIBUTING.md guidelines.
• My code follows the style guidelines of this project.
• 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.
• New and existing unit tests pass locally with my changes.
• I have added tests that prove my fix is effective or that my feature works.
• I have deleted all non-relevant text in this pull request template.

Summary by CodeRabbit

• Chores
  • Optimized Python source distribution by removing non-essential files and directories to reduce package size.

[coderabbitai]

Warning

Review limit reached

@d33bs, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 46 minutes and 37 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2f27ba7c-3919-4c83-aa92-8a7cb8596920

📥 Commits

Reviewing files that changed from the base of the PR and between 1228eb7 and 42308fc.

📒 Files selected for processing (1)

• MANIFEST.in

📝 Walkthrough

Walkthrough

MANIFEST.in is updated with packaging rules to exclude test data directories, documentation, example directories, and parquet files from Python source distributions, reducing distribution size and removing non-essential content.

Changes

Source Distribution Packaging

Layer / File(s)	Summary
Manifest packaging rules MANIFEST.in	Six lines added to MANIFEST.in to prune tests/data, docs, example, and examplehuman_warehouse_with_images directories, plus exclude all *.parquet files from source distributions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A rabbit hops through packaging rules,
Tidying distribution pools,
Tests and docs we bid farewell,
Smaller wheels now serve so well!
*~ CodeRabbit* 🌟

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Limit PyPI deployments' is directly related to the main change: updating MANIFEST.in to exclude non-essential files and directories to reduce package size, which aligns with limiting deployment data transfer.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

MANIFEST.in (1)

5-5: MANIFEST.in: exclude *.parquet and prune examplehuman_warehouse_with_images

• MANIFEST.in line 5 (exclude *.parquet): no .parquet files exist anywhere in the repo (outside .git), so the non-recursive nature won’t currently matter; keep only if you expect future parquet assets under subdirectories (otherwise it’s effectively redundant).
• MANIFEST.in line 6 (prune examplehuman_warehouse_with_images): that directory isn’t present anywhere in the repo and appears to be a potentially stale/typo path; verify intent and remove/correct if needed.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@MANIFEST.in` at line 5, Remove or update the two stale MANIFEST.in entries:
either delete the "exclude *.parquet" line (or change it to a recursive pattern
like "global-exclude *.parquet") only if you don't expect parquet assets in
subdirectories, and remove or correct the "prune
examplehuman_warehouse_with_images" entry if that directory does not exist (or
replace it with the intended directory name). Verify intent and commit the
minimal change: remove redundant entries if not needed, or make the
patterns/paths accurate.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@MANIFEST.in`:
- Line 6: The prune directive "prune examplehuman_warehouse_with_images" in
MANIFEST.in is stale because no such directory exists; remove that line or
replace it with the correct existing path if you intended to exclude a real
directory (e.g., the actual example directory name), ensuring the directive
exactly matches the target directory name; update MANIFEST.in so that only valid
prune/include/exclude targets remain and run packaging checks to confirm no
missing/unused entries.

---

Nitpick comments:
In `@MANIFEST.in`:
- Line 5: Remove or update the two stale MANIFEST.in entries: either delete the
"exclude *.parquet" line (or change it to a recursive pattern like
"global-exclude *.parquet") only if you don't expect parquet assets in
subdirectories, and remove or correct the "prune
examplehuman_warehouse_with_images" entry if that directory does not exist (or
replace it with the intended directory name). Verify intent and commit the
minimal change: remove redundant entries if not needed, or make the
patterns/paths accurate.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ea3d1afe-3fbe-4f3d-98eb-e2322675fb65

📥 Commits

Reviewing files that changed from the base of the PR and between b0f3907 and 1228eb7.

📒 Files selected for processing (1)

• MANIFEST.in

[d33bs]

Thanks @gwaybio !