New submission from Windson Yang :
At the beginning of https://docs.python.org/3.7/library/io.html#io.RawIOBase,
we declared that
> Binary I/O (also called buffered I/O)
and
> Raw I/O (also called unbuffered I/O)
But we didn't mention if Text I/O use buffer or not which led t
Windson Yang added the comment:
I found the document is not that clear when I try to understand what happens
when Python read/write a file. I'm not sure who also needs this information. As
you said, It wouldn't help the user program in Python. However, make it more
clear maybe
New submission from Windson Yang :
I got two errors when I try to build python document with
make venv
make html
The first one is
> Since v2.0, Sphinx uses "index" as master_doc by default. Please add
> "master_doc = 'contents'" to your conf.py
Change by Windson Yang :
--
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue38125>
___
___
Python-bugs-list
New submission from Windson Yang :
bisect_left should be similar to bisect_right. However, the current implement
didn't reflect it. A little bit update for the bisect_left function could make
the user easy to understand their relation.
def bisect_left(a, x, lo=0, hi=None):
if l
Windson Yang added the comment:
Any update?
--
___
Python tracker
<https://bugs.python.org/issue36093>
___
___
Python-bugs-list mailing list
Unsubscribe:
Windson Yang added the comment:
Any update?
--
___
Python tracker
<https://bugs.python.org/issue36453>
___
___
Python-bugs-list mailing list
Unsubscribe:
Windson Yang added the comment:
I'm not sure it's a bug. When you write binary data to file (use BufferedIOBase
by default). It actually writes the data to a buffer. That is why tell() gets
out of sync. You can follow the instrument belolw. For instance, call flush()
after writ
Windson Yang added the comment:
I think we should mention it at the document, like in the tell() function.
--
___
Python tracker
<https://bugs.python.org/issue36
New submission from Windson Yang :
> PyBytes_FromStringAndSize(const char *v, Py_ssize_t len):
> Return a new bytes object with a copy of the string v as value and length len
> on success, and NULL on failure. If v is NULL, the contents of the bytes
> object are uninitialized.
Windson Yang added the comment:
Sure. Feel free to edit my PR.
--
___
Python tracker
<https://bugs.python.org/issue37073>
___
___
Python-bugs-list mailin
Change by Windson Yang :
--
keywords: +patch
pull_requests: +13600
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/13713
___
Python tracker
<https://bugs.python.org/issu
Change by Windson Yang :
--
pull_requests: +13602
pull_request: https://github.com/python/cpython/pull/13715
___
Python tracker
<https://bugs.python.org/issue37
Change by Windson Yang :
--
keywords: +patch
pull_requests: +13651
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/13765
___
Python tracker
<https://bugs.python.org/issu
Windson Yang added the comment:
I just create a PR for it.
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue37086>
___
___
Python-bug
Change by Windson Yang :
--
keywords: +patch
pull_requests: +13652
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/13768
___
Python tracker
<https://bugs.python.org/issu
Windson Yang added the comment:
Hi, Andrés Delfino. Are you still working on it?
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue15
Windson Yang added the comment:
Sorry, I forgot about this PR, I will update the patch depends on review soon :D
--
___
Python tracker
<https://bugs.python.org/issue29
Change by Windson Yang :
--
pull_requests: +14813
pull_request: https://github.com/python/cpython/pull/15064
___
Python tracker
<https://bugs.python.org/issue29
Windson Yang added the comment:
I just updated the PR
--
___
Python tracker
<https://bugs.python.org/issue29750>
___
___
Python-bugs-list mailing list
Unsub
Change by Windson Yang :
--
assignee: -> docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python
___
Python tracker
<https://bugs.python.org/issu
Windson Yang added the comment:
I get NameError for both versions in both 3.4.4, 3.5.2, 3.6.4, 3.7.1
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
I will fix this issue after we have consensus with the future of .pth file in
#33944
--
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
I will work on it if no one wants to create a PR for this next week.
--
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
I agreed with Serhiy. I also found the function decode the output with latin-1,
I think it will be better to use utf-8 instead.
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
If I understand correctly, this may be better.
Try to find the module name. If path is omitted or None, the list of directory
names given by sys.path is searched. If path is built-in or frozen modules,
return None instead.
BTW, the dosstring should also be
Windson Yang added the comment:
I can't reproduce on my laptop. Would you please upload a minimal example?
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/is
Windson Yang added the comment:
As Jonathan Gossage said, I think it may break some code to fix this issue,
maybe we could just add a warning on the document?
--
___
Python tracker
<https://bugs.python.org/issue35
Change by Windson Yang :
--
keywords: +patch
pull_requests: +10277
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue35325>
___
___
Py
Windson Yang added the comment:
Any ideas? Or I will create a PR in a week without 'CPython implementation
detail'
--
___
Python tracker
<https://bugs.python.o
Change by Windson Yang :
--
pull_requests: +10373
___
Python tracker
<https://bugs.python.org/issue35367>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Windson Yang :
--
keywords: +patch
pull_requests: +10374
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue35267>
___
___
Py
Windson Yang added the comment:
>From the docs
Changed in version 3.7: Hostname or IP address is matched by OpenSSL during
handshake. The function match_hostname() is no longer used. In case OpenSSL
refuses a hostname or IP address, the handshake is aborted early and a TLS
al
Change by Windson Yang :
--
keywords: +patch
pull_requests: +10494
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue35105>
___
___
Py
Windson Yang added the comment:
I agreed with @Raymond Hettinger, I will update the PR from your suggestion if
no other ideas in next week.
--
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
> The documentation should state that "pathname" will be None (not the empty
> string) for built-in and frozen modules in order to be in line with the
> implementation.
Both the "file" and "pathname" will be None for
Windson Yang added the comment:
I found that Requests library use urllib3 library which looks like ignore the
user info part (in request_context
https://github.com/urllib3/urllib3/blob/master/src/urllib3/poolmanager.py#L208).
Did I miss something or we should also ignore it
New submission from Windson Yang :
Most of the example in https://docs.python.org/3/library/dataclasses.html miss
code like
from dataclasses import dataclass, field
from typing import List
I think we should add this statement in the code snippet.
--
assignee: docs@python
components
Windson Yang added the comment:
I'm not sure if we should put
from dataclasses import dataclass
everywhere or we should put it just in the first example as I did in the PR.
--
___
Python tracker
<https://bugs.python.org/is
Windson Yang added the comment:
I reviewed the patch months ago, maybe we need a core developer review this
path?
--
___
Python tracker
<https://bugs.python.org/issue34
Windson Yang added the comment:
Why requests library didn't raise an error because urllib3 (the library
requests using) ignore the auth part right now
> Currently we expect our users to handle authentication headers themselves.
> It's unfortunate that we silently strip
Windson Yang added the comment:
I wonder if possible that c not in ASCIILETTERS when we get KeyError?
if c in ASCIILETTERS:
raise s.error('bad escape %s' % this, len(this))
--
nosy: +Windson Yang
___
Python track
Windson Yang added the comment:
This is not a bug, it would be better to submit your ideas to python-ideas
mail-list
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
Hello, jcrmatos. Maybe you can submit your idea to python-ideas mail-list.
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue35
Windson Yang added the comment:
IMHO, we don't need to add the option. We can return the smallest value from
the **table** instead of the code below.
if len(table) == 1:
return table[0][0]
[1] https://github.com/python/cpython/blob/master/Lib/statistics.py#L502
--
Windson Yang added the comment:
I only tested stats.mode() from scipy, data = 'BBAAC' should also return 'A'.
But in your code **return return Counter(seq).most_common(1)[0][0]** will
return B. Did I miss something?
--
___
Py
Windson Yang added the comment:
I think right now we can
> Change mode() to return the first tie instead of raising an exception. This
> is a behavior change but leaves you with the cleanest API going forward.
as well as
> Add a Deprecation warning to the current behavior of mo
Windson Yang added the comment:
>From the docs
>https://docs.python.org/3/library/socket.html#socket.socket.makefile, the
>default mode for makefile() is 'r' (only readable). In your example, just use
>S = s.makefile(mode='w') instead.
Windson Yang added the comment:
Hello, Zahash Z. May I ask what is your expected behavior for the return value
of __repr__() from PriorityQueue. I'm agreed with Stéphane Wirtel, I don't
think returning all the items would be a good idea, maybe we can improve it in
another way. Fo
New submission from Windson Yang :
AFAIK, the email address should support non-ASCII character (from
https://stackoverflow.com/questions/760150/can-an-email-address-contain-international-non-english-characters
and SMTPUTF8
option from
https://docs.python.org/3/library/smtplib.html
Windson Yang added the comment:
Btw, from the docs
https://docs.python.org/3/library/smtplib.html#smtplib.SMTP.sendmail
> msg may be a string containing characters in the ASCII range, or a byte
> string. A string is encoded to bytes using the ascii codec, and lone \r and
> \n c
Windson Yang added the comment:
IIUC, your script (using Sklearn/tensorflow) will cause an infinite loop when
building with pyinstaller. Would you mind upload an example script so I can try
to reproduce it?
--
nosy: +Windson Yang
___
Python
Windson Yang added the comment:
sls, are you working on this feature now?
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue29539>
___
___
Windson Yang added the comment:
Yes, Maxwell. I guess the docs are misleading, the code locate in
https://github.com/python/cpython/blob/master/Objects/weakrefobject.c#L748
if (callback == Py_None)
callback = NULL;
if (callback == NULL)
/* return existing weak reference
Windson Yang added the comment:
It looks to me the fix is easy, we just will return NULL and raise TypeError
when the callback is not callable, None, or NULL. I'm not an expert in C, but I
would love to create a PR for it if you don't
Windson Yang added the comment:
Just to be clear, as Raymond said, when we have two non-sortable objects (for
instance, two instances which their class didn't implement the __lt__ and
__gt__ methods), we should compare their repr() without sort() like now.
--
nosy: +Windson
Change by Windson Yang :
--
versions: +Python 2.7, Python 3.4, Python 3.5, Python 3.6, Python 3.7
___
Python tracker
<https://bugs.python.org/issue36230>
___
___
Windson Yang added the comment:
My point is careful about the non-sortable object. My mistake, this should be
an enhancement, not a bug.
--
versions: -Python 2.7, Python 3.4, Python 3.5, Python 3.6, Python 3.7
___
Python tracker
<ht
New submission from Windson Yang :
I think we should document the behavior as below, (maybe at
https://docs.python.org/3/reference/expressions.html#operator-precedence)
>>> 1 or 0 and 3
1
>>> 0 or 1 and 3
3
Please correct me if we already document it.
--
ass
Windson Yang added the comment:
Thank you Serhiy, we did document here:
> The expression x and y first evaluates x; if x is false, its value is
> returned; otherwise, y is evaluated and the resulting value is returned.
> The expression x or y first evaluates x; if x is true, its
Windson Yang added the comment:
SilentGhost, I think you give a great example to explain this behavior. If the
behavior is obvious to you, we can close this issue.
--
___
Python tracker
<https://bugs.python.org/issue36
Change by Windson Yang :
--
versions: +Python 2.7, Python 3.4, Python 3.5
___
Python tracker
<https://bugs.python.org/issue18697>
___
___
Python-bugs-list mailin
Windson Yang added the comment:
I agreed with @Matheus, it would be better than the current implementation
--
nosy: +Windson Yang
versions: +Python 3.6, Python 3.7 -Python 2.7, Python 3.4, Python 3.5
___
Python tracker
<https://bugs.python.
Change by Windson Yang :
--
versions: -Python 2.7
___
Python tracker
<https://bugs.python.org/issue18697>
___
___
Python-bugs-list mailing list
Unsubscribe:
Windson Yang added the comment:
I can't find the Note in the current document
--
nosy: +Windson Yang
versions: +Python 3.4, Python 3.5 -Python 3.7, Python 3.8
___
Python tracker
<https://bugs.python.org/is
Windson Yang added the comment:
Please ignore the last message, the docs locate in 3.4 and 3.5
--
___
Python tracker
<https://bugs.python.org/issue26
Windson Yang added the comment:
In the docs
https://docs.python.org/3/extending/newtypes.html#weak-reference-support points
out:
> /* Clear weakrefs first before calling any destructors */
So maybe we should also update the docs after fix this bug. Anyway, I will try
to understand/
Windson Yang added the comment:
The fixed looks easy, we call `PyObject_CallFinalizerFromDealloc` before
PyObject_ClearWeakRefs. But I can't come up with the use case for testing when
generator resurrects from `PyObject_CallFinalizer`. Any
Windson Yang added the comment:
I would like to work on this and make a PR.
--
nosy: +Windson Yang
type: -> enhancement
versions: +Python 3.7, Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python.org/issu
Windson Yang added the comment:
This looks interesting. Let me try to fix it.
--
nosy: +Windson Yang
versions: +Python 3.8, Python 3.9 -Python 3.6
___
Python tracker
<https://bugs.python.org/issue36
Windson Yang added the comment:
My base idea would be some unittests for the function like:
class ExtendPathBaseTests(unittest.TestCase):
def test_input_string(self):
path = 'path'
name = 'foo'
self.assertEqual('path', pkgutil.ext
New submission from Windson Yang :
Is it an expected behavior the get_importer function only returns the first
valid path_hook(importer) from sys.path_hooks?
def get_importer(path_item):
"""Retrieve a finder for the given path item
The returned find
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12795
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue14817>
___
___
Py
Windson Yang added the comment:
I added some tests in the PR. Actually, there are some tests for extend_path
already (see
https://github.com/python/cpython/blob/master/Lib/test/test_pkgutil.py#L235).
However, I didn't test every line of the code in the extend_path fun
New submission from Windson Yang :
> The tokenize() generator requires one argument, readline, which must be a
> callable object which provides the same interface as the io.IOBase.readline()
> method of file objects. Each call to the function should return one line of
> input as
Windson Yang added the comment:
I can find some example in the docs that didn't `import the correct module`
even in the first example. Should we add the `import` statement for all of them?
--
nosy: +Windson Yang
___
Python tracker
&
Windson Yang added the comment:
I agreed most of the documents won't need the change, but some documents like
https://docs.python.org/3/library/dataclasses.html#dataclasses.field didn't
mention we have to run `from typing import List` and I guess most developers
not familiar
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12816
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12817
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12818
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12819
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Change by Windson Yang :
--
pull_requests: +12820
___
Python tracker
<https://bugs.python.org/issue36682>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12821
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12826
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Windson Yang added the comment:
Yes, I can make a PR for it.
--
___
Python tracker
<https://bugs.python.org/issue36654>
___
___
Python-bugs-list mailin
Change by Windson Yang :
--
keywords: +patch
pull_requests: +12871
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue36654>
___
___
Py
New submission from Windson Yang :
We don't have the base example for priority queue in
https://docs.python.org/3.8/library/heapq.html#priority-queue-implementation-notes,
We can add something like:
> q = Q.PriorityQueue()
> q.put(10)
> q.put(1)
> q.put(5)
> while not
Windson Yang added the comment:
on macOS 10.14.4, I got `ValueError: offset must be a timedelta representing a
whole number of minutes, not datetime.timedelta(0, 29143).` I will do some
research to see why this happen.
--
nosy: +Windson Yang
Windson Yang added the comment:
Thanks, SilentGhost, you are right. I will leave this to a Windows expert
instead.
--
___
Python tracker
<https://bugs.python.org/issue36
Windson Yang added the comment:
> Maybe a line should be added in the documentation to prevent people using
> this as a validator without more check?
I don't expect uuid.UUID could be used as a validator myself, but I agreed we
can warn users in the documentation if lots of t
Windson Yang added the comment:
In your first case, *any positive index except 2 will work*, For example:
L = [0, 1, 2]
L[::1], *rest = "abcdef" # L became ['a']
or
L[::3], *rest = "abcdef" # L became ['a', 1, 2]
I found iff when you change the length
Windson Yang added the comment:
Another question will be are we going to replace the * in our source code in
the future? Since I found lots of our code use 'from xxx import *' pattern.
--
nosy: +Windson Yang
___
Python track
Windson Yang added the comment:
I found the issue located in
https://github.com/python/cpython/blob/master/Lib/email/_parseaddr.py#L277
elif self.field[self.pos] in '.@':
# email address is just an addrspec
# this isn't very efficient since we start over
se
Windson Yang added the comment:
Frome the answer from Alnitak
(https://stackoverflow.com/questions/12355858/how-many-symbol-can-be-in-an-email-address).
Maybe we should raise an error when the address has multiple @ in it.
--
___
Python tracker
Windson Yang added the comment:
Just to make sure, the expected behavior would be the items should not be
copied because of the permission and the error messages above, right?
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.
Windson Yang added the comment:
Hello, @Rémi, are you still working on this issue?
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue18
Windson Yang added the comment:
An easy fix would be
"Raise ValueError if paths contain (note: use contain instead of contains) both
absolute and relative pathnames or the path are on the different drives."
--
nosy: +Windson Yang
Change by Windson Yang :
--
keywords: +patch
pull_requests: +13211
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue30535>
___
___
Py
Windson Yang added the comment:
I created a PR for it. TBO, meta_path is not a good name since it doesn't
contain any *path* at all.
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/is
Windson Yang added the comment:
Where are the documents actually?
--
nosy: +Windson Yang
___
Python tracker
<https://bugs.python.org/issue9267>
___
___
Pytho
1 - 100 of 168 matches
Mail list logo
