Skip to content

Add I0 normalization for plots and fix live field selection#290

Merged
rodolakis merged 3 commits into
mainfrom
289-add-normalization-by-i0
Mar 26, 2026
Merged

Add I0 normalization for plots and fix live field selection#290
rodolakis merged 3 commits into
mainfrom
289-add-normalization-by-i0

Conversation

@rodolakis

Copy link
Copy Markdown
Collaborator
  • close Add normalization by I0 #289

    • Add optional I0 column in stream field picker; divide Y by I0 in to_datasets with shape checks; reflect I0 in labels and plot_options
    • Pass I0 through live plot field mapping; apply normalization on live refresh
    • Keep checkbox state across table rebuilds: sync preferred_fields from current model before setStream, restore I0 after refresh, update on Replace emit

- Add optional I0 column in stream field picker; divide Y by I0 in to_datasets
  with shape checks; reflect I0 in labels and plot_options
- Pass I0 through live plot field mapping; apply normalization on live refresh
- Keep checkbox state across table rebuilds: sync preferred_fields from current
  model before setStream, restore I0 after refresh, update on Replace emit
@rodolakis rodolakis requested a review from prjemian March 25, 2026 17:55
@rodolakis rodolakis self-assigned this Mar 25, 2026
@rodolakis rodolakis added the enhancement New feature or request label Mar 25, 2026
@rodolakis rodolakis linked an issue Mar 25, 2026 that may be closed by this pull request
SciPy imports NumPy's version string; mocked NumPy breaks that during
autodoc. Mock scipy alongside numpy. Alias importlib.metadata.version
as distribution_version before assigning Sphinx's short version string.
When PyQt5 is mocked during autodoc, allKeys() returns a non-iterable
mock object, causing to_dict() to raise TypeError. Sphinx's
object_description catches this and re-raises as ValueError, failing
the docs build. Wrap __repr__ in try/except to fall back gracefully.

@prjemian prjemian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rodolakis rodolakis merged commit 454665a into main Mar 26, 2026
5 checks passed
@rodolakis rodolakis deleted the 289-add-normalization-by-i0 branch March 26, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add normalization by I0

2 participants