Menu

#98 .x file importer swaps ambient and emissive mat props!

closed-fixed
Assimp lib (91)
1
2013-05-30
2012-10-25
No

Hello!

Suspected bug: the .x file importer reads emissive material properties, and interprets them as ambient material properties.

This seems strange, since emissive properties could be mapped to emissive properties, and ambient properties are usually a copy of diffuse material properties!

Even stranger, this seems to have been deliberately introduced between revision 25 and revision 41: http://assimp.svn.sourceforge.net/viewvc/assimp/trunk/code/XFileImporter.cpp?r1=25&r2=41

Is this a bug, or is there a deliberate reason for this behaviour? I've looked around, but couldn't see why this behaviour would be desirable...

Warm regards,
Alex

Discussion

  • Thomas Ziegenhagen

    • status: open --> open-fixed
     
  • Thomas Ziegenhagen

    Committed as R1322. But I'm not sure if this actually fixes the issue. The documentation I found on the net is contradicting as to what these material colours really mean. I just hope I didn't destroy anyone's code with this.

     
  • Alexander Zawadzki

    • priority: 5 --> 1
    • assigned_to: nobody --> ulfjorensen
     
  • Alexander Zawadzki

    Thanks!

    I hope that it won't destroy anyone's code. It should do what (most people) would expect now unless there's anything really funny going on...

    Best wishes,
    Alex

     
  • Nobody/Anonymous

    The best defiance gold http://www.mmogm.com/defiance/ certainly are magnitude Six nevertheless appear to another Seventy two. I quite like I obtain a second approach. I didnrrrt have to get into the irritation of the need cruise ship backside have to wait and then suddenly attain the defiance gold http://www.mmogm.com/defiance/ I want to. I like these folks were taylor more effective. I provide another 71/2 and I think it is exactly what I should have got requested. If organizations an outlet We could have taken the situation straight back to I would have got. These are undoubtedly my personal earliest and I think persist match since it is extremely tricky to try and do one thing about them now.

     
  • Nobody/Anonymous

    SourceForge.net: Open Asset Import Library: Detail: 3580054 - .x file importer swaps ambient and emissive mat props! fmyxhp <a href="http://suvcwaz.org/redbottomshoes/" title="christian louboutin sale">christian louboutin sale</a>

     
  • Nobody/Anonymous

    SourceForge.net: Open Asset Import Library: Detail: 3580054 - .x file importer swaps ambient and emissive mat props! mhzepef [url=http://www.kingdomofsmiles.com/]christian louboutin replica[/url]

     
  • Thomas Ziegenhagen

    • status: open-fixed --> closed-fixed