• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

mantidproject / vesuvio / 14185788940
50%

Build:
DEFAULT BRANCH: main
Ran 01 Apr 2025 02:32AM UTC
Jobs 1
Files 12
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

31 Mar 2025 09:33AM UTC coverage: 42.937% (+1.9%) from 40.994%
14185788940

push

github

web-flow
Refactor System tests (#169)

* Split yspace class into back and front

* Take out part for isolating lightest mass and resolution

Took out function for isolating lightest mass data out of the fitting
algorithm. Also took out resolution function.
This makes more sense as the algorithm should act on the workspace that
is going to fit.

* Changed implementation of isolating the lightest mass

Previous implementation was using Minus(), which I realised changes the
Error values due to error propagation.
So I changed the implementation to only change the dataY values

* Add unit tests for isolating mass and resolution

Since I took the functions for isolating the mass and calculating the
resolution out of the y-space fit, I added these unit tests since I am
not changing their implementation from now onwards.

* Replace system test for reduction

Previous system tests were using a file saved in npz, which is not used by the users and was quite cumbersome to work with.
This change outputs the relevant workspaces to a reduction folder (useful for users + useful for testing).
It also means that the code for testing the workspaces becomes much simpler and testing becomes more comprehensive.

* Set up system test for fitting routine

The results of the benchmark need to be updated because currently the
fit in y space is not dealing well with zeros in TOF

* Correct function for isolating mass to account for zeros

In my previous implementation of the function to isolate the mass I forgot to account for masked values with zeros.

* Remove last column of fit results table

The comparison between fit results is failing on github runners, and I have a suspicion that the number of iterations might differ.
This column is not really relevant anyway, so I removed it to see if it passes the tests on github.

54 of 111 new or added lines in 4 files covered. (48.65%)

10 existing lines in 4 files now uncovered.

1149 of 2676 relevant lines covered (42.94%)

0.43 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
5
37.17
-7.32% src/mvesuvio/main/run_routine.py
10
64.07
3.54% src/mvesuvio/util/analysis_helpers.py
18
44.39
3.71% src/mvesuvio/analysis_reduction.py
24
9.92
0.42% src/mvesuvio/analysis_fitting.py

Uncovered Existing Lines

Lines Coverage ∆ File
2
44.39
3.71% src/mvesuvio/analysis_reduction.py
2
37.17
-7.32% src/mvesuvio/main/run_routine.py
3
9.92
0.42% src/mvesuvio/analysis_fitting.py
3
64.07
3.54% src/mvesuvio/util/analysis_helpers.py
Jobs
ID Job ID Ran Files Coverage
1 14185788940.1 01 Apr 2025 02:32AM UTC 12
42.94
GitHub Action Run
Source Files on build 14185788940
  • Tree
  • List 12
  • Changed 4
  • Source Changed 0
  • Coverage Changed 4
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • c4b42ad4 on github
  • Prev Build on main (#13937516528)
  • Next Build on main (#14210174206)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc