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

Workflow with multiple combo tasks doesn't load #5084

Closed
mcbouslog opened this issue Jul 28, 2023 · 12 comments
Closed

Workflow with multiple combo tasks doesn't load #5084

mcbouslog opened this issue Jul 28, 2023 · 12 comments
Labels
bug Something isn't working

Comments

@mcbouslog
Copy link
Contributor

Package

  • lib-classifier

Describe the bug

A workflow with multiple combo tasks fails to load:

To Reproduce

Steps to reproduce the behavior:

  1. Go to workflow with multiple combo tasks FEM classify page
  2. Note tasks don't load
  3. Go to workflow with multiple combo task PFE classify page
  4. Note tasks load

Expected behavior

Tasks load.

Additional context

With steps/pages editor this won't be an issue, can use steps/pages editor instead.

@mcbouslog mcbouslog added the bug Something isn't working label Jul 28, 2023
@eatyourgreens
Copy link
Contributor

eatyourgreens commented Jul 29, 2023

Workflow 24807 has a steps array, so it will ignore the combo tasks and use the steps that were added by the project builder.

Screenshot of workflow 24807 in the dev console, showing that it has a steps array with 5 steps.

@eatyourgreens
Copy link
Contributor

When this happened to me, it was because projects with transcription enabled automatically add workflow.steps to all workflows.
zooniverse/Panoptes-Front-End#6585

@eatyourgreens
Copy link
Contributor

eatyourgreens commented Jul 29, 2023

I turned off the transcription task for your test project, then built a combo task workflow. This works:

This is a really annoying bug in the project builder.

@eatyourgreens
Copy link
Contributor

More annoying because you can't delete workflow.steps once it's been created, except directly via the Panoptes API.

@mcbouslog
Copy link
Contributor Author

Thanks @eatyourgreens ! Interesting and indeed frustrating.

I think I'm only seeing the first step / 2 combo tasks on https://frontend.preview.zooniverse.org/projects/markbouslog/test-project-mb/classify/workflow/24812 ? I don't see the second step / 3rd combo task and it's Done (instead of Next) on the first step.

Screenshot 2023-07-29 at 7 34 27 AM

@eatyourgreens
Copy link
Contributor

Looks ok to me.

Screenshot of two question tasks with a Next button underneath them.

@eatyourgreens
Copy link
Contributor

Oh wait, that's the wrong page. You might need to dig into the code to figure out how the next step is calculated. The next task should be set on the combo task.

@eatyourgreens
Copy link
Contributor

We used combo tasks to build Engaging Crowds projects so they should just work.

@eatyourgreens
Copy link
Contributor

I've gone through and explicitly set the next task for each answer.

@eatyourgreens
Copy link
Contributor

Seems to be working now. I think the single question task determines the next step when you choose an answer, so that you can build branching workflows.

@mcbouslog
Copy link
Contributor Author

Yes, that appears to have fixed it, thanks for working through this.

The impetus behind this issue is this Notes from Nature Lab workflow - https://www.zooniverse.org/lab/9485/workflows/24808. I've seen the same issue/resolution in that the question task answers in the first combo did not have a next task defined, but once I selected a next task for the question task answers, the steps/combos work as expected.

Yep the single question task answers and combo next task are conflicting somewhere in https://github.com/zooniverse/front-end-monorepo/blob/master/packages/lib-classifier/src/store/helpers/convertWorkflowToUseSteps/convertWorkflowToUseSteps.js, but it's not jumping out to me. I'll try and take a closer look during the week.

@eatyourgreens
Copy link
Contributor

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

No branches or pull requests

2 participants