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

Prevent the project builder from making a recursive decision tree if they add in a new question before the previous one entered into the workflow #2512

Open
mschwamb opened this issue Apr 30, 2016 · 11 comments · Fixed by #5925 · May be fixed by #7097

Comments

@mschwamb
Copy link
Contributor

I've now done this for all 3 workflows I've developed with the project builder.I write the first question and then we end up adding in one before. In the workflow builder, the drop down menu lets you select the current task to be the next task . I seem to always select it accidentally the first go around. it seems pretty straigh forward to not include dropdown menu for the next task.

@mschwamb
Copy link
Contributor Author

mschwamb commented Nov 6, 2016

On the 'Are there polygonal ridges in this image question? and I can set the responses to bring me to the same question I'm on make it recursive.

screen shot 2016-11-06 at 12 34 53 pm

@jelliotartz
Copy link
Contributor

Closing as this has not received comments or action for over a year.

@mschwamb mschwamb reopened this Nov 8, 2017
@mschwamb
Copy link
Contributor Author

mschwamb commented Nov 8, 2017

I still think this is an issue. I hit it everytime I start a new build. So rather than reposting. I'm reopening. Feel free to close if you're not going to fix it.

@stale
Copy link

stale bot commented Mar 8, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Mar 8, 2018
@eatyourgreens
Copy link
Contributor

I think Scribes of the Cairo Genizah are using a recursive workflow where a set of tasks repeat until done, but I don’t think anyone needs to link a task back to itself either.

@stale stale bot removed the stale label Mar 8, 2018
@mrniaboc
Copy link
Contributor

This seems like pretty low priority to me, but not sure if it's a wontfix?

@eatyourgreens
Copy link
Contributor

There are some recursive workflows live on zooniverse.org, but I can't think of any cases where you'd want to link a task back to itself. This should be an easy fix when someone has an hour or so free to look at it.

@mschwamb
Copy link
Contributor Author

mschwamb commented May 10, 2018

Up to y'all to close it or not, but to me it looked like something not polished and looked like a bug with an easy fix. Allowing a project builder to shift around the order of drawing tools would be a higher priority fix than this #4472 in my opinion

@eatyourgreens
Copy link
Contributor

Penguin Watch have just run into this bug, on their Wild Egg Hunt workflow (reported via email by @yshish.)

@eatyourgreens
Copy link
Contributor

Re-opening this because the fix in #5925 isn't working any more, and the project builder now lets you link a task back to itself (under the next task dropdown.)

@eatyourgreens eatyourgreens reopened this May 9, 2024
@eatyourgreens eatyourgreens linked a pull request May 9, 2024 that will close this issue
18 tasks
@eatyourgreens
Copy link
Contributor

#7097 should finally fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants