Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invert Scale Correction #60

Closed
neomonkeus opened this issue Nov 12, 2012 · 10 comments
Closed

Invert Scale Correction #60

neomonkeus opened this issue Nov 12, 2012 · 10 comments
Assignees
Labels
Bug Issue with specific functionality not working as intended but overall plugin is working. Improvement The issue improves on existing functionality User Interface Improve/impact how the user interacts with the plugin via Blender UI

Comments

@neomonkeus
Copy link
Member

The Scale factor is inverted. Importing with a scale correction of 10 results in the object being 10x smaller rather than 10x larger.

@amorilia
Copy link
Member

Scale correction is a setting shared between export and import - so it's just a convention. Personally I think it is psychologically easier to work with values like 10 or 100, rather than 0.1 or 0.01 (also more difficult to tune in blender's UI). Maybe all we need is better documentation of the meaning of the value (i.e. factor to make things smaller on import, and factor to make things larger again on export).

@neomonkeus
Copy link
Member Author

Well I imagine that a bigger number should result in the object being imported as bigger rather than smaller.

@amorilia
Copy link
Member

My point is that the same argument holds for export.

@neomonkeus
Copy link
Member Author

Yes but there is currently no way to distingush which way that scaling works. i.e does 10 result in SCALEx10 or SCALE*(1/10). In either case, I would prefer that scaling be such that the 10 would increase the relative scale both on import and export. If someone decides to scale it by 0.5 on import, the relative export mirrored should be 2 to ensure that import=export, which ultimately the user can change on export.

@amorilia
Copy link
Member

Well, to my understanding, scale correction of 10 means 10 nif units = 1 blender unit. Such explanation works either way, right?

@neomonkeus
Copy link
Member Author

Think we are getting towards the crux of the problem. Scale correction is a bit of a misnomer, something like "nif -> blender units".

The main issue I have is that the mental mappings do not match. Putting in every more increasing values, relatively decrease size for import. Either way a larger value should produce a larger result.I propose two changes;

Choose a base unit: Personally I would like the Blender unit, but the nif unit should be absolute. This also leaves scope for doing crazy stuff with pulling the internal mapping of units should we decide to add that as a feature?

Show don't tell. Have the UI show exactly what altering this value will do. This is easily to achieved. Display the 1 nif unit = x blender units, where x is caluculated based on the above assumption of how the value should work and have an update function to update the UI value of the "unit correction/mapping" display.

@amorilia
Copy link
Member

Agreed. This is an easy one (just needs updating of the description of the setting) - perhaps @Ghostwalker71 or @Aaron1178 want to take this one?

@amorilia
Copy link
Member

Whilst looking at some other code, I noticed that the description is already correct. So I guess it's just a matter of a choosing a better name.

@neomonkeus
Copy link
Member Author

I meant to post this link as it was what drew me to the issue in the first place - http://forums.nexusmods.com/index.php?/topic/828733-blender-not-exporting-scale/

@ghost ghost assigned Ghostwalker71 Mar 20, 2013
@Ghostwalker71
Copy link
Contributor

I finally managed to work in the changes needed to make scale correction function as it does in blender.

Ghostwalker71 added a commit that referenced this issue Jun 17, 2013
neomonkeus added a commit that referenced this issue Jun 17, 2013
issue #60 Invert Scale Correction. Scale correction now functions the
@neomonkeus neomonkeus self-assigned this Apr 1, 2016
TagnumElite pushed a commit to TagnumElite/blender_niftools_addon that referenced this issue Dec 5, 2019
TagnumElite pushed a commit to TagnumElite/blender_niftools_addon that referenced this issue Dec 5, 2019
issue niftools#60 Invert Scale Correction. Scale correction now functions the
neomonkeus pushed a commit to neomonkeus/blender_nif_plugin that referenced this issue Jan 21, 2020
neomonkeus added a commit to neomonkeus/blender_nif_plugin that referenced this issue Jan 21, 2020
issue niftools#60 Invert Scale Correction. Scale correction now functions the
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issue with specific functionality not working as intended but overall plugin is working. Improvement The issue improves on existing functionality User Interface Improve/impact how the user interacts with the plugin via Blender UI
Projects
None yet
Development

No branches or pull requests

3 participants