Thanks for your quick comment Gordon. > To me Its not a bug , its an unsupported feature :) so you cannot really > reports it a as bug I agree that the distinction between features and bugs is a fine line most of the time.
Most projects use the tickets/bugs/issues system to keep track of features to be implemented in the next releases, or to document decisions related to not implementing specific features. > There are many features of what Collada can support that are not supported in > OSG or other apps. ( this is also the same of other model formats) > Not every feature of of every formats is needed in OSG in general or really > makes sense If OSG supports a particular format I would expect the extend of the support to be as wide as reasonably possible, and otherwise the areas not implemented should be clearly stated the documentation. > I for one would Not want to use camera data in my application when loading a > DAE model (or any format), my application controls the camera not a model, I > just want to load the models, what if I load 30 Dae models and they all have > different cameras , which one should I use :(m but that’s me and my apps Under that argument then the COLLADA module should not load any camera. However this clearly not the case. What makes me consider the current status more a bug than a "missing feature" is the mentioned piece of code and comment that mentions > http://www.openscenegraph.org/projects/osg/browser/OpenSceneGraph/trunk/src/osgPlugins/dae/daeRSceneObjects.cpp?rev=12791#L617 // TODO The current osg::CameraView does not support an orthographic view to my understanding in current OSG, CameraView does support orthographic view and is that sense, at the very least the comment is incorrect (and should be updated to explain why OSG will not implement this feature); or, more reasonably the missing feature can be easily implemented now. > Too me your request is more an Application specific thing and you could > extend your application to create that Ortho camera for you, it would be a > good way to learn more about OSG It is not specific to my application in the sense that, in my application, some renderings will be done using perspective projection and other ones with orthographic projections. In my application the camera information is meant to be stored in a file, and I considered the Collada files a reasonable (cross 3d tools) choice (to store both the camera information and the models). I cannot just hard code the camera once and for all. As I asked before, is still someone in the reading list familiar with this part of the code ? Any idea of how trivial or non trivial it is to fix this issue ? I am not familiar with the specifics of the camera matrices handling in OSG, but I would believe the issue to be "not very had" to resolve. Best regards, rodrigob. On Sat, Oct 8, 2011 at 10:24 PM, Gordon Tomlinson <[email protected]> wrote: > To me Its not a bug , its an unsupported feature :) so you cannot really > reports it a as bug > > There are many features of what Collada can support that are not supported in > OSG or other apps. ( this is also the same of other model formats) > Not every feature of of every formats is needed in OSG in general or really > makes sense > > I for one would Not want to use camera data in my application when loading a > DAE model (or any format), my application controls the camera not a model, I > just want to load the models, what if I load 30 Dae models and they all have > different cameras , which one should I use :(m but that’s me and my apps > > Too me your request is more an Application specific thing and you could > extend your application to create that Ortho camera for you, it would be a > good way to learn more about OSG > > > > > ______________________________________________________________________________ > Gordon Tomlinson > > [email protected] > www.photographybyGordon.com > www.vis-sim.com > www.gordontomlinson.com > IM: [email protected] > ______________________________________________________________________________ > > -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf Of Rodrigo > Benenson > Sent: Saturday, October 08, 2011 9:26 AM > To: OpenSceneGraph Users > Subject: [osg-users] BUG report: Collada orthographic support > > Thanks Jean-Sébastien for the quick answer. > If the mailing list is the bug report channel, then here I go: > > I am currently working on an application that uses openscenegraph to > render collada animations. > When running my code on the collada files of my interest I get the > following warning: > > Orthographic in <camera> 'Camera-camera' not supported > > not support orthographic projection will clearly "mess up" the rendering > output. > Looking into the code I found > http://www.openscenegraph.org/projects/osg/browser/OpenSceneGraph/trunk/src/osgPlugins/dae/daeRSceneObjects.cpp?rev=12791#L617 > > 616 // TODO The current osg::CameraView does not support an > orthographic view > 617 OSG_WARN << "Orthographic in <camera> '" << > dcamera->getId() << "' not supported" << std::endl; > > Looking at the code it seems that the information from the collada > file is properly extracted, but the actual CameraView setup is not > "supported". > However looking at the examples there is for instance > http://www.openscenegraph.org/projects/osg/browser/OpenSceneGraph/trunk/examples/osghud/osghud.cpp#L47 > > which shows that orthographic projection is supported in the current > version of OpenSceneGraph. > > In my eyes then, implementing the support of orthographic projection > when loading collada file seems like a missing feature that should be > documented in a bug report. > > I am not familiar enough with OpenSceneGraph to just "come a patch" > but the issues seems simple enough to fix. Is anyone familiar with > this part of the code ? > > Regards, > rodrigob. > > > On Sat, Oct 8, 2011 at 2:11 PM, Jean-Sébastien Guay > <[email protected]> wrote: >> Hi Rodrigo, >> >>> I just spent 20 minutes looking around in >>> http://www.openscenegraph.org I could not find how users are supposed >>> to report bugs (I did find how to send patches, but not how to report >>> bugs). >>> >>> Could someone point me out how to do this ? >> >> Actually you just found it :-) >> >> The OSG project uses the mailing list for pretty much everything. That's >> just the way it has always been. There are plans to change this process in >> the future, which leads to the next point: >> >>> ps: I am surprised that emails are used for patches, instead of the >>> facilities of https://github.com/openscenegraph/osg >> >> The official OSG repo is hosted on openscenegraph.org and uses SVN. It is >> not using git yet, though there are plans in the coming year to move to git. >> The github repo is a mirror of the openscenegraph SVN repo maintained by >> members of this mailing list. So patches you would submit on github would >> just be lost. >> >> Hope this helps, >> >> J-S >> -- >> ______________________________________________________ >> Jean-Sébastien Guay [email protected] >> http://www.cm-labs.com/ >> http://whitestar02.dyndns-web.com/ >> _______________________________________________ >> osg-users mailing list >> [email protected] >> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org >> > _______________________________________________ > osg-users mailing list > [email protected] > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > > _______________________________________________ > osg-users mailing list > [email protected] > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > _______________________________________________ osg-users mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
