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

Errors with optional Augur_PHB inputs #211

Closed
kevinlibuit opened this issue Oct 10, 2023 · 2 comments · Fixed by #212
Closed

Errors with optional Augur_PHB inputs #211

kevinlibuit opened this issue Oct 10, 2023 · 2 comments · Fixed by #212
Labels
done This issue has been addressed

Comments

@kevinlibuit
Copy link
Contributor

🐛

📝 Describe the Issue

Users have reported issues with with optional Augur_PHB inputs, specifically colors_tsv.

🔁 How to Reproduce

  • Was this run on the Terra platform?
    • Terra on GCP

🎣 Expected Behavior

Error out of Augur_PHB:

  • Error_1:"Failed to evaluate 'colors_tsv' (reason 1 of 1): Evaluating select_first([colors, mpxv_defaults.colors]) failed: select_first was called with 2 empty values. We needed at least one to be filled."
  • Error_2: "Failed to evaluate 'genes' (reason 1 of 1): Evaluating select_first([genes, mpxv_defaults.genes]) failed: select_first was called with 2 empty values. We needed at least one to be filled."

💾 Version Information

v1.2.0

ℹ️ Additional Information

Observed by multiple users

@kapsakcj
Copy link
Contributor

Looks like this is related to @jrotieno 's recent PR: #160

One of the users mentioned that the workflow ran successfully with v1.1.0 but failed when they upgraded to v1.2.0

@jrotieno
Copy link
Contributor

Looks like this is related to @jrotieno 's recent PR: #160

One of the users mentioned that the workflow ran successfully with v1.1.0 but failed when they upgraded to v1.2.0

Separate and should be fixed by #212

@sage-wright sage-wright added the done This issue has been addressed label Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done This issue has been addressed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants