Skip to content

New testing structure with auto-pipelines generation #542

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
Merged
merged 63 commits into from
Feb 20, 2025
Merged

New testing structure with auto-pipelines generation #542

merged 63 commits into from
Feb 20, 2025

Conversation

dkazanc
Copy link
Collaborator

@dkazanc dkazanc commented Jan 29, 2025 •
edited
Loading

Fixes #ISSUE

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have made corresponding changes to the documentation

dkazanc added 30 commits January 29, 2025 11:40
@dkazanc
another cpu generated pipeline added, tests reorganisation, a new CI …
d727a3b 
…for pipelines
@dkazanc
fixing tests1
e2075ce
@dkazanc
Merge branch 'main' into pipelinebuilder
18f8b83
@dkazanc
offloading yaml pipeline generator scripts to httomo-backends
2ad2604
@dkazanc
fixes
fac0dad
@dkazanc
fixes2
f349547
@dkazanc
changes in the building script
d67b90d
@dkazanc
adding pipelines_full folder
cf0137b
@dkazanc
docs update, tests modification and helping scripts
79ee9bc
@dkazanc
fixing ui tests
68986f9
@dkazanc
fixing pipelines ci
c410949
@dkazanc
first test for big data
bc1189d
@dkazanc
removing raw data folder
75c70bd
@dkazanc
tests/scripts/
7df98b4
@dkazanc
scripts to generate ground truth extract for comparison in the test
dc784d0
@dkazanc
modifying test to exclude image saving
80c713e
@dkazanc
modifying flag to output folder
692b1e4
@dkazanc
fixing path to npz file
332bdc1
@dkazanc
fixing the bigdata test
ac9b66a
@dkazanc
further bigtest corrections
6e2d3c5
@dkazanc
corrections to script and test
5f3945b
@dkazanc
further tests fixes
c9ad334
@dkazanc
temp test debugging
8b38153
@dkazanc
temp test debugging2
4aa44ab
@dkazanc
the fix in the name of the file to be opened
cbbca77
@dkazanc
further test corrections
478354f
@dkazanc
working Jenkins test
13872be
@dkazanc
renaming flags, adding more tests
d5c4d20
@dkazanc
Merge branch 'main' into pipelinebuilder
33f1ae3
@dkazanc
updates to tests and docs
845a106
@dkazanc dkazanc marked this pull request as ready for review February 20, 2025 12:16
@dkazanc
Copy link
Collaborator Author

dkazanc commented Feb 20, 2025

The tests on Jenkins now in green and the framework tests pass for me locally.

image

yousefmoazzam
yousefmoazzam reviewed Feb 20, 2025
View reviewed changes
Copy link
Collaborator

@yousefmoazzam yousefmoazzam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, some comments here and there, mainly about code comments and commented-out code.

@dkazanc dkazanc merged commit ee9916f into main Feb 20, 2025
3 of 4 checks passed
@dkazanc dkazanc deleted the pipelinebuilder branch February 20, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yousefmoazzam yousefmoazzam yousefmoazzam left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dkazanc @yousefmoazzam