I tested on the 54x100 that it was created for.
Are you implying that it should only work on a 54x100 tree??? That's bunk as it would make purchasing this sequence pretty much worthless to most people. And if that's true then I want a refund
Did you actually test it or just jump to conclusions like you did earlier? Because I also tested on a 54x100 model and 16x50 model and it still looks wrong. (see attached pics of both 54x100 tree and 16x50 tree) Did you test it the same way you "didn't see it anywhere in the song" even though its pretty obvious if you actually looked instead of jumping to conclusions or assuming I'm stupid? The main reason you don't see me with a lot of posts here is because I do research and lots of testing before posting any issues or frivolous questions.
#1 In the original post by Sean, he states that this Full House sequence was modified from the original 54x100 to look better on their 16x50 tree. So why should I expect it to only work on a 54x100 tree? Even then, it should look right on those two model sizes but it doesn't (see attached pics)
#2 If we are only supposed to expect these sequences to work on models that exactly match Sean's layout then what's the point of buying sequences? I don't think anybody else has the exact same layout as Sean.
#3 It worked beautifully with my models on version .54 even though my models don't exactly match Sean's. No problems copying the effects to different models. It didn't break by having different models, it broke by changing the code for pinwheel.
#4 With .55 and .56 even testing with Sean's layout (his xlights_rgbeffects.xml file is included with the purchased sequence) it doesn't render properly. See attached pic of Sean's house with two tree's and matrix all snapped at about the 3:30 mark where they should all be showing a non-rotating cross. Only the matrix is somewhat similar to the original intended effect.
I think I've provided more than enough evidence to show that the behaviour of this effect has changed in ways that negatively impact this sequence. I understand there were good reasons to rewrite the pinwheel effect. I started reporting this in the other thread because I thought it was a bug in the .55 pinwheel update and I also am aware that was addressed in .56. So now, this fantastic sequence is the unfortunate victim.
I moved the discussion over here because I believe the new updated pinwheel effect is going to impact anybody who purchases this sequence which makes use of the old pinwheel behavior. And I would like to believe that Lilia is deservedly proud of her work and would want to make sure that it looks just as good on the current xLights release.
One final note: I have tried deleting the .fseq file and rerender. Makes no difference. I've tried this with my own show directory+models, with Sean's rgbeffects file+directory, and with a completely new, separate test directly and models. Always the same.
