-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Contribution branch #2339
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
base: main
Are you sure you want to change the base?
Contribution branch #2339
Conversation
- Add --stats CLI flag to display comprehensive conversion metrics - Show document processing statistics (success rate, throughput, timing) - Display detailed pipeline operation timings for performance analysis - Automatically enable profiling when stats are requested - Support both single document and batch processing analytics - Include unit tests for the new functionality - Add comprehensive documentation and demo examples This feature helps users identify processing bottlenecks, optimize workflows, and benchmark performance across different systems.
❌ DCO Check Failed Hi @ManeeGit, your pull request has failed the Developer Certificate of Origin (DCO) check. This repository supports remediation commits, so you can fix this without rewriting history — but you must follow the required message format. 🛠 Quick Fix: Add a remediation commitRun this command: git commit --allow-empty -s -m "DCO Remediation Commit for Maneesh Malepati <[email protected]>
I, Maneesh Malepati <[email protected]>, hereby add my Signed-off-by to this commit: 602223c57944ea66d6013523e44f011f37217150
I, Maneesh Malepati <[email protected]>, hereby add my Signed-off-by to this commit: 21504a300d92b20b49a532e6b96a1e02bdf0f7a0
I, Maneesh Malepati <[email protected]>, hereby add my Signed-off-by to this commit: 524212a45c423586e6a92d67d93d6ccbb7a56e54"
git push 🔧 Advanced: Sign off each commit directlyFor the latest commit: git commit --amend --signoff
git push --force-with-lease For multiple commits: git rebase --signoff origin/main
git push --force-with-lease More info: DCO check report |
Related Documentation Checked 2 published document(s). No updates required. You have 5 draft document(s). Publish docs to keep them always up-to-date |
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🔴 Enforce conventional commitThis rule is failing.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
@@ -1,3 +1,9 @@ | |||
## [Unreleased] |
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.
Please do not touch the CHANGELOG. This will be done by the CI/CD automatically.
@@ -0,0 +1,32 @@ | |||
# Example Contribution |
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.
Please remove this. We have similar steps outlined in the CONTRIBUTING file. If you would like to propose changes there, please do it in a standalone PR.
@@ -0,0 +1,91 @@ | |||
#!/usr/bin/env python3 |
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.
We don't need an example for the CLI. Please remove.
@@ -0,0 +1,107 @@ | |||
# Performance Statistics Feature |
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.
We won't add a standalone page for this feature. Please rework it and add it as an addition to advanced_options.md
.
The section could be about the performance statistics in general and a subsection (shorter than this) for how to use it in the CLI.
@ManeeGit Please see the changes requested above. Also, make sure to sign-off your commits. You can apply the remediation commit, but this must be the last one of the contribution. |
@ManeeGit any update on the comments above? We would like to quickly finalize this work, or we will close it as stale. |
Checklist: