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

Question about OST_Node when its parents is SUBQUESTION_Node #14

Open
ybz79 opened this issue Nov 5, 2024 · 1 comment
Open

Question about OST_Node when its parents is SUBQUESTION_Node #14

ybz79 opened this issue Nov 5, 2024 · 1 comment

Comments

@ybz79
Copy link

ybz79 commented Nov 5, 2024

Hi,
First thanks for your works.

I find that when generating OST_step after SUBQUESTION_Node,the parameter parent_is_subquestion of do_action_generate_ost_step() is also set to False.

elif self.node_type is Node_Type.SUBQUESTION:

    if not self.disable_a1:
        do_action_generate_ost_step()


    do_action_generate_direct_answers()


    do_action_generate_subquestions()

https://github.com/zhentingqi/rStar/blob/main/run_src/MCTS_for_reasoning.py#L920-L923

I think under this setting, the solution trace is incomplete. If I have missed anything, please let me know. I would greatly appreciate it.

@JiahangXu
Copy link
Collaborator

Thank you for your question! This was actually a mistake in our code cleanup. I’ve checked our dev version, and it does have parent_is_subquestion=true here. I’ve now fixed the issue, and you can pull the latest code to see my updates.

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

2 participants