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.

Sign up for GitHub

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

Jump to bottom

Report workflow hangs if there are no observations #1337

Closed
jacobtylerwalls opened this issue Oct 3, 2023 · 1 comment · Fixed by #1355
Closed

Report workflow hangs if there are no observations #1337

jacobtylerwalls opened this issue Oct 3, 2023 · 1 comment · Fixed by #1355
Assignees
@njkim
Labels
bug Something isn't working

Comments

@jacobtylerwalls
Copy link
Member

jacobtylerwalls commented Oct 3, 2023 •
edited
Loading

Tried a couple projects, and generating the reports all failed like this on the last step. Let me know if you see the same thing.

2023-09-21 15:07:13 django.core.exceptions.ValidationError: ['“” is not a valid UUID.']
2023-09-21 15:07:13 [21/Sep/2023 19:07:13] "GET /api/bulk_disambiguated_resource_instance?v=beta&resource_ids= HTTP/1.1" 500 190147

Originally posted by @jacobtylerwalls in #1271 (comment)
@jacobtylerwalls jacobtylerwalls changed the title Report workflow fails if there are no observations Report workflow hangs if there are no observations Oct 3, 2023
@chiatt chiatt moved this to 🔖 Ready in afs-pipeline Oct 11, 2023
@chiatt chiatt added the bug Something isn't working label Oct 11, 2023
@njkim njkim self-assigned this Oct 13, 2023
@njkim njkim moved this from 🔖 Ready to 🏗 In progress in afs-pipeline Oct 13, 2023
njkim added a commit that referenced this issue Oct 13, 2023
@njkim
Check the obervation ids before request, #1337
72fe3a1
@njkim njkim mentioned this issue Oct 13, 2023
Merged
@njkim njkim linked a pull request Oct 13, 2023 that will close this issue
The project report no observation, #1337 #1355
Merged
jacobtylerwalls added a commit that referenced this issue Oct 13, 2023
@jacobtylerwalls
Merge pull request #1355 from archesproject/1337_project_report_no_ob…
d23f7b6 
…servation

The project report no observation, #1337
@jacobtylerwalls
Copy link
Member Author

Fixed in #1355

@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in afs-pipeline Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@njkim njkim

Labels
bug Something isn't working
Projects
afs-pipeline
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

The project report no observation, #1337 archesproject/arches-for-science
3 participants
@chiatt @njkim @jacobtylerwalls