Телеграмм чат группы piterpy_meetup страница 2910

parrot.ru

Рейтинг популярных групп и каналов

В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

PiterPy Meetup

674 membersпожаловаться на группу

2020 March 10

S

Stan in PiterPy Meetup

Боброний

Давайте я ещё раз изложу свою позицию:

Я создал код функции с таким параметром, оно не сломалось
Я передал этот параметр в функцию, оно не сломалось
Я вызвал help(), оно не сломалось
И когда я вызываю inspect.signature, оно не должно ломаться

Передай два таких аргумента и верни их сумму, и я ещё раз подумаю)

источник

13:35пожаловаться #1

EM

Egor Molodik in PiterPy Meetup

Боброний

Давайте я ещё раз изложу свою позицию:

Я создал код функции с таким параметром, оно не сломалось
Я передал этот параметр в функцию, оно не сломалось
Я вызвал help(), оно не сломалось
И когда я вызываю inspect.signature, оно не должно ломаться

Не должно согласно чему?

источник

13:36пожаловаться #2

Б

Боброний in PiterPy Meetup

Не должно согласно чему?

Моему мнению же

источник

13:36пожаловаться #3

Б

Боброний in PiterPy Meetup

Передай два таких аргумента и верни их сумму, и я ещё раз подумаю)

Не очень понял, над чем ты хочешь подумать, но вот, держи:

def f():
    print(sum(locals().values()))


f.__code__ = f.__code__.replace(
    co_nlocals=2,
    co_argcount=2,
    co_varnames=('@', '&',)
)

f(**{'@': 1, '&': 2})  # 3

источник

13:38пожаловаться #4

S

Stan in PiterPy Meetup

Ну а я говорю о том, что locals по определению - это имена переменных и аргументов внутри функции. co_varnames - то же самое, только у inspect. В конце концов, тебе имя co_varnames говорит о том, что это имена переменных :) Соответственно, эти имена должны следовать некоторым правилам.

Если уж на то пошло, я бы сказал, что f(**{"a-b":1}) не должно работать, но тут надо хорошо подумать.

источник

13:39пожаловаться #5

Б

Боброний in PiterPy Meetup

Ну а я говорю о том, что locals по определению - это имена переменных и аргументов внутри функции. co_varnames - то же самое, только у inspect. В конце концов, тебе имя co_varnames говорит о том, что это имена переменных :) Соответственно, эти имена должны следовать некоторым правилам.

Если уж на то пошло, я бы сказал, что f(**{"a-b":1}) не должно работать, но тут надо хорошо подумать.

co_varnames это не то же самое)

источник

13:39пожаловаться #6

S

Stan in PiterPy Meetup

Боброний

co_varnames это не то же самое)

image_2020-03-10_13-40-47.png

мм?

источник

13:40пожаловаться #7

Б

Боброний in PiterPy Meetup

image_2020-03-10_13-40-47.png

мм?

А, да, пардон, то же самое

источник

13:41пожаловаться #8

Б

Боброний in PiterPy Meetup

Ну а я говорю о том, что locals по определению - это имена переменных и аргументов внутри функции. co_varnames - то же самое, только у inspect. В конце концов, тебе имя co_varnames говорит о том, что это имена переменных :) Соответственно, эти имена должны следовать некоторым правилам.

Если уж на то пошло, я бы сказал, что f(**{"a-b":1}) не должно работать, но тут надо хорошо подумать.

Но тем не менее, оно работает

источник

13:41пожаловаться #9

S

Stan in PiterPy Meetup

Боброний

Не очень понял, над чем ты хочешь подумать, но вот, держи:

def f():
    print(sum(locals().values()))


f.__code__ = f.__code__.replace(
    co_nlocals=2,
    co_argcount=2,
    co_varnames=('@', '&',)
)

f(**{'@': 1, '&': 2})  # 3

Тут ты хитришь) Готов поспорить, если я тебя попрошу написать таким образом функцию с одним аргументом, которая добавит к аргументу "/posts/" и сходит через urlopen по образованному урлу, чтобы забрать оттуда json, ты тоже будешь стучаться по ключу в locals. Но это приводит нас к моему предыдущему сообщению - что мешает тогда пользоваться kwargs или вообще передавать словарь как extra_data.

