|
|
|
Awesome! So does this mean the flatten land tool can actually flatten now?
Seen it in action, very smooth and works a treat superb
 If this is assigned, it won't show up in searches when Lindens are looking for unclaimed issues
It would be most helpful if we could set the selected land's absolute target Z in meters by "the numbers". It's insanely difficult to do stuff by some vaguely defined "force". Give us access to the numbers please.
Version of Gig's patch for 1.20 RC1
I will take on the incorporation of this patch, or at least give feedback on its status. This will likely be a slow process as I am not primarily a viewer developer any more.
The internal JIRA contains this recommendation from the Rx team:
Pretty minor. Attached patch containing reordered ui elements per RX, generated against our internal featurettes branch. Should still work against most branches.
Screenshot of the rearranged UI (Dazzle skin, natch). Thorough!
Thanks for working on this. This plus the new snapshot options patch will really remove two thorns from my side that I have to deal with whenever I use a linden build.
I found some small bugs relating to the storage of the force setting – some of the code was confused whether the stored setting was the logarithm or the actual force. Corrected patch attached.
Committed to the featurettes branch. Realistically it will probably be at least a month before it percolates out to a public viewer.
While you're shuffling the terrain tools around, would you mind taking a look at
Attached screenshot (safer-terraform-gui.png) of suggested safer, more logical and less cramped GUI.
1) Apply to selection opposite Select Land and spaced away from other controls. 2) Brush controls grouped together with a heading. 3) Slider renamed to Strength. 4) "Show Owners" moved to Advanced controls. 5) Advanced controls grouped logically. To step back a minute... I'm not sure it makes sense to have redundant buttons here for abandon land, about land, and buy land. Show owners is probably best moved to the view menu.
I'd happily lose the Buy and Abandon buttons, but I do use the About button there while joining and subdividing parcels.
Show Owners is already a redundant copy of of View>Land Owners. Neither name is a particularly good description of what it does. For that reason I moved it down to the "Advanced" area, definitely advanced anyway as you have to know what the colours mean. Hesitate to remove it completely though; while it's not particularly useful, you can guarantee there's SOMEONE that uses it all the time I had a bit more of a play this evening, and ended up with safer-terraform-gui2. 1) Changed the brush size into a slider rather than a combo. Less clicks and takes up less space. 1) Trimmed "Land" off the end of the radio buttons, except the first. These are all a bit crowded still. In purist terms should probably be icons (with tool tips), but that's lots more work and would hold things up. 2) Moved the parcel area with the other parcel info stuff. 3) Renamed Show Owners to Show Status, with a tool tip to clarify a bit. It's meaningless unless you know what the different colours signify, so added a "?" button next to it which should display a colour key. The View>Land Owners menu item should probably also be renamed as Show Parcel Status to match. 4) Added titles to the sections, though I'm not convinced whether that's helpful or just clutter. GIGS! you stole my featurette!
i just sat down to implement a strength slider in the terraform tool - and i was stunned to see there's already one in there. great minds think alike. or something. Heh, you could polish this one a little bit if you want on the server side. The server reacts badly to certain fields having a large value for "seconds" in the packet.
The server seems to discard the seconds parameter for "smooth selection". For "revert selection", if it is given a large seconds value, the land takes on random values in my experimentation. I believe all the brushes respond well to this patch, but the selection/area tools have trouble on the server end. For this patch I just made the force slider not apply to packets that the server did strange things with, so this patch is still good to go, but it could use that server side polish if you want. WooHoo Accepted.
I love this patch (terraforming in SL is my "job") and will be glad to see it make the official viewer. Was playing around with the strength slider and found an issue (1.21rc1). If you set the strength slider to MAX and use REVERT to revert land that is slightly off of the default it goes WILD. I have spires reaching 200m in the air on mainland and some go all the way down to 0.
I am leaving a sample here http://slurl.com/secondlife/Dimitrios/101/99/51 No body better AR me for leaving this proof of bug here... This was a known issue that I discovered during the development of this patch about 8 months ago. At that time I filed a report with security@lindenlab.com, as well as reporting it in person to several lindens, letting them know the griefing potential of this.
My original patch deliberately ignored the power setting when select land revert was used, to avoid triggering this bug prior to it being fixed. Somehow that was lost in the process of merging this to Linden code. (The real fix for this of course is to fix the server side) One of my estate managers was showing me the glitch today with the revert tool. She was able to make a hole down to 'zero' altitude, and after that, the terrain's raw file appeared to be corrupted. I downloaded it to see what it looked like, and both the height field and the multiply field were entirely at zero (black). I restarted the region, and after that the hole had turned into a 512m high spire, so I downloaded the raw file again to see if it was different. Still both height and multiply were entirely zero – this is for the entire region mind you, not just the parcel she was using to demonstrate.
Just thought I'd pass this along, since it definately seems to have a serverside component to the problem. It's entirely server side. The client can "hide" the problem like it did before these new tools were added, but the fix has to happen on the server side.
Hey, can y'all create a separate SVC issue for the wild n' crazy Revert behavior? I agree with Gigs – the server-side fix is the real one. We could remove the slider control from the Revert tool, too, I guess, but that seems like a paper fix.
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 |
|
New Feature
Resolved
Normal
Terraform tool variable "strength"
Obviously there's a lot of tweakables here. The lowest setting is about half of the current tools, the highest is about 20x as powerful for brushes, about 5 times as powerful for raise/lower selection.
Note that some "selection" operations ignore the strength parameter on the server side (such as roughen).