update changelog to prepare for 8.0.0-alpha1#236
Conversation
|
@githubdoe if you have time, kindly approve (or reject) the other opened pull requests. No pressure about the timeline I just want to ensure you see them. |
RevisionHistory.html
Outdated
There was a problem hiding this comment.
What's RMS plot again? Is this where you fixed the spinnable arrow thing that shows different diameters? Maybe you can explain slightly better. Also I don't want to scare anyone into thinking all analysis with 7.4.0 were bad and need to be thrown out. So maybe (if it's true) say "fix minor bug...".
The terminology that I think Dale uses:
3d plot (top left plot)
contour plot (top right plot)
profile plot (one (or sometimes 16) diameter of the wavefront(s))
There was a problem hiding this comment.
Spintable was fixed in 7.4.0 : Fix discrepency in wavefront 2D cut angle versus wavefront profile
RMS plot: as Dale explained. Batch process, you see RMS values and "click to select" was broken
|
George. The RMS plots appear when pressing the filter button and when doing batch process to let you monitor the process by seeing the astig and rms values of each wave front. You can easily then see which ones might be outliers and then decide if they should be used or not. Usually outliers in RMS are caused by low signal to noise ratio causing many unwrap errors. Those plots let one hover over the points on the graph and pop up the wave front associated with them just like I did with the astig polar plots. He broke that hover and/or click on function to begin with and has now corrected it. |
|
Ah! Got it. |
gr5
left a comment
gr5
left a comment
There was a problem hiding this comment.
I don't care if newest is at the top or the bottom. Every time I open that file I am surprised that the newest is at the bottom but I don't think users look at it as they see the same info on the releases page on github.
I will revert. I agree people probably do not look at it but still... |
|
I reversed order of the revisions. You might want to look at last commit also 312675a. I got Copilot help for review. |
I just checked all those. Looks good. It even got an "li" tag. |
|
@gr5 sorry. Additional changes again |
|
@githubdoe Kindly review #238 and #240 when you have time. Than I will release v8.0.0-alpha1 I have nothing planned/masked to come after those. If I find time I might work on #234 but I probably won't have time soon. |
3 participants
close #225
First of all: yes I messed with this file.
I'm sorry but I think it's actually more readable like this. I used an autoformatter and it helped find out many places that had incorrect html tags. html is in fact quite permissive.
So kindly just focus on test of 8.0.0 or look at final result in your browser.
Also I propose that we invert the order of the list. Maybe have 8.0.0 at top of file and v1 at bottom. What do you think ?