Skip to content

estimate_params format #62

Open
Open
@teubert

Description

@teubert

Requirement Text
Allow users to specify input to estimate_params or calc_error as either (times, inputs, ouputs) or (combined)

where combined is a list of simresults, a simresults, a dataframe, or a list of data frames.

Background Information
Right now, we require (times, inputs, outputs), however that requires reformatting the downloaded data or data from simulations. This change would make it easier for users

Suggested Solution
Make default for inputs, outputs = None, check format of times. If simresults, dataframe, or list of either. Reformat input and call recursively.

DoD

  • Implement feature
  • Add to example
  • Add tests
  • update documentation
  • Add to change notes for next release

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions