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]: 混合复习模式下,今日待复习笔记未到期时,不推送新笔记 #67

Closed
tangwells opened this issue Jul 22, 2024 · 8 comments · Fixed by #69
Closed
Labels
bug Something isn't working

Comments

@tangwells
Copy link

Describe the bug

在混合复习模式下,如果今日待复习笔记基本复习完毕,剩余1~2个今日笔记都是须几分钟后再复习的时候,不会自动推送新笔记进行复习。
image
image

To Reproduce

  1. 新笔记队列里存在笔记
  2. 今日待复习笔记只有几个,且待复习时间都在几分钟后。
  3. 选择打开一篇笔记进行复习功能

Expected behavior

希望能在当前情况下能正常推送新笔记,或者退出混合复习模式?

Screenshots

No response

OS of your device

mac os sonoma 14.5

Obsidian version

v1.6.7

Plugin version

最新版本

Installer version

No response

Additional context

No response

Config file

No response

@tangwells tangwells added the bug Something isn't working label Jul 22, 2024
@Newdea
Copy link
Collaborator

Newdea commented Jul 22, 2024

  1. 如果待复习笔记是当天已经复习过的,必须时间到了才能复习。这个与混合序列无关。
  2. 如果不想用混合序列,可以调节第二个滑条,值等于第一个滑条值为优先复习到期的(即之前版本的),值等于零为优先复习新笔记

希望能在当前情况下能正常推送新笔记

没明白你的意思,新笔记是有推送的。还是说就是指退出混合序列,像之前版本的到期优先?

@Newdea
Copy link
Collaborator

Newdea commented Jul 22, 2024

另外,这个设置后要重启插件或ob才生效,我下版改成立即生效吧。

@tangwells
Copy link
Author

目前我的设置是第一个滑条数值为7、第二个滑条数值为5,且已重启 ob 保证设置生效,同时今日待复习笔记已清空。我所反馈问题的实际情况如下:

IMG_2883.MOV

@tangwells
Copy link
Author

tangwells commented Jul 22, 2024

这里先定义一下:

  • 临期笔记:指重来的旧笔记与正在学习的新笔记,它们的到期时间都在某时刻(当下)的几分钟后。

如视频所见,当今日待复习笔记队列中只存在「临期笔记」或者没有笔记时,插件并不会推送新笔记而是提示“复习完毕,明日再来“(注意,如视频所示,此时实际上新笔记队列中是有笔记的)。个人认为这是由于新版本的 mix queue 功能所导致的bug。
(这个功能实际上个人十分喜欢,大大提高了每日新笔记学习的进度。)

至于前面所提到的:

希望能在当前情况下能正常推送新笔记

实际只是希望当今日待复习笔记队列中只存在「临期笔记」或者没有笔记时,在不需要再对 mix queue 功能进行设置调整的情况下,能正常地推送新笔记。

@tangwells
Copy link
Author

tangwells commented Jul 22, 2024

emmm,大佬 这个问题似乎不会非常稳定的发生,但确实存在。如果出现提示复习完成而没有推送新笔记的情况,只要我多选择几次“打开一篇笔记开始复习”,就又开始正常推送新笔记了。

@Newdea
Copy link
Collaborator

Newdea commented Jul 22, 2024

找到原因了,是到期笔记数还没达到就没有到期笔记时,就会出现这个问题,已经修改了,你试下

main_fixMixQue.zip

@tangwells
Copy link
Author

报告大佬,经过2天对于笔记复习模块的高强度使用,所反馈的问题未再发生。

@Newdea
Copy link
Collaborator

Newdea commented Jul 26, 2024 via email

@Newdea Newdea mentioned this issue Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants