bec/csaxs_bec
0
0
Fork 2
Code Issues 17 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

Update user instruction data analysis #243

Closed
zhang_y7 wants to merge 2 commits from zhang_y7/csaxs_bec:update_user_instruction_data_analysis into main
pull from: zhang_y7/csaxs_bec:update_user_instruction_data_analysis
merge into: :main
Conversation 3 Commits 2 Files Changed 6
+118 -48
zhang_y7 commented 2026-07-03 10:15:40 +02:00
Contributor
Copy Link
Copy Source

We moved the data analysis part out of the SAXS section and created a dedicated section for that.

We moved the data analysis part out of the SAXS section and created a dedicated section for that.
👍 1
zhang_y7 added 2 commits 2026-07-03 10:15:40 +02:00
create a seperate section for data analysis eecc6cf37b
Update data analysis section
CI for csaxs_bec / test (pull_request) Failing after 32s
Details
8cfc82dfe6
zhang_y7 requested review from diaz 2026-07-03 10:15:41 +02:00
zhang_y7 requested review from menzel 2026-07-03 10:15:41 +02:00
zhang_y7 requested review from wakonig_k 2026-07-03 10:15:41 +02:00
menzel commented 2026-07-03 14:46:21 +02:00
Member
Copy Link
Copy Source

Closing/re-opening to re-trigger CI — no content changes. The earlier failure was the fork-PR checkout bug fixed in #245 (now on main); re-opening reruns against the fixed workflow.

Closing/re-opening to re-trigger CI — no content changes. The earlier failure was the fork-PR checkout bug fixed in #245 (now on main); re-opening reruns against the fixed workflow.
menzel closed this pull request 2026-07-03 14:46:21 +02:00
menzel reopened this pull request 2026-07-03 14:47:53 +02:00
menzel scheduled this pull request to auto merge when all checks succeed 2026-07-03 14:50:57 +02:00
menzel commented 2026-07-03 15:49:47 +02:00
Member
Copy Link
Copy Source

The failing check was a CI bug that broke the checkout step for any PR opened from a fork. I fixed it in #245, which is now merged to main. However, CI runs the workflow from the branch, so #243 needs to actually contain the fix. Easiest would be for zhang_y7 to update the branch onto latest main.
That'll re-trigger CI with the fixed workflow and the checkout should pass.

The failing check was a CI bug that broke the checkout step for any PR opened from a fork. I fixed it in #245, which is now merged to main. However, CI runs the workflow from the branch, so #243 needs to actually contain the fix. Easiest would be for zhang_y7 to update the branch onto latest main. That'll re-trigger CI with the fixed workflow and the checkout should pass.
menzel canceled auto merging this pull request when all checks succeed 2026-07-03 15:49:55 +02:00
zhang_y7 commented 2026-07-03 15:58:24 +02:00
Author
Contributor
Copy Link
Copy Source

Thanks for the instruction. I will close this pull request, update my branch and created pull request again. Thanks all for looking into this issue.

Thanks for the instruction. I will close this pull request, update my branch and created pull request again. Thanks all for looking into this issue.
zhang_y7 closed this pull request 2026-07-03 15:58:30 +02:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
diaz
wakonig_k
menzel
Labels
Clear labels
issue::not confirmed
Used if an issue has not yet been confirmed by a developer
 issue::not reproducible
Used if an issue is not reproducible
 issue::reproducible
Used if a developer was able to reproduce the issue
 priority::critical
Should be done as soon as possible
 priority::needed
An important update to the code base
 priority::nice to have
Not really needed but a good improvement
 scope::backend
An issue regarding the backend services
 scope::frontend
An issue regarding the user-facing services
 stage::in development
This issue is currently being worked on
 stage::on hold
Something is blocking the development
 stage::pre_release
For issues that have been merged into a pre-release branch and are waiting for their release.
stage::ready for review
For issues that have been resolved and are ready for review
 stage::selected for development
This issue has been selected for development but has not been scheduled yet
 stage::weekly backlog
This issue has been selected for development for the next development cycle
 type::bug
It's a bug!
 type::discussion
A discussion is wanted/needed
 type::doc
Improvements to the documentation
 type::feature
A new feature
 type::improvement
Neither a feature nor a bug but an improvement to the code base
 type::test
Missing tests or improvements to the test suite
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
acerbo_a (acerbo_a) appel_c (appel_c) bec-gitea celcer_t (celcer_t) debenjak_l (debenjak_l) diaz (diaz) divall_e (divall_e) guizar_m (guizar_m) haghta_s (haghta_s) holler (holler) lombosi_c (lombosi_c) menzel (menzel) perl_d (perl_d) usov_i (usov_i) wakonig_k (wakonig_k) wang_x1 (wang_x1) wyzula_j (wyzula_j) zamofing_t (zamofing_t)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bec/csaxs_bec#243