Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Legacy workspaces raise an unguided error and the converters are never called #14

Copy link
Copy link

Description

@PierreRaybaut
Issue body actions

Summary

convert_legacy_peak_fit_params() and convert_legacy_peak_creation_params() are available from Sigima but referenced nowhere in DataLab-Web. Opening an HDF5 workspace saved before the height-based migration (metadata keyed by amp) makes Evaluate fit raise a bare LegacyPeakParameterizationError, with no explanation and no way forward.

Refusing to silently reinterpret an area as a height is scientifically correct. Surfacing a raw exception is not.

Proposed resolution

Catch the error at the runtime boundary and either offer an explicit "convert legacy parameters" action or display a message stating what changed and which release introduced it. Add a CHANGELOG entry: nothing currently warns users that pre-0.6.4 workspaces need attention.

Not addressed on the fix/fitting-audit branch.

Reactions are currently unavailable

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    Morty Proxy This is a proxified and sanitized view of the page, visit original site.