Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues/#509 Новые поля #576

Merged
merged 16 commits into from
Mar 27, 2023
Merged

Issues/#509 Новые поля #576

merged 16 commits into from
Mar 27, 2023

Conversation

andrew097
Copy link
Contributor

Классу Playlist добавлены новые поля:

  • custom_wave - новая модель, описание плейлиста
  • pager - пагинация

Классу TrackShort добавлено поле original_index.
Классу Track добавлены новые поля.

Copy link
Owner

@MarshalX MarshalX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Спасибо большое! Сорян что сумбурно так прокомментил, нахожусь чутка в движении

yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/track.py Outdated Show resolved Hide resolved
yandex_music/track/track.py Show resolved Hide resolved
yandex_music/track/track.py Outdated Show resolved Hide resolved
yandex_music/playlist/playlist.py Show resolved Hide resolved
yandex_music/track/LyricsInfo.py Outdated Show resolved Hide resolved
Copy link
Owner

@MarshalX MarshalX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Привет, нашёл несколько косяков после переименования класса на R128

yandex_music/track/LyricsInfo.py Outdated Show resolved Hide resolved
yandex_music/playlist/customwave.py Outdated Show resolved Hide resolved
yandex_music/playlist/customwave.py Outdated Show resolved Hide resolved
Attributes:

title (:obj:`str`): Название плейлиста.
animation_url (:obj:`str`): Ссылка на анимированную обложку в json формате (кусочки PNG в BASE64, раскладка).
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кажется стоит дописать методы-сокращения и другие полезные методы для работы с этим полем. например download_animation. Если там base64 пнгшка, то метод аля save_animation с указанием filename: str куда сохранить файл на диск

yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/r128.py Outdated Show resolved Hide resolved
yandex_music/track/track.py Outdated Show resolved Hide resolved
yandex_music/track/track.py Outdated Show resolved Hide resolved
@andrew097
Copy link
Contributor Author

Добавил методы, но правильно собрать анимацию не хватает умений.
Как я понял, должна получиться анимация моей волны, на главном экране при входе через браузер.

@MarshalX
Copy link
Owner

Добавил методы, но правильно собрать анимацию не хватает умений. Как я понял, должна получиться анимация моей волны, на главном экране при входе через браузер.

А можешь прислать .txt файлом значение этого поля с анимацией?

@andrew097
Copy link
Contributor Author

andrew097 commented Dec 14, 2022

Добавил методы, но правильно собрать анимацию не хватает умений. Как я понял, должна получиться анимация моей волны, на главном экране при входе через браузер.

А можешь прислать .txt файлом значение этого поля с анимацией?

с 7-ой строки и до конца
20221212_120732_yamusic2.log

@MarshalX
Copy link
Owner

Добавил методы, но правильно собрать анимацию не хватает умений. Как я понял, должна получиться анимация моей волны, на главном экране при входе через браузер.

А можешь прислать .txt файлом значение этого поля с анимацией?

с 7-ой строки и до конца 20221212_120732_yamusic2.log

Это lottie анимация. Не будем ничего добавлять в ЯМ либу для работы с этим т.к. существуют отдельные либы для лотти. Просто нужно правильно задокументировать данное поле.

@andrew097
Copy link
Contributor Author

Добавил методы, но правильно собрать анимацию не хватает умений. Как я понял, должна получиться анимация моей волны, на главном экране при входе через браузер.

А можешь прислать .txt файлом значение этого поля с анимацией?

с 7-ой строки и до конца 20221212_120732_yamusic2.log

Это lottie анимация. Не будем ничего добавлять в ЯМ либу для работы с этим т.к. существуют отдельные либы для лотти. Просто нужно правильно задокументировать данное поле.

CustomWave.download_animation оставлю, чтобы json возвращать или тоже не стоит?

@MarshalX
Copy link
Owner

Добавил методы, но правильно собрать анимацию не хватает умений. Как я понял, должна получиться анимация моей волны, на главном экране при входе через браузер.

А можешь прислать .txt файлом значение этого поля с анимацией?

с 7-ой строки и до конца 20221212_120732_yamusic2.log

Это lottie анимация. Не будем ничего добавлять в ЯМ либу для работы с этим т.к. существуют отдельные либы для лотти. Просто нужно правильно задокументировать данное поле.

CustomWave.download_animation оставлю, чтобы json возвращать или тоже не стоит?

В целом все методы удаляем. Вижу там есть гифки некие, но там работа только с ассетами и без учета фреймов идет склейка в гифку. Что не совсем правильная и точная реализация лотти. Просто делаем документацию: JSON анимация Lottie

@MarshalX
Copy link
Owner

@glebliutsko глянь плиз если есть время

@MarshalX MarshalX merged commit a152e4d into MarshalX:dev Mar 27, 2023
@MarshalX
Copy link
Owner

Спасибо!

@andrew097 andrew097 deleted the issues/#509_NewFields branch April 17, 2023 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants