transformers
rewrite _process_parameter_type in auto_docstring.py to improve usability
#42431
Merged
Overview
Commits
30
Changes
View On GitHub

rewrite _process_parameter_type in auto_docstring.py to improve usability #42431

Rocketknight1 merged 30 commits into huggingface:main from Yacklin:rewrite-_process_parameter_type-to-improve-usability
Yacklin
Yacklin rewrite to improve its usability
a9025ed9
Rocketknight1
HuggingFaceDocBuilderDev
Yacklin
Yacklin Yacklin force pushed from bf593b73 to f951d0c2 189 days ago
Yacklin
Yacklin
Yacklin
Yacklin Yacklin force pushed from f951d0c2 to 98d26eab 187 days ago
Rocketknight1
Rocketknight1 commented on 2025-12-01
Yacklin
Rocketknight1
Yacklin
Yacklin rewrite to improve its usability
f87cd5fb
Yacklin Clarify comment about function parameter elements
bf887c06
Yacklin Yacklin force pushed from 98d26eab to bf887c06 184 days ago
Yacklin Yacklin requested a review from Rocketknight1 Rocketknight1 184 days ago
cturan
Yacklin
Rocketknight1
Rocketknight1
Yacklin
Yacklin
Yacklin Update implementation of _process_parameter_type
4771d2a6
Yacklin rewrite to improve its usability
c3396171
Yacklin Clarify comment about function parameter elements
09149159
Yacklin Yacklin force pushed from bf887c06 to 09149159 184 days ago
Yacklin Merge branch 'rewrite-_process_parameter_type-to-improve-usability' o…
31a6a479
Yacklin reformat it a little bit
0a8e1d39
Yacklin Merge branch 'rewrite-_process_parameter_type-to-improve-usability' o…
02f68ba1
Yacklin reformat it a little bit
3e5244fc
Yacklin used a wrong ruff version..... this one should be good
bc14531d
Yacklin
Rocketknight1
Rocketknight1 commented on 2025-12-05
Yacklin update the string manipulation
840c915d
Yacklin Yacklin requested a review from Rocketknight1 Rocketknight1 182 days ago
Yacklin
Yacklin
Yacklin
Rocketknight1 Merge branch 'main' into rewrite-_process_parameter_type-to-improve-u…
ce95c957
Rocketknight1 Trying for more consistency
46a064b5
Rocketknight1 make fixup
f9c09997
Yacklin
Yacklin
Yacklin
Yacklin commented on 2025-12-11
Rocketknight1 Try another approach
a0bc6275
Yacklin
Rocketknight1
Rocketknight1 Don't include "None" in the out_str when we're already setting optional
ecb90008
Yacklin
Rocketknight1 Add some new-style types to GPT-J to see what happens
f6978527
Rocketknight1 Correct use of UnionType
58dca011
Rocketknight1 make fixup
c91f05ba
Rocketknight1 Add a little snarky comment about typing just because
1abc7165
Rocketknight1 Correctly return the same strings as the old function
e9c848ce
Rocketknight1 Drop unnecessary args
d45e7d56
Rocketknight1
Yacklin Remove redundant args information
faefabfe
Yacklin
Yacklin commented on 2025-12-15
Yacklin add one more elif statement to deal with the case when type hint is None
a7796838
Yacklin
Rocketknight1
Yacklin
Yacklin
Yacklin add if statement to handle the parameter with default value
0f351883
github-actions
github-actions
Rocketknight1
Rocketknight1 Revert GPT-J changes
4e5a4ad1
Rocketknight1 Merge remote-tracking branch 'origin/rewrite-_process_parameter_type-…
6f87cc4e
Rocketknight1 Merge branch 'main' into rewrite-_process_parameter_type-to-improve-u…
a5092e52
Rocketknight1
Rocketknight1 approved these changes on 2025-12-18
Rocketknight1 Trigger tests
e6c67bb3
Rocketknight1 Rocketknight1 merged 728f34c3 into main 170 days ago
Yacklin
Yacklin Yacklin deleted the rewrite-_process_parameter_type-to-improve-usability branch 170 days ago
Yacklin

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Terms Of Use Privacy Policy Refund Policy Impressum