Skip to content

Commit

Permalink
check if samples is None before taking subsamples - do we need to con…
Browse files Browse the repository at this point in the history
…sider subsamples wrt samples summary?
  • Loading branch information
rhayes777 committed Apr 22, 2024
1 parent c935644 commit c9dcac9
Showing 1 changed file with 15 additions and 19 deletions.
34 changes: 15 additions & 19 deletions autofit/non_linear/analysis/indexed.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,19 +58,15 @@ def __getattr__(self, item):
return getattr(self.analysis, item)

def make_result(
self,
samples_summary: SamplesSummary,
paths: AbstractPaths,
samples: Optional[SamplesPDF] = None,
search_internal: Optional[object] = None,
analysis: Optional[object] = None,
self,
samples_summary: SamplesSummary,
paths: AbstractPaths,
samples: Optional[SamplesPDF] = None,
search_internal: Optional[object] = None,
analysis: Optional[object] = None,
):
return self.analysis.make_result(
samples_summary,
paths,
samples,
search_internal,
analysis
samples_summary, paths, samples, search_internal, analysis
)


Expand All @@ -96,12 +92,12 @@ def __init__(self, *analyses):
)

def make_result(
self,
samples_summary: SamplesSummary,
paths: AbstractPaths,
samples: Optional[SamplesPDF] = None,
search_internal: Optional[object] = None,
analysis: Optional[object] = None,
self,
samples_summary: SamplesSummary,
paths: AbstractPaths,
samples: Optional[SamplesPDF] = None,
search_internal: Optional[object] = None,
analysis: Optional[object] = None,
):
"""
Associate each model with an analysis when creating the result.
Expand All @@ -110,9 +106,9 @@ def make_result(
analysis.make_result(
samples_summary,
paths,
samples.subsamples(model),
samples.subsamples(model) if samples is not None else None,
search_internal,
analysis
analysis,
)
for model, analysis in zip(samples_summary.model, self.analyses)
]
Expand Down

0 comments on commit c9dcac9

Please sign in to comment.