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.

Sign up for GitHub

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

Jump to bottom

AeroPerformanceMap: camelCase for damping derivatives #686

Closed
MarAlder opened this issue Feb 4, 2021 · 1 comment
Closed

AeroPerformanceMap: camelCase for damping derivatives #686

MarAlder opened this issue Feb 4, 2021 · 1 comment
Labels
breaking change This modification eliminates the compatibility of older data sets
Milestone
cpacs 3.3

Comments

@MarAlder
Copy link
Collaborator

MarAlder commented Feb 4, 2021 •
edited
Loading

cpacs/vehicles/aircraft/model/analyses/aeroPerformance/aeroMap/aeroPerformanceMap/dampingDerivatives/negative(positive)Rates:

For consistent nomenclature in CPACS 3.3 (aeroCaseStateType and fqRatingsType), we suggest capitalizing the S in d..d..Star:

We should keep treating the whole differential operator as a single "word". As an alternative to the above proposal, one could of course write everything in lower case. However, since star is an independent term, it can be better interpreted this way.

grafik
@MarAlder MarAlder added the breaking change This modification eliminates the compatibility of older data sets label Feb 4, 2021
MarAlder added a commit that referenced this issue Feb 4, 2021
@MarAlder
capitalizing Star in dampingDerivatives (#686)
2005ad2
@MarAlder MarAlder added this to the cpacs 3.3 milestone Feb 4, 2021
@MarAlder
Copy link
Collaborator Author

No objection - will introduced like this in the v3.3-RC version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change This modification eliminates the compatibility of older data sets
Projects
None yet
Milestone
cpacs 3.3
Development

No branches or pull requests
1 participant
@MarAlder