источник

13:43пожаловаться #10

Б

Боброний in PiterPy Meetup

Тут ты хитришь) Готов поспорить, если я тебя попрошу написать таким образом функцию с одним аргументом, которая добавит к аргументу "/posts/" и сходит через urlopen по образованному урлу, чтобы забрать оттуда json, ты тоже будешь стучаться по ключу в locals. Но это приводит нас к моему предыдущему сообщению - что мешает тогда пользоваться kwargs или вообще передавать словарь как extra_data.

Мне кажется, ты уходишь от темы

источник

13:43пожаловаться #11

Б

Боброний in PiterPy Meetup

Я тут не пытаюсь доказать, что это суперполезный и очень широкоприменимый кейс.

источник

13:44пожаловаться #12

S

Stan in PiterPy Meetup

Чтобы не распространяться дальше - я стою на том, что в locals и в co_varnames должны быть валидные имена, потому что этого требует их семантика. То, что сейчас в каких-то случаях туда можно передать шлак и оно не падает - это не доказательство того, что оно должно работать всегда. И то, что inspect.signature падает - это правильно.

источник

13:44пожаловаться #13

Б

Боброний in PiterPy Meetup

Я просто указываю на то, что в реализация inspect.signature какое-то своё видение на то, как должна выглядеть функция, которое отличается от реализации самой функции в питоне

источник

13:45пожаловаться #14

Б

Боброний in PiterPy Meetup

Чтобы не распространяться дальше - я стою на том, что в locals и в co_varnames должны быть валидные имена, потому что этого требует их семантика. То, что сейчас в каких-то случаях туда можно передать шлак и оно не падает - это не доказательство того, что оно должно работать всегда. И то, что inspect.signature падает - это правильно.

А я стою на том, что в таком случае это должно быть зашито в реализации функции при её создании, а не только в inspect

источник

13:46пожаловаться #15

Б

Боброний in PiterPy Meetup

Либо не зашито нигде

источник

13:46пожаловаться #16

S

Stan in PiterPy Meetup

Боброний

Я просто указываю на то, что в реализация inspect.signature какое-то своё видение на то, как должна выглядеть функция, которое отличается от реализации самой функции в питоне

Мы опять уходим в детали. но я не вижу в поведении signature ничего отличного от того, как Python видит функции синтаксически. Если на то пошло, то проблема была на этапе .replace, когда ты в co_varnames кладёшь невалидный идентификатор, но там это, скорее всего, обусловлено тем, что это NamedTuple, а не какой-нибудь его подкласс, который делает обширную валидацию.

источник

13:47пожаловаться #17

Б

Боброний in PiterPy Meetup

Мы опять уходим в детали. но я не вижу в поведении signature ничего отличного от того, как Python видит функции синтаксически. Если на то пошло, то проблема была на этапе .replace, когда ты в co_varnames кладёшь невалидный идентификатор, но там это, скорее всего, обусловлено тем, что это NamedTuple, а не какой-нибудь его подкласс, который делает обширную валидацию.

Нет, там есть проверки

источник

13:48пожаловаться #18

Б

Боброний in PiterPy Meetup

Мы опять уходим в детали. но я не вижу в поведении signature ничего отличного от того, как Python видит функции синтаксически. Если на то пошло, то проблема была на этапе .replace, когда ты в co_varnames кладёшь невалидный идентификатор, но там это, скорее всего, обусловлено тем, что это NamedTuple, а не какой-нибудь его подкласс, который делает обширную валидацию.

def f(): ...
f.__code__ = f.__code__.replace(
    co_argcount=2,
    co_varnames=()
)

Traceback (most recent call last):
  File "/Users/arseny.boikov/Library/Preferences/PyCharm2019.3/scratches/scratch_15.py", line 4, in <module>
    f.__code__ = f.__code__.replace(
ValueError: code: varnames is too small

источник

13:49пожаловаться #19

Б

Боброний in PiterPy Meetup

Нельзя просто вот так вот создать невалидный кусок кода

источник

13:50пожаловаться #20