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

add leastDistance as OptimisationMethodEnumeration #242

Merged
merged 2 commits into from
Oct 11, 2022

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Sep 12, 2022

replaces: #223

@sgrossberndt
Copy link
Contributor

https://htmlpreview.github.io/?https://github.com/VDVde/OJP/blob/OptimisationMethod_extended/docs/generated/OJP.html

The annotation "Least distance in meter. Mostly used for ALTERNATIVE MODE OF OPERATION" will not be displayed in the HTML

@ue71603
Copy link
Contributor Author

ue71603 commented Oct 4, 2022

@sgrossberndt What should I do to correct this?

@sgrossberndt
Copy link
Contributor

I approved the change. I don't see an easy option to fix this. The current HTML definition simply does not display annotations for constants. The annotation is for now just visible in the XSD source. I just wanted to give this hint so other reviewers know why it is not displayed.

@ue71603
Copy link
Contributor Author

ue71603 commented Oct 5, 2022

Thanks for the

I approved the change. I don't see an easy option to fix this. The current HTML definition simply does not display annotations for constants. The annotation is for now just visible in the XSD source. I just wanted to give this hint so other reviewers know why it is not displayed.

@skinkie It might be rather difficult to extend this part in the XSL. What do you think, we leave it at that? So the detailed description of the ENUM is only in the XSD? The values are in the doc.

@sgrossberndt sgrossberndt merged commit df70e37 into changes_for_v1.1 Oct 11, 2022
@sgrossberndt sgrossberndt deleted the OptimisationMethod_extended branch October 11, 2022 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc updated enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants