geocraft icon indicating copy to clipboard operation
geocraft copied to clipboard

Add 'horizontal scale' and 'elevation scale' options

Open amyworrall opened this issue 8 years ago • 4 comments

We think our current project would look better if everything (both distances and heights) were scaled by a factor of 1.2.

Any idea where I'd hook into in order to add that?

amyworrall avatar Jul 20 '16 18:07 amyworrall

Exactly the same place as before for distance. For height, these lines https://github.com/cgutteridge/geocraft/blob/master/lib/Minecraft/Projection.pm#L101 & 102 should be really easy to hack. They get the surface DSM (top of stuff like buildings) and terrain DTM (ground level of buildings and trees) in metres for a given lat/long. A simple multiplier there should fix it...

Gimmie a few minutes to look. It should be easy to add...

On 20/07/2016 19:57, amyworrall wrote:

We think our current project would look better if everything (both distances and heights) were scaled by a factor of 1.2.

Any idea where I'd hook into in order to add that?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cgutteridge/geocraft/issues/13, or mute the thread https://github.com/notifications/unsubscribe-auth/AAfK6zHcpANXozliJOTaLx-vC5S21izmks5qXm-vgaJpZM4JRGLk.

Christopher Gutteridge -- http://users.ecs.soton.ac.uk/cjg

University of Southampton Open Data Service: http://data.southampton.ac.uk/ You should read the ECS Web Team blog: http://blogs.ecs.soton.ac.uk/webteam/

cgutteridge avatar Jul 20 '16 21:07 cgutteridge

Hey. OK --scale option now committed. I found some time between Preacher, rum and arguing about the future of the labour party (or confusingly discussing it anyhow)

On 20/07/2016 19:57, amyworrall wrote:

We think our current project would look better if everything (both distances and heights) were scaled by a factor of 1.2.

Any idea where I'd hook into in order to add that?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cgutteridge/geocraft/issues/13, or mute the thread https://github.com/notifications/unsubscribe-auth/AAfK6zHcpANXozliJOTaLx-vC5S21izmks5qXm-vgaJpZM4JRGLk.

Christopher Gutteridge -- http://users.ecs.soton.ac.uk/cjg

University of Southampton Open Data Service: http://data.southampton.ac.uk/ You should read the ECS Web Team blog: http://blogs.ecs.soton.ac.uk/webteam/

cgutteridge avatar Jul 21 '16 00:07 cgutteridge

Loving your improvements!

Can generate_world.pl line 67 be

        "scale=f", \$scale,

So that we can use fractional scales? I've tested it and it works.

amyworrall avatar Jul 21 '16 16:07 amyworrall

done.

On 21/07/2016 17:46, amyworrall wrote:

Loving your improvements!

Can generate_world.pl line 67 be

|"scale=f", $scale, |

So that we can use fractional scales? I've tested it and it works.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cgutteridge/geocraft/issues/13#issuecomment-234313140, or mute the thread https://github.com/notifications/unsubscribe-auth/AAfK66HfsCwjPAdobCHt1yQKuov7ng_Dks5qX6JWgaJpZM4JRGLk.

Christopher Gutteridge -- http://users.ecs.soton.ac.uk/cjg

University of Southampton Open Data Service: http://data.southampton.ac.uk/ You should read the ECS Web Team blog: http://blogs.ecs.soton.ac.uk/webteam/

cgutteridge avatar Jul 21 '16 20:07 cgutteridge