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

Move methods database from httomo to httomo-backends #6

Merged
merged 53 commits into from
Jan 9, 2025

Conversation

namannimmo10
Copy link
Member

No description provided.

@namannimmo10
Copy link
Member Author

Tests currently failing locally. WIP.

@yousefmoazzam
Copy link
Contributor

Tests currently failing locally. WIP.

No worries, there's a helpful "draft state" for a PR if it's not ready for reviewing (and is a decent way to prevent merging the PR before it's ready), you can change the state of the PR to a draft in the "Reviewers" section in the top right by clicking "Convert to draft":
convert-pr-to-draft

@namannimmo10 namannimmo10 marked this pull request as draft October 31, 2024 15:59
@namannimmo10 namannimmo10 marked this pull request as ready for review November 11, 2024 13:51
@namannimmo10 namannimmo10 force-pushed the methodsdtbs branch 5 times, most recently from d7252f7 to 141ceb0 Compare November 11, 2024 14:32
@yousefmoazzam yousefmoazzam marked this pull request as draft December 6, 2024 14:49
Reflects the change made to the list of CuFFT versions in
DiamondLightSource/httomo#531.
CI should test the version of `httomo-backends` in the branch associated
with the PR, not an older version of `httomo-backends` from PyPI.
This won't be necessary when the following issue is resolved in
`httomolibgpu`:
DiamondLightSource/httomolibgpu#170.
@yousefmoazzam yousefmoazzam marked this pull request as ready for review January 9, 2025 15:11
@yousefmoazzam yousefmoazzam merged commit d339fbb into main Jan 9, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants