hongshaorou/openrocket
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[fixes #1164] Account for different RockSim FinSet location directions

Browse Source
This commit is contained in:
SiboVG 2022-02-28 00:24:14 +01:00
parent d614c77fa7
commit d2acd0ed01
1 changed files with 22 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
core/src/net/sf/openrocket/file/rocksim/importt/FinSetHandler.java
View File

@ -146,6 +146,11 @@ class FinSetHandler extends AbstractElementHandler {
private final RockSimAppearanceBuilder appearanceBuilder;
/**
* Checks whether the location is already loaded in or not
*/
private boolean locationLoaded = false;
/**
* Constructor.
*
@ -180,12 +185,27 @@ class FinSetHandler extends AbstractElementHandler {
finish = RocksimFinishCode.fromCode(Integer.parseInt(content)).asOpenRocket();
}
if (RocksimCommonConstants.XB.equals(element)) {
// Opposite value accounts for the different relative distance directions used
location = Double.parseDouble(content) / RocksimCommonConstants.ROCKSIM_TO_OPENROCKET_LENGTH;
// Account for the different relative distance directions used
// Issue Ref: https://github.com/openrocket/openrocket/issues/881
location = -Double.parseDouble(content) / RocksimCommonConstants.ROCKSIM_TO_OPENROCKET_LENGTH;
if (axialMethod != null) {
if (axialMethod == AxialMethod.BOTTOM) {
location = -location;
}
}
this.locationLoaded = true;
}
if (RocksimCommonConstants.LOCATION_MODE.equals(element)) {
axialMethod = RocksimLocationMode.fromCode(Integer.parseInt(content)).asOpenRocket();
// If the location is loaded before the axialMethod, we still need to correct for the different relative distance directions
if (locationLoaded) {
if (axialMethod == AxialMethod.BOTTOM) {
location = -location;
}
}
}
if (RocksimCommonConstants.FIN_COUNT.equals(element)) {
finCount = Integer.parseInt(content);