Skip to content

Added showcase notebook for huber loss#30

Open
msaca-okse wants to merge 2 commits into
TomographicImaging:mainfrom
msaca-okse:main
Open

Added showcase notebook for huber loss#30
msaca-okse wants to merge 2 commits into
TomographicImaging:mainfrom
msaca-okse:main

Conversation

@msaca-okse

Copy link
Copy Markdown

Describe your contribution

I have made a small user showcase notebook, that shows the effect of using the Huber loss function over a least squares data fidelity.

Checklist when you are ready to request a review

  • I have performed a self-review of my code
  • I have created a new folder, containing my contributions which are in the form of jupyter notebooks, with any necessary supporting python files, and a LICENSE file.
  • I have added a description of my contribution(s) to the top of my file(s)
  • If publicly available, I have added a link to the dataset used near the top of my file(s)
  • I have added the CIL version I ran with near the top of my file(s)
  • The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in CIL-User-Showcase.
  • I confirm that the contribution does not violate any intellectual property rights of third parties.
  • I confirm that I have added license headers to all of the files I am contributing (with a license of my choice)
  • Change pull request label to 'Waiting for review'

Note: for an example of a contribution, where a license header, description, data link and CIL version has been added, please
see: example_contribution

@github-project-automation github-project-automation Bot moved this to Todo in UM 2026 Jan 30, 2026
@casperdcl casperdcl moved this from Todo to In Progress in UM 2026 Jan 30, 2026
@paskino

paskino commented Feb 2, 2026

Copy link
Copy Markdown

The original data of the steel wire is https://github.com/DiamondLightSource/Savu/raw/refs/heads/master/test_data/data/24737_fd.nxs

This is a NeXuS file from Diamond Light Source and it's not normalised and it has one outlier in the top detector row which we removed in our test data (and transformed to absorption).

@lauramurgatroyd

Copy link
Copy Markdown
Member

Hi @msaca-okse thank you for your contribution!
The notebook looks great. I noticed a couple of very minor typos - it won't let me suggest these changes so I will try to edit the file myself and push the change.
I will also be looking at your Huber loss PR to CIL. We will aim to get the Huber Loss function into CIL, before merging this showcase.

Signed-off-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants