HiRISE DTM from PDS IMG (Update) #51754

Closed
opened 2017-06-08 23:07:58 +02:00 by Nicholas Wolf · 21 comments

Hello,

I work on the team responsible for the HiRISE DTM from PDS IMG Blender add-on. The person on our team that originally wrote the add-on is long since gone, and I don't think it has been updated since Blender 2.55. I recently wanted to use our data in Blender but discovered a number of issues with the add-on.

I've overhauled the add-on for our use here and would like to submit this new version to be included with Blender (and to act as its maintainer). The main updates/improvements over the previous version:

  • Fixed incorrect UV map generation
  • Fixed mesh positioning/scale issues
  • Readability improvements
  • Organized module layout
  • Speed improvements, mostly from optimizing the down-sampling algorithm used in the previous version
  • Reload terrain meshes at higher resolutions while preserving textures, location, scale, rotation, etc.
  • Panel that displays metadata about terrain meshes

I wasn't sure if you would want me to update the wiki page for the old version of the add-on or to create a new wiki page. I've attached the HTML documentation that I'm currently building from the add-on (docs.zip) and will be happy to transcribe the information in there to wherever is most appropriate. The "quickstart" page, there, gives a brief overview of using the add-on.

The add-on code itself is attached (hirise-dtm-importer.zip) and has been tested with Blender 2.78 on Windows 7, Mac OS X, and Ubuntu Linux.

hirise-dtm-importer.zip
docs.zip

Cheers,
Nic Wolf

Hello, I work on the team responsible for the [HiRISE DTM from PDS IMG ](https://wiki.blender.org/index.php/Extensions:2.6/Py/Scripts/Import-Export/HiRISE_DTM_from_PDS_IMG) Blender add-on. The person on our team that originally wrote the add-on is long since gone, and I don't think it has been updated since Blender 2.55. I recently wanted to use our data in Blender but discovered a number of issues with the add-on. I've overhauled the add-on for our use here and would like to submit this new version to be included with Blender (and to act as its maintainer). The main updates/improvements over the previous version: - Fixed incorrect UV map generation - Fixed mesh positioning/scale issues - Readability improvements - Organized module layout - Speed improvements, mostly from optimizing the down-sampling algorithm used in the previous version - Reload terrain meshes at higher resolutions while preserving textures, location, scale, rotation, etc. - Panel that displays metadata about terrain meshes I wasn't sure if you would want me to update the wiki page for the old version of the add-on or to create a new wiki page. I've attached the HTML documentation that I'm currently building from the add-on (`docs.zip`) and will be happy to transcribe the information in there to wherever is most appropriate. The "quickstart" page, there, gives a brief overview of using the add-on. The add-on code itself is attached (`hirise-dtm-importer.zip`) and has been tested with Blender 2.78 on Windows 7, Mac OS X, and Ubuntu Linux. [hirise-dtm-importer.zip](https://archive.blender.org/developer/F625770/hirise-dtm-importer.zip) [docs.zip](https://archive.blender.org/developer/F625769/docs.zip) Cheers, Nic Wolf
Author

Changed status to: 'Open'

Changed status to: 'Open'
Author

Added subscriber: @nicwolf

Added subscriber: @nicwolf
Member

Added subscriber: @BrendonMurphy

Added subscriber: @BrendonMurphy
Brendon Murphy self-assigned this 2017-06-09 00:47:05 +02:00
Member

hi, feel free to update the wiki page as you need, If you need help with the wiki page i can help you, but I prefer if you can do.
Please also test the addon is working in current nightly builds. Thanks

hi, feel free to update the wiki page as you need, If you need help with the wiki page i can help you, but I prefer if you can do. Please also test the addon is working in current nightly builds. Thanks
Author

I can handle updating the wiki page, I've requested an account in bf-committers and will put the new content once that's gone through.

Checked the add-on against today's nightlies:

Linux: OK

Mac OSX: Throws a no module named numpy() error --- this seems like an issue with the build? The numpy egg is in .../site-packages but numpy itself isn't anywhere in the Python search path. Appending an entry to sys.path that points to numpy fixes the issue and the add-on works fine.

Windows: Crashes. Unsure of the cause, will investigate.

I can handle updating the wiki page, I've requested an account in bf-committers and will put the new content once that's gone through. Checked the add-on against today's nightlies: Linux: OK Mac OSX: Throws a `no module named numpy()` error --- this seems like an issue with the build? The `numpy` egg is in `.../site-packages` but `numpy` itself isn't anywhere in the Python search path. Appending an entry to `sys.path` that points to `numpy` fixes the issue and the add-on works fine. Windows: Crashes. Unsure of the cause, will investigate.
Member

Added subscriber: @LazyDodo

Added subscriber: @LazyDodo
Member

In #51754#439724, @nicwolf wrote:
Windows: Crashes. Unsure of the cause, will investigate.

I did some testing as well, this only seems to crash in the msvc2013 release build, debug build is fine, so are the msvc2015 builds. Have to look further into this

> In #51754#439724, @nicwolf wrote: > Windows: Crashes. Unsure of the cause, will investigate. I did some testing as well, this only seems to crash in the msvc2013 release build, debug build is fine, so are the msvc2015 builds. Have to look further into this
Member

Added subscribers: @MartijnBerger, @brecht

Added subscribers: @MartijnBerger, @brecht
Member

I fixed the windows build, the next nightly build should be working.

@MartijnBerger / @brecht mind taking a peek at the numpy issues on mac?

I fixed the windows build, the next nightly build should be working. @MartijnBerger / @brecht mind taking a peek at the numpy issues on mac?

I believe the numpy on macOS issue was solved yesterday in {rBL61895}.

I believe the numpy on macOS issue was solved yesterday in {rBL61895}.
Author

I will need wiki access to update the documentation, posting here at @BrendonMurphy's request.

I will need wiki access to update the documentation, posting here at @BrendonMurphy's request.
Author

Tested against the current nightly on Mac OS X, the numpy issue persists but the add-on works.

Updated source, attached. Only change is the rename to 'io_convert_image_to_mesh_img'.

io_convert_image_to_mesh_img.zip

Tested against the current nightly on Mac OS X, the numpy issue persists but the add-on works. Updated source, attached. Only change is the rename to 'io_convert_image_to_mesh_img'. [io_convert_image_to_mesh_img.zip](https://archive.blender.org/developer/F628725/io_convert_image_to_mesh_img.zip)
Author

Tested OK on Windows and Ubuntu Linux.

Tested OK on Windows and Ubuntu Linux.
Member

hi, @nicwolf,
I just need to be certain this is good to go & which files it's supposed to be replacing.

https://developer.blender.org/diffusion/BA/browse/master/io_convert_image_to_mesh_img/ is the release addon

https://developer.blender.org/diffusion/BAC/browse/master/io_import_LRO_Lola_MGS_Mola_img.py is the contrib addon

are both these addons doing the same thing? do we need both?
is your addon a replace/upgrade for the release or contrib addon?

Thanks.

hi, @nicwolf, I just need to be certain this is good to go & which files it's supposed to be replacing. https://developer.blender.org/diffusion/BA/browse/master/io_convert_image_to_mesh_img/ is the release addon https://developer.blender.org/diffusion/BAC/browse/master/io_import_LRO_Lola_MGS_Mola_img.py is the contrib addon are both these addons doing the same thing? do we need both? is your addon a replace/upgrade for the release or contrib addon? Thanks.
Author

Hi @BrendonMurphy,

This is good to go and will be replacing the add-on in the release add-ons: https://developer.blender.org/diffusion/BA/browse/master/io_convert_image_to_mesh_img/

The two add-ons are unique --- the one in the contrib add-ons works with data from a different mission.

Hi @BrendonMurphy, This is good to go and will be replacing the add-on in the release add-ons: https://developer.blender.org/diffusion/BA/browse/master/io_convert_image_to_mesh_img/ The two add-ons are unique --- the one in the contrib add-ons works with data from a different mission.
Member

Committed, closing as resolved.
@nicwolf, feel free to send any updates

Committed, closing as resolved. @nicwolf, feel free to send any updates
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Member

thanks

thanks
Author

Hey @BrendonMurphy, I would like to update the (now outdated) documentation for the old version of this add-on: https://developer.blender.org/diffusion/BA/browse/master/io_convert_image_to_mesh_img/

Responses to my request for a wiki account on bf-comitters suggested that I raise the request here?

Hey @BrendonMurphy, I would like to update the (now outdated) documentation for the old version of this add-on: https://developer.blender.org/diffusion/BA/browse/master/io_convert_image_to_mesh_img/ Responses to my request for a wiki account on bf-comitters suggested that I raise the request here?

Added subscriber: @phaseIV-3

Added subscriber: @phaseIV-3

If anyone reads this, the functionality of the Hirise DTM importer has been restored and works with Blender 2.8+.
The updated addon can be found at https://github.com/phaseIV/Blender-Hirise-DTM-Importer

If anyone reads this, the functionality of the Hirise DTM importer has been restored and works with Blender 2.8+. The updated addon can be found at https://github.com/phaseIV/Blender-Hirise-DTM-Importer
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#51754
No description provided.