This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2016年09月14日 19:17 by belopolsky, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 928 | merged | cheryl.sabella, 2017年03月31日 15:39 | |
| PR 915 | cheryl.sabella, 2017年04月01日 20:00 | ||
| PR 3953 | merged | python-dev, 2017年10月11日 13:30 | |
| Messages (13) | |||
|---|---|---|---|
| msg276478 - (view) | Author: Alexander Belopolsky (belopolsky) * (Python committer) | Date: 2016年09月14日 19:17 | |
The time module defines timezone, altzone, tzname and daylight constants that store information about system timezone. This information is derived from the timezone rules that are in effect at the module loading time, but may be incorrect for the times in the past. The documentation should warn the user about this and recommend using the tm_zone and tm_gmtoff attributes in the result of time.localtime() to obtain timezone information. See #22799 and msg148208. |
|||
| msg290945 - (view) | Author: Cheryl Sabella (cheryl.sabella) * (Python committer) | Date: 2017年03月31日 22:53 | |
We were discussing changing the docstring on the pull request and I had additional questions, so I am moving them here for discussion: I've been looking at pydoc math and pydoc time (and the the module.c source). Before I go down the wrong path, I'm not sure I fully understand your comment - "(Also, markup can be improved in the time module docstring. Compare pydoc time and pydoc math.)" The time one is more vebose in the description and it lists the Variables and Functions in the text and then defines them with markup further down. Are you saying the variables and functions should be removed from being written in text within the description? Thanks. |
|||
| msg290946 - (view) | Author: Alexander Belopolsky (belopolsky) * (Python committer) | Date: 2017年03月31日 22:59 | |
The time and math modules are probably the oldest Python modules, but math have seen more development recently, so it should serve as a good model for how things should be organized. Yes, I believe re-listing module functions and constants in the module docstring is redundant and error-prone. |
|||
| msg290947 - (view) | Author: Alexander Belopolsky (belopolsky) * (Python committer) | Date: 2017年03月31日 23:11 | |
A question for Victor: Should we split the "Constants" section into "Clock ID constants" and "Timezone constants"? (See PR 928.) |
|||
| msg290950 - (view) | Author: STINNER Victor (vstinner) * (Python committer) | Date: 2017年03月31日 23:30 | |
IMHO it's worth it to have a dedicated section for CLOCK_xxx constants. It would allow to mention which functions use them. |
|||
| msg290974 - (view) | Author: Cheryl Sabella (cheryl.sabella) * (Python committer) | Date: 2017年04月01日 09:37 | |
The datetime module has a separate section for strptime and strftime. Any interest in splitting those functions into their own section? |
|||
| msg290975 - (view) | Author: Cheryl Sabella (cheryl.sabella) * (Python committer) | Date: 2017年04月01日 09:41 | |
Also, back to the original title of saying timezone, altzone, etc are deprecated - there are examples in the datetime docs that use those constants. Should that be updated to tm_zone and tm_gmtoff? |
|||
| msg290983 - (view) | Author: Cheryl Sabella (cheryl.sabella) * (Python committer) | Date: 2017年04月01日 12:31 | |
I think I may need to apologize. I'm new to this and was looking through the open docs issues and saw this one. Just now, I noticed that the 'assigned to' was filled in. I had only been looking at tickets that didn't have a pull request and not looking at the other values. I am so sorry if I started working on this when I shouldn't have and if I've caused more problems instead of helping. |
|||
| msg290999 - (view) | Author: Alexander Belopolsky (belopolsky) * (Python committer) | Date: 2017年04月01日 17:28 | |
Cheryl, There is no need to apologize. I assigned this issue to myself when I created it because I had a few ideas about refactoring the time module documentation and thought I would get to it soon. As often happens other priorities interfered. Contributions are welcome regardless of the "Assigned To" status on the issue. In fact, assigned issues may get a quicker response from the assignee than unassigned ones. When looking for simple issues to work on, look for an "easy" keyword. I did not mark this issue as "easy" because I planned to rearrange the module documentation in addition to adding a deprecation note. |
|||
| msg291000 - (view) | Author: Alexander Belopolsky (belopolsky) * (Python committer) | Date: 2017年04月01日 17:32 | |
> The datetime module .. Yes, the datetime module documentation can be improved, but let's keep this issue focused on the time module. Please open a new issue for the datetime module improvements. |
|||
| msg304133 - (view) | Author: Berker Peksag (berker.peksag) * (Python committer) | Date: 2017年10月11日 13:29 | |
New changeset 703ff381ffa946c23e7e25b0ae93a636a2607a40 by Berker Peksag (Cheryl Sabella) in branch 'master': bpo-28157: Improvements for the time module documentation (GH-928) https://github.com/python/cpython/commit/703ff381ffa946c23e7e25b0ae93a636a2607a40 |
|||
| msg304137 - (view) | Author: Berker Peksag (berker.peksag) * (Python committer) | Date: 2017年10月11日 13:50 | |
New changeset 764969a4b9ed7c6d2adbc04269f9b4fa392a1eed by Berker Peksag (Miss Islington (bot)) in branch '3.6': [3.6] bpo-28157: Improvements for the time module documentation (GH-928) https://github.com/python/cpython/commit/764969a4b9ed7c6d2adbc04269f9b4fa392a1eed |
|||
| msg304138 - (view) | Author: Berker Peksag (berker.peksag) * (Python committer) | Date: 2017年10月11日 13:51 | |
This issue can be closed now. Thank you, everyone! |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:36 | admin | set | github: 72344 |
| 2017年10月11日 13:51:47 | berker.peksag | set | status: open -> closed type: enhancement messages: + msg304138 resolution: fixed stage: patch review -> resolved |
| 2017年10月11日 13:50:24 | berker.peksag | set | messages: + msg304137 |
| 2017年10月11日 13:30:34 | python-dev | set | keywords:
+ patch pull_requests: + pull_request3928 |
| 2017年10月11日 13:29:16 | berker.peksag | set | nosy:
+ berker.peksag messages: + msg304133 |
| 2017年04月01日 20:00:38 | cheryl.sabella | set | pull_requests: + pull_request1134 |
| 2017年04月01日 17:34:38 | belopolsky | set | keywords: - 3.2regression |
| 2017年04月01日 17:32:41 | belopolsky | set | messages: + msg291000 |
| 2017年04月01日 17:28:27 | belopolsky | set | keywords:
+ 3.2regression messages: + msg290999 |
| 2017年04月01日 12:31:19 | cheryl.sabella | set | messages: + msg290983 |
| 2017年04月01日 09:41:17 | cheryl.sabella | set | messages: + msg290975 |
| 2017年04月01日 09:37:30 | cheryl.sabella | set | messages: + msg290974 |
| 2017年03月31日 23:30:56 | vstinner | set | messages: + msg290950 |
| 2017年03月31日 23:11:11 | belopolsky | set | messages: + msg290947 |
| 2017年03月31日 22:59:13 | belopolsky | set | nosy:
+ mark.dickinson, vstinner messages: + msg290946 |
| 2017年03月31日 22:53:42 | cheryl.sabella | set | nosy:
+ cheryl.sabella messages: + msg290945 |
| 2017年03月31日 17:02:48 | Mariatta | set | stage: needs patch -> patch review versions: + Python 3.7 |
| 2017年03月31日 15:39:01 | cheryl.sabella | set | pull_requests: + pull_request825 |
| 2016年09月15日 22:58:42 | belopolsky | link | issue15941 superseder |
| 2016年09月14日 19:17:21 | belopolsky | link | issue22799 superseder |
| 2016年09月14日 19:17:06 | belopolsky | create | |