Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0008292opensim[REGION] OpenSim Corepublic2018-02-17 15:352018-02-19 16:14
Assigned To 
PlatformOSOS Version
Product Versionmaster (dev code) 
Target VersionFixed in Version 
Summary0008292: terrain load-tile and save-tile fail with varregions
DescriptionTerrain load-tile does not appear to function correctly with varregions.
Steps To ReproduceSetup 3x3 grid of 1024x1024 varregions at (1000, 1000)

Be in the (root) region.

Run command terrain load-tile filename.png 12 12 1000 1000. This will throw invalid pointer exception.

Run command terrain load-tile filename.png 4 4 1000 1000. This will only load the first terrain tile.

Run command terrain save-tile output.png 12 12 1000 1000. Will throw invalid pointer exception.
TagsNo tags attached.
Git Revision or version number
Run Mode Grid (Multiple Regions per Sim)
Physics EngineBulletSim
EnvironmentMono / Linux32
Mono VersionOther
Attached Files

- Relationships

-  Notes
TomTheDragon (reporter)
2018-02-17 15:41

I am preparing a patch for review once I have run some more in-depth testing.
UbitUmarov (administrator)
2018-02-17 16:24

well at this point i can only tell that code doesn't make much sense to me
neither does a comand based on grid positions.
sounds like mega regions logic, with incorrect upgrade to var regions, we didn't noticed till now :(
i will wait for your patch, since you are already working on it
UbitUmarov (administrator)
2018-02-17 16:31

btw looks like part of the issue is the use off current region dimensions, where it should be Constants.RegionSize (ie 256m)
TomTheDragon (reporter)
2018-02-17 17:04

Yes, part of the issue is definitely use of current region dimensions. They are still required, but many should be changed. This code is a mess, trying to clean it up a little...

How does a command based on grid positions not make sense? If you have another preference, please let me know and I will see if it can possibly be implemented. I don't see any other way of specifying the bottom-left corner region.
UbitUmarov (administrator)
2018-02-17 17:09

those are region relative coords.. made sense in megaregions times
those comands are just obsolete.
note also that the load will only load a tile, losing all the rest of the region
(by tile the code assumed 256x256 i think)
again made sense on mega regions...
TomTheDragon (reporter)
2018-02-17 18:25

Is there a new standard for indexing Varregions? The wiki page for Varregion states "the region coordinates are specified in 256m region count". Should I be using something else?

With my current, minimal changes, the load-tile will load an entire file in at the given grid location.
UbitUmarov (administrator)
2018-02-17 18:28

grid positions are in multiples of 256m so, no changes
i just don't see why use them to load/save heighmaps when we are on the region console. Made sense in mega region days...
TomTheDragon (reporter)
2018-02-17 18:35

It's mainly because we have multiple regions for sims. We are at the region console but we don't know which regions we want it to affect.
UbitUmarov (administrator)
2018-02-17 18:53

one could change to the relevant region on console, making it a lot more easier than remembering grid locations etc
but never the less we still need a fixed loadtile comand to suport files saved on megaregions.
watcher64 (reporter)
2018-02-19 16:14

"We are at the region console but we don't know which regions we want it to affect."

This confuses me ..

various console commands to see what regions you want to affect ..

show regions
change region
terrain load
terrain save

- Issue History
Date Modified Username Field Change
2018-02-17 15:35 TomTheDragon New Issue
2018-02-17 15:41 TomTheDragon Note Added: 0032547
2018-02-17 16:24 UbitUmarov Note Added: 0032549
2018-02-17 16:31 UbitUmarov Note Added: 0032550
2018-02-17 17:04 TomTheDragon Note Added: 0032551
2018-02-17 17:09 UbitUmarov Note Added: 0032552
2018-02-17 18:25 TomTheDragon Note Added: 0032553
2018-02-17 18:28 UbitUmarov Note Added: 0032554
2018-02-17 18:35 TomTheDragon Note Added: 0032555
2018-02-17 18:53 UbitUmarov Note Added: 0032556
2018-02-19 16:14 watcher64 Note Added: 0032557

Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker