Saturation indicator #106
Saturation indicator #106
Conversation
|
The implementation here
Is this algorithm sufficient? Is this the interface we need? Is there any test data (beside the example posted in the issue)? |
I think so...?
We should probably have a visualization tool too. (i.e. notebook in code shelf)
Unfortunately not. But it will be needed from the day 1 of hot-comissioning. |
What kind of visualization do you have in mind? |
Yes...! So... maybe the final interface for user(instrument scientist), including visualization can be written later from here...? |
That can be fixed. 👍 I just scheduled a meeting with Soren this afternoon to talk about what interface he needs. |
|
Example signal as function of gain and wavelength (input to function): Saturation kicks in around 
"Saturation indicator", the maximum indicates that saturation has kicked in: 
To avoid saturation, set the gain slightly to the left of the peak. |
|
These figures display the indicator for signals with three different levels of noise. 
This figure shows what the indicator looks like for Sun's example data in the issue description: |
docs/tools/index.md
Outdated
| sharpness-and-focus-point | ||
| maximum_resolution_achievable | ||
| saturation_indicator |
There was a problem hiding this comment.
Is the notebook called saturation_indicator missing...?
And maximum_resolution_achievable is also missing from the menu: https://remote-unzip.deno.dev/scipp/essimaging/artifacts/3961599931/tools/index.html
There was a problem hiding this comment.
Good catch 👍 added those by mistake.
| "laplace_2d", | ||
| "maximum_resolution_achievable", | ||
| "resample", | ||
| "saturation_indicator", |
There was a problem hiding this comment.
The API reference doesn't seem to be rendered properly.
There was a problem hiding this comment.
Strange, might be because of some white space in the docs comment.
I'll try to fix it.
There was a problem hiding this comment.
Okay I'm not sure why the api reference is not rendered properly here. Do you know @YooSunYoung ?
There was a problem hiding this comment.
Ah sorry I missed the comment. It looks like you fixed it though : D ...!
| "laplace_2d", | ||
| "maximum_resolution_achievable", | ||
| "resample", | ||
| "saturation_indicator", |
There was a problem hiding this comment.
Ah sorry I missed the comment. It looks like you fixed it though : D ...!
Fixes #24
Not sure if this is actually the tool that is requested, or if the approach does what we need, but I think it's good to have a starting point for discussion.