#112 Labels should be importance weighted

v1.5
closed-fixed
Core (65)
5
2007-12-10
2007-10-25
Dawoon Jung
No

Here is a patch I posted on the forum some time back,
that corrects locations slider behavior. A small change needs to be made to render.cpp, and each
platform gui's locations slider may need to be tweaked a little:

http://shatters.net/forum/viewtopic.php?t=11390

(Oct 17) Updated to take into account fov:

float d_vis = (14.8f + 86.9f * effSize) * (float)
sqrt(minFeatureSize/50.0f);

if ((fov/FOV)*cpos.distanceFromOrigin() < d_vis &&
(cpos - origin) * viewNormald > 0.0)

I use only (fov/FOV), not pixelSize, because pixelSize is a function of window area and I don't want to scale the label density with window area.

Discussion

  • Dawoon Jung

    Dawoon Jung - 2007-10-25
    • labels: 602284 --> Core
    • assigned_to: nobody --> cjlaurel
     
  • Chris Laurel

    Chris Laurel - 2007-10-25

    Logged In: YES
    user_id=159994
    Originator: NO

    But for a given field of view, doesn't it make sense to have fewer labels when there are less pixels to fit them on? Otherwise, users running at low resolutions could end up with overlapping labels.

     
  • Dawoon Jung

    Dawoon Jung - 2007-10-26

    Logged In: YES
    user_id=882500
    Originator: YES

    Yes, but this patch already ensures the number of labels varies with FOV. The label density is the same, but the total number of visible labels (density * area) will adapt.

     
  • Dawoon Jung

    Dawoon Jung - 2007-10-29

    Logged In: YES
    user_id=882500
    Originator: YES

    Ah, sorry I misunderstood what you were saying.
    Changing the FOV via the "," and "." keys produces the desired behavior
    but resizing the window/view (which also ends up changing the FOV) indeed looks incorrect.
    I'll have another go at this patch..

     
  • Dawoon Jung

    Dawoon Jung - 2007-10-30

    Logged In: YES
    user_id=882500
    Originator: YES

    File Added: render.cpp.patch

     
  • Dawoon Jung

    Dawoon Jung - 2007-10-30

    Patch (with fixes for FOV, window size)

     
  • Chris Laurel

    Chris Laurel - 2007-11-16
    • milestone: --> v1.5
     
  • Chris Laurel

    Chris Laurel - 2007-12-10
    • status: open --> closed
     
  • Chris Laurel

    Chris Laurel - 2007-12-10
    • status: closed --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks