Collada - Support for Camera's sensor Extension (FCOLLADA, Modo401, blender) #32834

New Issue

Philippe Blanc · 2012-10-09T23:46:18+02:00

Philippe Blanc commented

2012-10-09 23:46:18 +02:00

%%%Add camera's sensor import for to the following Collada extension:

FCOLLADA - http://collada.org/mediawiki/index.php/Camera_FCollada_extension
Modo401 - http://collada.org/mediawiki/index.php/Imager_modo_401_extension
blender - new proposed extension

I am working on an export patch as well.

The patch is straightforward and should integrate nicely with the current blender's Collada code. The only tricky part is how I added support for the tag, which is a little on the hackish side but:

work well
doesn't disturb the current code
can be reused by future Collada extension%%%

%%%Add camera's sensor import for to the following Collada extension: FCOLLADA - http://collada.org/mediawiki/index.php/Camera_FCollada_extension Modo401 - http://collada.org/mediawiki/index.php/Imager_modo_401_extension blender - new proposed extension I am working on an export patch as well. The patch is straightforward and should integrate nicely with the current blender's Collada code. The only tricky part is how I added support for the <param> tag, which is a little on the hackish side but: - work well - doesn't disturb the current code - can be reused by future Collada extension%%%

Philippe Blanc commented

2012-10-09 23:46:18 +02:00

Changed status to: 'Open'

Philippe Blanc commented

2012-10-10 04:21:05 +02:00

%%%Attached a proof of concept on what it will looks like to export the camera sensor using FCOLLADA, Modo401 and Blender Extension. Modo401 is giving me some hard time:
a) The documentation @http://collada.org/mediawiki/index.php/Imager_modo_401_extension is not really clear about the use of the tag instead of the tag. My previous patch will load the Modo401 extension regardless but I don't have a copy of Modo for testing it.
b) Collada API doesn't seem to provide a way to define and then write a tag with all the needed parameters (name, type). There are a work around by using addExtraTechniqueTextblock but that seem hackish at best (not sure what need to happen for the end of line). More information about that here: https://code.google.com/p/opencollada/source/search?q=addExtraTechniqueParameter&origq=addExtraTechniqueParameter&btnG=Search+Trunk%%%

%%%Attached a proof of concept on what it will looks like to export the camera sensor using FCOLLADA, Modo401 and Blender Extension. Modo401 is giving me some hard time: a) The documentation @http://collada.org/mediawiki/index.php/Imager_modo_401_extension is not really clear about the use of the <imager> tag instead of the <extra> tag. My previous patch will load the Modo401 extension regardless but I don't have a copy of Modo for testing it. b) Collada API doesn't seem to provide a way to define and then write a <param> tag with all the needed parameters (name, type). There are a work around by using addExtraTechniqueTextblock but that seem hackish at best (not sure what need to happen for the end of line). More information about that here: https://code.google.com/p/opencollada/source/search?q=addExtraTechniqueParameter&origq=addExtraTechniqueParameter&btnG=Search+Trunk%%%

Philippe Blanc commented

2012-11-06 13:02:10 +01:00

%%%ping! If you need me to resubmit the patch or if you have any questions...%%%

Philippe Blanc commented

2013-04-30 01:52:53 +02:00

%%%ping! any thought? not necessary on the patch itself but at least on the functionality? Should I rework it?%%%

Nathan Letwory commented

2014-05-12 20:53:57 +02:00

Added subscriber: @jesterking

Nathan Letwory commented

2014-05-12 21:05:42 +02:00

@visualfox, I've started working through open COLLADA tasks. Could you update your patch to current master? I have a hard time getting a good grip of the code, since the source code lines don't seem to match anymore.

I'd appreciate it if you could do the update for your patch as a differential edit. Thanks!

@visualfox, I've started working through open COLLADA tasks. Could you update your patch to current master? I have a hard time getting a good grip of the code, since the source code lines don't seem to match anymore. I'd appreciate it if you could do the update for your patch as a differential edit. Thanks!

Philippe Blanc commented

2014-05-28 16:14:53 +02:00

@jesterking I will try to do that this weekend. I will have to reset a working environement for it. I may break down the patch in smaller bit.

Thank you for reviewing the patch, I appreciate it

@jesterking I will try to do that this weekend. I will have to reset a working environement for it. I may break down the patch in smaller bit. Thank you for reviewing the patch, I appreciate it

Brendon Murphy commented

2015-09-08 04:49:52 +02:00

Closed as duplicate of #43298

Brendon Murphy closed this issue

2015-09-08 04:49:52 +02:00

Campbell Barton commented

2015-09-08 05:31:58 +02:00

Changed status from 'Duplicate' to: 'Open'

Campbell Barton reopened this issue

2015-09-08 05:31:58 +02:00

Dalai Felinto commented

2019-12-23 18:47:40 +01:00

Added subscriber: @dfelinto

Dalai Felinto commented

2019-12-23 18:47:40 +01:00

Changed status from 'Confirmed' to: 'Archived'

Dalai Felinto closed this issue

2019-12-23 18:47:40 +01:00

Dalai Felinto commented

2019-12-23 18:47:40 +01:00

Hi, thanks for your patch.

We are undergoing a Tracker Curfew where we are automatically closing old patches.

If you think the patch is still relevant please update and re-submit it. For new features make sure there is a clear design from the user level perspective.

Hi, thanks for your patch. We are undergoing a [Tracker Curfew ](https://code.blender.org/?p=3861) where we are automatically closing old patches. If you think the patch is still relevant please update and re-submit it. For new features make sure there is a clear design from the user level perspective.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Collada - Support for Camera's sensor Extension (FCOLLADA, Modo401, blender) #32834