um/web
um/web
98
194
Fork 2
Code Issues 9 Pull Requests Packages Releases 5 Wiki Activity

改进/重构计划 #9

New Issue
Open
opened 2022-11-24 07:24:35 +00:00 by um-dev · 9 comments
um-dev commented 2022-11-24 07:24:35 +00:00
Owner
Copy Link
  • 将 加密算法、Metadata、Album Cover 分离
  • 更新各个加密算法实现到最新版本
  • 更新UI框架
- [ ] 将 加密算法、Metadata、Album Cover 分离 - [ ] 更新各个加密算法实现到最新版本 - [ ] 更新UI框架
um-dev commented 2022-11-24 07:32:52 +00:00
Author
Owner
Copy Link

代码改进

  • Investigating 删除纯JS实现 #5 (comment)
    • 性能差距
    • 兼容性
    • 二进制体积
  • 写入 Metadata 的 Add / Replace #5 (comment)
  • 坚持使用数组存放歌手信息 #5 (comment)
### 代码改进 - [ ] **Investigating** 删除纯JS实现 https://git.unlock-music.dev/um/web/pulls/5#issuecomment-905 - 性能差距 - 兼容性 - 二进制体积 - [ ] 写入 Metadata 的 Add / Replace https://git.unlock-music.dev/um/web/pulls/5#issuecomment-865 - [ ] 坚持使用数组存放歌手信息 https://git.unlock-music.dev/um/web/pulls/5#issuecomment-894
nullptr-0 commented 2022-11-24 07:54:51 +00:00
Contributor
Copy Link

音乐标签编辑功能改进

  • 增加支持编辑的音乐标签种类

  • 增加对mp3、flac以外格式的音乐标签写入支持

  • 增加歌词匹配和编辑

## 音乐标签编辑功能改进 - [ ] 增加支持编辑的音乐标签种类 - [ ] 增加对mp3、flac以外格式的音乐标签写入支持 - [ ] 增加歌词匹配和编辑
lsr commented 2022-11-24 18:33:58 +00:00
Owner
Copy Link

关于 WASM，感觉可以多开几个仓库放 C++ 代码，然后 CI 自动构建发布（自建或 npm 命名空间）后导入使用。

这样就不需要只做前端的贡献者安装完整的 emscripten / cmake 工具链。

关于 WASM，感觉可以多开几个仓库放 C++ 代码，然后 CI 自动构建发布（自建或 npm 命名空间）后导入使用。 这样就不需要只做前端的贡献者安装完整的 emscripten / cmake 工具链。
um-dev commented 2022-11-24 18:36:39 +00:00
Author
Owner
Copy Link

关于 WASM，感觉可以多开几个仓库放 C++ 代码，然后 CI 自动构建发布（自建或 npm 命名空间）后导入使用。

这样就不需要只做前端的贡献者安装完整的 emscripten / cmake 工具链。

我也倾向于这样处理

> 关于 WASM，感觉可以多开几个仓库放 C++ 代码，然后 CI 自动构建发布（自建或 npm 命名空间）后导入使用。 > > 这样就不需要只做前端的贡献者安装完整的 emscripten / cmake 工具链。 > > 我也倾向于这样处理
Yuki1536 commented 2022-11-28 08:31:15 +00:00
Contributor
Copy Link

代码改进

  • Investigating 删除纯JS实现 #5 (comment)
    • 性能差距
      • 兼容性
      • 二进制体积

同意@xhacker-zzz，js decoder作为wasm失败或不支持的fallback。#5

  • 增加歌词匹配和编辑

问个小问题。输入的文件只有音乐文件，通过元数据匹配歌词后，是页面上来个歌词弹框预览+编辑么
批量解锁的情况下，怎么处理比较好

> ### 代码改进 > - [ ] **Investigating** 删除纯JS实现 https://git.unlock-music.dev/um/web/pulls/5#issuecomment-905 > - 性能差距 > - 兼容性 > - 二进制体积 同意@xhacker-zzz，`js decoder`作为`wasm`失败或不支持的fallback。[#5](https://git.unlock-music.dev/um/web/pulls/5#issuecomment-886) > - [ ] 增加歌词匹配和编辑 问个小问题。输入的文件只有音乐文件，通过元数据匹配歌词后，是页面上来个歌词弹框预览+编辑么 批量解锁的情况下，怎么处理比较好
nullptr-0 commented 2022-11-28 09:12:27 +00:00
Contributor
Copy Link

代码改进

  • Investigating 删除纯JS实现 #5 (comment)
    • 性能差距
    • 兼容性
    • 二进制体积

同意@xhacker-zzz，js decoder作为wasm失败或不支持的fallback。#5

  • 增加歌词匹配和编辑

问个小问题。输入的文件只有音乐文件，通过元数据匹配歌词后，是页面上来个歌词弹框预览+编辑么
批量解锁的情况下，怎么处理比较好
目前计划是加歌词窗口

> > ### 代码改进 > > - [ ] **Investigating** 删除纯JS实现 https://git.unlock-music.dev/um/web/pulls/5#issuecomment-905 > > - 性能差距 > > - 兼容性 > > - 二进制体积 > > 同意@xhacker-zzz，`js decoder`作为`wasm`失败或不支持的fallback。[#5](https://git.unlock-music.dev/um/web/pulls/5#issuecomment-886) > > > > - [ ] 增加歌词匹配和编辑 > > > 问个小问题。输入的文件只有音乐文件，通过元数据匹配歌词后，是页面上来个歌词弹框预览+编辑么 > 批量解锁的情况下，怎么处理比较好 目前计划是加歌词窗口
lsr commented 2022-11-29 17:24:35 +00:00
Owner
Copy Link

js decoder作为wasm失败或不支持的fallback。#5

我的看法是没有必要。这个项目本来就不支持过于古老的浏览器… 人手也不足，还要维护两份不同语言的相同实现。

而算法实现基于同一份代码衍生的。

> `js decoder`作为`wasm`失败或不支持的fallback。[#5](https://git.unlock-music.dev/um/web/pulls/5#issuecomment-886) 我的看法是没有必要。这个项目本来就不支持过于古老的浏览器… 人手也不足，还要维护两份不同语言的相同实现。 而算法实现基于同一份代码衍生的。
nullptr-0 commented 2023-01-19 04:44:25 +00:00
Contributor
Copy Link

代码改进

  • Investigating 删除纯JS实现 #5 (comment)
    • 性能差距
      • 兼容性
      • 二进制体积
  • 写入 Metadata 的 Add / Replace #5 (comment)
  • 坚持使用数组存放歌手信息 #5 (comment)

已在#29中删除重复的JS实现

> ### 代码改进 > - [ ] **Investigating** 删除纯JS实现 https://git.unlock-music.dev/um/web/pulls/5#issuecomment-905 > - 性能差距 > - 兼容性 > - 二进制体积 > - [ ] 写入 Metadata 的 Add / Replace https://git.unlock-music.dev/um/web/pulls/5#issuecomment-865 > - [ ] 坚持使用数组存放歌手信息 https://git.unlock-music.dev/um/web/pulls/5#issuecomment-894 > 已在[#29]中删除重复的JS实现 [#29]: https://git.unlock-music.dev/um/web/pulls/29
nullptr-0 commented 2023-01-19 04:49:10 +00:00
Contributor
Copy Link

关于 WASM，感觉可以多开几个仓库放 C++ 代码，然后 CI 自动构建发布（自建或 npm 命名空间）后导入使用。

这样就不需要只做前端的贡献者安装完整的 emscripten / cmake 工具链。

已在#29中分离WASM至QmcWasm和KgmWasm

> 关于 WASM，感觉可以多开几个仓库放 C++ 代码，然后 CI 自动构建发布（自建或 npm 命名空间）后导入使用。 > > 这样就不需要只做前端的贡献者安装完整的 emscripten / cmake 工具链。 > > 已在[#29]中分离WASM至[QmcWasm]和[KgmWasm] [#29]: https://git.unlock-music.dev/um/web/pulls/29 [QmcWasm]: https://github.com/xhacker-zzz/QmcWasm [KgmWasm]: https://github.com/xhacker-zzz/KgmWasm
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
bug

Something is not working

  
crypto

加密算法相关

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
platform-android

安卓客户端下载的文件

  
platform-mac

Mac 客户端下载的文件

  
platform-win

Windows 客户端下载的文件

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
crypto
duplicate
enhancement
help wanted
invalid
platform-android
platform-mac
platform-win
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: um/web#9
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?