Closed riversn closed 2 years ago
Ok - I think this is how it works now (First time using this cool framework called mermaid
to make diagrams)
flowchart TD;
nextImage-.->goodPatternFollow;
nextImage-.->nullPatternFollow;
goodPatternFollow-->updateOffset;
nullPatternFollow-->storedOffset;
updateOffset-->storedOffset;
storedOffset-->nextImage;
storedOffset-->composer;
nullOffset--Initial Value-->storedOffset;
storedOffset:::someclass;
classDef someclass fill:#f96;
nullOffset:::nullclass;
classDef nullclass fill:#f69;
And you would like it to be like this?
flowchart TD;
nextImage-.->goodPatternFollow;
nextImage-.->nullPatternFollow;
goodPatternFollow-->updateOffset;
nullPatternFollow-->storedOffset;
updateOffset-->storedOffset;
storedOffset-->nextImage;
storedOffset-.-toggle;
nullOffset--Initial Value-->storedOffset;
storedOffset:::someclass;
classDef someclass fill:#f96;
nullOffset:::nullclass;
classDef nullclass fill:#f69;
nullOffset-.-toggle;
toggle-->composer;
@riversn does this look correct?
Yeah
I think I fixed this? I noticed that the Feature Layout
in the composer always uses the follower pattern offset, but the Feature Grid
never does. So, now the Feature Grid
always uses the follower pattern offset. I think this removes the confusion as the composer will now always reflect the viewer and any grid location changes will be in relation to what you are looking at, not what you are looking at +/- the possible follower pattern. That being said - I might have missed the nail - we'll see.
one annoying thing I noticed is teaching a new pattern after successful pattern rec - best to do it on the very first image of the population and then you don't run into that issue
I want the composer to never shift the grid position, force pattern in the viewer, or running should be the only time the pattern rec gets applied to a grid.
I think you've made it the opposite now?
Maybe it's okay if there's a visual indicator in the composer saying that the grid was shifted?
Yeah I did the opposite - I can flip it just as easily.
But right now, If PR happens and then you stop the run and open composer, you can edit the grid and feature and those changes will be consistent with null & future PR offsets.
If I do the opposite, you would basically have to go to back to the image you set up the PR on for the grid to line up again.
In the version you have now - it is half and half - the grid never gets the PR and the feature always gets the PR. Whichever you decide (always use or never use) they will both at least be the same.
maybe always on is fine if there's a manual on and off pattern rec button.
okay - moved to #119
When a pattern follower is set, sometimes composer will open with the pattern follower engaged and it's annoying. A quick button to 'turn off and back on' once a pattern has been made would be nice.
The issue is that if you reopen the composer and it does a pattern follow, if you recenter the grid it'll be off when you do a run.