pandas-dev / pandas

Flexible and powerful data analysis / manipulation library for Python, providing labeled data structures similar to R data.frame objects, statistical functions, and much more
https://pandas.pydata.org
BSD 3-Clause "New" or "Revised" License
43.41k stars 17.83k forks source link

DOC: Fixed validate_title_capitalization warnings in most recent what… #59146

Closed kajor3k closed 1 month ago

kajor3k commented 3 months ago

…snew doc files. Sorted exceptions list alphabetically, for better maintainability, proposed name change from CAPITALIZATION_EXCEPTIONS to CAPITALIZATION_EXCLUSIONS. (#32550)

Aloqeely commented 3 months ago

Thanks for the PR! I don't think it's that important to update old release notes' capitalization, updating other parts of the documentation is probably more important.

But, this is still appreciated!

kajor3k commented 3 months ago

Thanks for the PR! I don't think it's that important to update old release notes' capitalization, updating other parts of the documentation is probably more important.

But, this is still appreciated!

Thank you! You're right - there are more urgent stuff to be done. I started with few files and small improvements throughout the file for two reasons. First, I wanted to create a PR which could be a template for other newcomers in case they would like to pickup some of the files. I saw that the approach in which all files are being fixed at once is not welcome due to the size of PR, But if nobody else will comment under original story(#32550) I will gradually pickup other files an clean them as well. The second reason is that this story doesn't make sense if it won't be finished up by turning on this validation on code_checks. But before it can be done, the validation needs to be flawless I see that the validation is running, but it is not configured where I was looking for it.

kajor3k commented 3 months ago

There's one error I don't know how to tackle: doc/source/whatsnew/v2.2.2.rst:15:Heading capitalization formatted incorrectly. Please correctly capitalize "Pandas 2.2.2 is now compatible with numpy 2.0" to "pandas 2.2.2 is now compatible with NumPy 2.0"

"pandas" in underscore is already in the list of exclusions, but here it is the beginning of the title so it is expected to be upperscore

Aloqeely commented 3 months ago

but here it is the beginning of the title so it is expected to be upperscore

From the citing webpage: "When using the project name pandas, please use it in lower case, even at the beginning of a sentence." -- Not sure why this rule was made but it's quite interesting

kajor3k commented 3 months ago

but here it is the beginning of the title so it is expected to be upperscore

From the citing webpage: "When using the project name pandas, please use it in lower case, even at the beginning of a sentence." -- Not sure why this rule was made but it's quite interesting

Alright, thanks for your response! Then it is clear. pandas should be underscore even at the beginning of the sentence

rhshadrach commented 2 months ago

Not sure why this rule was made but it's quite interesting

This is generally how names are - they are styled in upper/lower case independent of other grammatical rules, and always done in a consistent manner. For example, words may start with an uppercase letter but they don't appear in the middle unless it is a name, e.g. McClain. And you would never see "IPhone" at the beginning of a sentence, it's always "iPhone". You see this with other packages as well, e.g. scikit-learn.

github-actions[bot] commented 1 month ago

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

mroeschke commented 1 month ago

Thanks for the pull request, but it appears to have gone stale. If interested in continuing, please merge in the main branch, address any review comments and/or failing tests, and we can reopen.