Skip to content

feat: apply processings from panel change#4254

Open
tpoisseau wants to merge 11 commits into
mainfrom
824-implement-processings-application-in-ui-on-operation-change
Open

feat: apply processings from panel change#4254
tpoisseau wants to merge 11 commits into
mainfrom
824-implement-processings-application-in-ui-on-operation-change

Conversation

@tpoisseau

Copy link
Copy Markdown
Contributor

fix: spectrum init when it imply processings

  • now it check if originalInfo / originalData exists. Only set them if not exists
  • disable filters apply if processings

tpoisseau added 2 commits July 8, 2026 14:08
fix: spectrum init when it imply `processings`

* now it check if originalInfo / originalData exists. Only set them if not exists
* disable filters apply if `processings`
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 8, 2026

Copy link
Copy Markdown

Deploying nmrium with  Cloudflare Pages  Cloudflare Pages

Latest commit: 50821b7
Status: ✅  Deploy successful!
Preview URL: https://5b60d7fd.nmrium.pages.dev
Branch Preview URL: https://824-implement-processings-ap.nmrium.pages.dev

View logs

@tpoisseau tpoisseau requested a review from targos July 8, 2026 12:40
Comment thread src/component/panels/hooks/use_processings_mutation.ts
@targos

targos commented Jul 8, 2026

Copy link
Copy Markdown
Member

I find a lot of issues by playing with the interface but I'm not sure I should report them here.

tpoisseau added 2 commits July 8, 2026 16:00
fallback to undefined instead null for better compatibility with other methods (like `isSpectrum1D(spectrum: Spectrum | undefined)`).
Use generic to infer from `defaultValue`
@targos

This comment was marked as resolved.

@targos

This comment was marked as resolved.

@targos

This comment was marked as resolved.

@tpoisseau tpoisseau requested a review from targos July 9, 2026 08:53
@tpoisseau

Copy link
Copy Markdown
Contributor Author

@targos

targos commented Jul 9, 2026

Copy link
Copy Markdown
Member

What's the issue?

@tpoisseau

Copy link
Copy Markdown
Contributor Author

Coffee name don't match, CI expect 1, 2, 3, ...

@targos

targos commented Jul 9, 2026

Copy link
Copy Markdown
Member

I don't know, but I guess it's related to the changes in src/data/data1d/Spectrum1D/initiateDatum1D.ts

@tpoisseau

Copy link
Copy Markdown
Contributor Author

git checkout main -- src/data/data1d/Spectrum1D/initiateDatum1D.ts

Solves this issue, but broke processings :/

@tpoisseau

Copy link
Copy Markdown
Contributor Author

coffee file is a very old nmrium file (unversioned), where display holds a name attribute.

changes from version 3 to version 4:
6- move 'name' object from the 'display' object to the'info' object

Due to nmrium component responsible of spectrum normalization and responsible to run filters. it was not an issue until now.
Core already do lot of normalization since nmr-load-save legacy package. And now the core is responsible to running the processing before the nmrium component receive it.

I'l try to understand how core, with read and migrations should be updated to have originalInfo name identical to info name in this context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants