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

Implement writeTreeDTD() #699

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

galou
Copy link
Contributor

@galou galou commented Nov 13, 2023

Add <?xml-model href="tree.dtd" type="application/xml-dtd"?> or <!DOCTYPE root SYSTEM "tree.dtd"> to use the benefits of DTD.

Signed-off-by: Gaël Écorchard <gael@km-robotics.cz>
@galou
Copy link
Contributor Author

galou commented Nov 13, 2023

I'm actually not sure whether <include/> is allowed within <TreeNodesModel/> or <BehaviorTree/>.

@galou
Copy link
Contributor Author

galou commented Nov 13, 2023

What about <SubTree/> within <TreeNodesModel/>? If they are accepted, I have the issue that DTD is not powerful enough to describe differently a <SubTree/> within <root/> and within <TreeNodesModel/> (I don't know much about DTD).

Signed-off-by: Gaël Écorchard <gael@km-robotics.cz>
@galou
Copy link
Contributor Author

galou commented Dec 22, 2023

@facontidavide what is blocking merging?

@facontidavide
Copy link
Collaborator

Could you build an example showing how this works in the folder examples?
Something like ex06_something_something.cpp

I'm actually not sure whether is allowed within or

It is not

What about within ? If they are accepted, I have the issue that DTD is not powerful enough to describe differently a within and within (I don't know much about DTD).

It is now. Look at example ex05_subtree_model

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants