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

【bug】最新版本(截至20241225的主线)代码,存在“结果缩略图和点击后的大图顺序不一致”的问题 #28

Open
keepChatAlive opened this issue Dec 25, 2024 · 19 comments

Comments

@keepChatAlive
Copy link

就是,比如缩略图顺序是1 2 3 4 5
点进去后顺序每次都是完全随机的,比如
这一次可能是5 2 4 3 1
下一次可能是1 3 5 2 4
点击第1张图,大图打开的可能是第5张图或者第3张图;缩略图和大图的对应关系完全没有规律
此问题是由commit=e105ba46c3fef59029ac391bf01b10a37c298231中的
// fix the order of the result list
private fun reorderList(originalList: List, orderList: List): List {
val photoMap = originalList.associateBy { it.id }
return orderList.mapNotNull { id -> photoMap[id] }
}
代码引入的,将上述代码段还原回提交之前,“结果缩略图和点击后的大图顺序不一致,一张图点击后是另一张图”的问题可以解决

@jelychow
Copy link
Collaborator

hello 周末会进行修复

@3410920394
Copy link

大哥现在能不能超过30个识别的上限?

@jelychow
Copy link
Collaborator

jelychow commented Jan 4, 2025

可以在代码里面调整参数

@keepChatAlive
Copy link
Author

可以在代码里面调整参数

我帮您优化了一些问题,包括这个问题,请查看PR

@3410920394
Copy link

可以在代码里面调整参数

大哥有安卓版的吗?

@3410920394
Copy link

纯纯小白看不懂呀。

@jelychow
Copy link
Collaborator

jelychow commented Jan 6, 2025

hello @3410920394 这个 app 就是 Android 版本的呀,如果您是纯小白,建议您这边先学习一点点 Android 知识

@3410920394
Copy link

hello @3410920394 这个 app 就是 Android 版本的呀,如果您是纯小白,建议您这边先学习一点点 Android 知识

我是这个1.3版本的这个。压缩包看不明白

@3410920394
Copy link

这个实图真的是特别有用。效果很好 就是不知道怎么修改它一次性识别的上限

@3410920394
Copy link

hello @3410920394 这个 app 就是 Android 版本的呀,如果您是纯小白,建议您这边先学习一点点 Android 知识

您发出来的版本我不知道如何修改

@jelychow
Copy link
Collaborator

jelychow commented Jan 6, 2025

@3410920394 建议配合 gpt 辅助学习一下即可

@3410920394
Copy link

@3410920394 建议配合 gpt 辅助学习一下即可

大哥加个q

@jelychow
Copy link
Collaborator

jelychow commented Jan 6, 2025

没事 你在这个 thread 留 comment 就好了,会给你解决的

@3410920394
Copy link

没事 你在这个 thread 留 comment 就好了,会给你解决的

我确实是一窍不通。

@keepChatAlive
Copy link
Author

没事 你在这个 thread 留 comment 就好了,会给你解决的

要不看下我PR?我手机里有几万张图,还真挺需要我自己写的那个功能的

@3410920394
Copy link

没事 你在这个 thread 留 comment 就好了,会给你解决的

要不看下我PR?我手机里有几万张图,还真挺需要我自己写的那个功能的

怎么说

@jelychow
Copy link
Collaborator

jelychow commented Jan 6, 2025

没事 你在这个 thread 留 comment 就好了,会给你解决的

要不看下我PR?我手机里有几万张图,还真挺需要我自己写的那个功能的

你好,给你留了几个comment,你看看,明天我测试一下功能,btw 如果你偏重性能的话,优先使用clip 模型,他解码速度比较快

@keepChatAlive
Copy link
Author

没事 你在这个 thread 留 comment 就好了,会给你解决的

要不看下我PR?我手机里有几万张图,还真挺需要我自己写的那个功能的

怎么说

去我拉的分支https://github.com/keepChatAlive/PicQuery/tree/master,直接下载代码,用android studio编译,就能获得已经改好识别800张图,支持“手动更新”索引的版本了

@keepChatAlive
Copy link
Author

没事 你在这个 thread 留 comment 就好了,会给你解决的

要不看下我PR?我手机里有几万张图,还真挺需要我自己写的那个功能的

你好,给你留了几个comment,你看看,明天我测试一下功能,btw 如果你偏重性能的话,优先使用clip 模型,他解码速度比较快

我好像没看到有评论?

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

No branches or pull requests

3 participants