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

Added cost func to the list of options displayed in log file #1249

Merged
merged 6 commits into from May 7, 2024

Conversation

AndrewLister-STFC
Copy link
Contributor

Description of work

The log file didn't output which cost func was used

Fixes #

Testing Instructions

Checklist

  • The title is of a format appropriate for a line in future release notes.
  • I have added the appropriate tags to the PR.
  • My PR represents one logical piece of work.
  • My PR fully fixes the issue linked. If new issues have been created, what are they?
  • I have added the appropriate tests to cover code that has been added.
  • I have updated the documentation in the relevant places to cover the changes.

@coveralls
Copy link
Collaborator

coveralls commented Feb 16, 2024

Coverage Status

coverage: 90.832%. remained the same
when pulling 03577fc on cost_func_options_write
into 76eb8d7 on master.

Copy link

github-actions bot commented Feb 16, 2024

Unit & System Test Results

  4 files  ±0    4 suites  ±0   6m 42s ⏱️ +4s
502 tests ±0  494 ✅ ±0   8 💤 ±0  0 ❌ ±0 
958 runs  ±0  904 ✅ ±0  54 💤 ±0  0 ❌ ±0 

Results for commit 03577fc. ± Comparison against base commit 76eb8d7.

♻️ This comment has been updated with latest results.

@jess-farmer jess-farmer merged commit 8ebf3ac into master May 7, 2024
10 of 11 checks passed
@jess-farmer jess-farmer deleted the cost_func_options_write branch May 7, 2024 10:49
@AndrewLister-STFC AndrewLister-STFC changed the title Add cost func to options output in log Added cost func to the list of options displayed in log file May 14, 2024
@AndrewLister-STFC AndrewLister-STFC added pr-fix Tag PR as bug fix in notes labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-fix Tag PR as bug fix in notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants