Python wrapper для местного api

0
21 комментарий
Написать комментарий...
Аккаунт удален

Комментарий недоступен

Ответить
Развернуть ветку
Сергей Сергеев
Автор
Ответить
Развернуть ветку
Аккаунт удален

Комментарий недоступен

Ответить
Развернуть ветку
Аккаунт удален

Комментарий недоступен

Ответить
Развернуть ветку
Водный Даниль

да будет срач! Отступы - табы, выравнивание - только пробелы. Тут не доеебешься.

Ответить
Развернуть ветку
Структурный холод

Вообще ты заморочился, мощно всё сделал. Круто! Можно немного оптимизировать код в плане строк и сделать на примере vk_api: api.method(parameter='foo'), и уже библиотека делает всё остальное. Я еще сделаю пуш скоро, пару вещей оптимизирую.

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Буду рад =)

Ответить
Развернуть ветку
Структурный холод

У тебя репозиторий Readonly, не могу коммит сделать

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Сорян, сейчас исправлю

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Скажи ник, я тогда в колабы тебя добавлю

Ответить
Развернуть ветку
Структурный холод

peterder72

Ответить
Развернуть ветку
Сергей Сергеев
Автор
Ответить
Развернуть ветку
why'

Посмотри мой реквест

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Круто, что добавил логер и зарефакторил код, но параллелить 3 запроса в секунду особого смысла не вижу, а вот читабельность кода сильно усложняет. Замерджил, но асинхронность конечно оверкил.

Ответить
Развернуть ветку
why'

Дело не столько в параллельности сколько в возможности иметь неблокирующий код.

Функцию с лимитом надо бы убрать и добавить semaphore по-хорошему, как-то не вспомнил

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Я сейчас чутка потестирую и залью это всё дело на PyPi

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Попытался запустить примерчик с гита, но не получилось. Говорит что

TypeError: _request() got an unexpected keyword argument 'files'

Типа files в

async with session.post(url, headers=self.headers, data=payload, files=files)

лишний

Ответить
Развернуть ветку
why'

data=files тогда видимо вместо payload в таком случае

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Примерно так и было. Починил и обновил репу

Ответить
Развернуть ветку
Victor Poluksht

Что сделает?

Ответить
Развернуть ветку
Сергей Сергеев
Автор

Да, точно

Ответить
Развернуть ветку
Читать все 21 комментарий
null