search

tjarkvandemerwe / tidyprompt

Create LLM prompting pipelines
https://tjarkvandemerwe.github.io/tidyprompt/
Other
7 stars 1 forks source link

answer_as_integer: add_instruction_to_prompt = FALSE unclear #4

Closed tjarkvandemerwe closed 1 month ago

tjarkvandemerwe commented 1 month ago

In de functie answer_as_integer, waarom is de default value voor de instructie toevoegen aan de prompt FALSE? Wil je juist niet dat de modification function altijd de prompt aanpast? Of gebeurt dat op een andere plek? En is dit wel een setting die we aanpasbaar willen maken?

answer_as_integer <- function(
    prompt_wrap_or_list, min = NULL, max = NULL, add_instruction_to_prompt = FALSE
)
lukakoning commented 1 month ago

'TRUE' is inderdaad logischer; klopt! Verder had ik deze optie vooral even toegevoegd om een fout op validatie te triggeren (om te testen of dat werkte), vandaar. Maar kan worden aangepast naar TRUE dus

tjarkvandemerwe commented 1 month ago

6cab1af5706bd01837e7882fabe4bea7da889a92 fix