Skip to content

Commit

Permalink
Merge pull request #216 from pepkit/dev
Browse files Browse the repository at this point in the history
0.12.1 point release
  • Loading branch information
donaldcampbelljr authored Feb 10, 2025
2 parents 9aa242c + f9460d3 commit 6ce7c3e
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
4 changes: 4 additions & 0 deletions docs/changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html) and [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) format.

## [0.12.1] - 2025-02-10
### Fixed
- fix checking for records during summarize failing because aggregate_results file had not yet been resolved [#215](https://github.com/pepkit/pipestat/issues/215)

## [0.12.0] - 2025-01-16
### Fixed
- portable report now has proper file extension in messaging.
Expand Down
2 changes: 1 addition & 1 deletion pipestat/_version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "0.12.0"
__version__ = "0.12.1"
16 changes: 8 additions & 8 deletions pipestat/pipestat.py
Original file line number Diff line number Diff line change
Expand Up @@ -938,14 +938,6 @@ def summarize(
"""

# Before proceeding check if there are any results at the specified backend
try:
current_results = self.select_records()
if len(current_results["records"]) < 1:
raise PipestatSummarizeError(f"No results found at specified backend")
except Exception as e:
raise PipestatSummarizeError(f"PipestatSummarizeError due to exception: {e}")

if output_dir:
self.cfg[OUTPUT_DIR] = output_dir

Expand All @@ -956,6 +948,14 @@ def summarize(

self.check_multi_results()

# Before proceeding check if there are any results at the specified backend
try:
current_results = self.select_records()
if len(current_results["records"]) < 1:
raise PipestatSummarizeError(f"No results found at specified backend")
except Exception as e:
raise PipestatSummarizeError(f"PipestatSummarizeError due to exception: {e}")

html_report_builder = HTMLReportBuilder(prj=self, portable=portable)
report_path = html_report_builder(
pipeline_name=self.cfg[PIPELINE_NAME],
Expand Down

0 comments on commit 6ce7c3e

Please sign in to comment.