d12frosted / d12frosted.io

Personal site
https://d12frosted.io
BSD 3-Clause "New" or "Revised" License
8 stars 3 forks source link

posts/2021-01-24-task-management-with-roam-vol6 #54

Open utterances-bot opened 2 years ago

utterances-bot commented 2 years ago

Task management with org-roam Vol. 6: Select a person and view related tasks

How to select a person and list all related tasks

https://d12frosted.io/posts/2021-01-24-task-management-with-roam-vol6.html

felixbuchholz commented 2 years ago

There’s a small typo in the function definition of vulpea-agenda-person I think:

In the second to last line, should it be let instead of dlet?

Thank you so much for your work on this series and the vulpea library! It’s pretty amazing :)

d12frosted commented 2 years ago

Hey @felixbuchholz . Frankly, I don't remember why exactly I used dynamically scoped let here (e.g. dlet), but indeed it works with lexical scope as well. Will update 💪 Thanks for reporting.

d12frosted commented 2 years ago

🤔 I have this line in the article.

dlet here is used for dynamic binding. If you are not using lexical scope, you can use regular let here.

Really curious why I used dynamic binding 🤔 Will double check tomorrow and if it really doesn't make any sense, will update 😸

d12frosted commented 2 years ago

@felixbuchholz I have updated the example + added usage of vulpea-select-from with vulpea-db-query-by-tags-some. This combination works faster than vulpea-select. Some information can be found in Performance section of Vulpea documentation.

felixbuchholz commented 2 years ago

Thank you for your super fast response! Ah sorry, I must have skipped that line in the article. My bad! It didn’t work for me with dlet, that’s why I used let. I’m also just starting out with emacs and org-roam, so it was just a bit of trial-and-error on my side. :) Amazing, that you even optimized the function :tada: