Skip to content
This repository has been archived by the owner on May 18, 2019. It is now read-only.

Low-res bricks import as normal-res #74

Closed
rioforce opened this issue May 15, 2014 · 3 comments
Closed

Low-res bricks import as normal-res #74

rioforce opened this issue May 15, 2014 · 3 comments
Labels

Comments

@rioforce
Copy link
Collaborator

The Low-res primitives do not actually import when you select the option. They import as normal-res primitives. High res works though. Any thoughts on a fix?

@rioforce rioforce added the bug label May 15, 2014
@le717 le717 added this to the 1.3 milestone May 15, 2014
@le717
Copy link
Owner

le717 commented May 15, 2014

Confirmed. I tested the script at 2335b1d, which was the resulting commit from #56 and the v1.1.5 release and it worked both times. So this broke sometime between the v1.1.5 release and now.

git bisect to the rescue!

@rioforce
Copy link
Collaborator Author

Actually, it was a mistake on my end of the importer. For some reason, I never installed the update to LDraw, and the script didn't tell me that it couldn't find the "8" folder, and it used normal-res instead. Do you still have this issue, @le717?

@le717
Copy link
Owner

le717 commented May 16, 2014

I never actually saw it. My confirmation came from looking at what you were telling me. 😛

This has brought up a bigger issue: no warning if the high res, or low res options are selected but the folders are missing. Closing this issue now, will open a new one about possible warning messages later.

@le717 le717 closed this as completed May 16, 2014
@le717 le717 added invalid and removed bug labels May 16, 2014
@le717 le717 removed this from the 1.2 milestone May 16, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants