-
Notifications
You must be signed in to change notification settings - Fork 630
Support for a single model.xml file
#2291
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
Merged
Merged
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
2555b95
Factor out to_xml_element methods for XML classes
pshriwise ad746cb
Writing all main nodes to a single XML file
pshriwise a09a61e
Writing all output under a single root node
pshriwise 7a9d8c9
Keeping old option to write separate XMLs so I can still use to tests…
pshriwise 0409652
Adding signatures for reading information from an XML node where nece…
pshriwise cdd2a6b
Correcting doc scope and some reads
pshriwise bec9bf6
Refactoring model read into it's own function.
pshriwise 81aa653
Factoring out some common lines between reader functions
pshriwise 9391f17
Adding import for single or multiple XMLs
pshriwise af6b0af
Fix loop variable overwrite
pshriwise 5ac6e87
Properly call read_plots_xml
pshriwise 2e71658
Other bug fixes
pshriwise b4dcc30
Removing walrus operator
pshriwise 611475a
Restructureing input function calls a little
pshriwise 021bb28
Adding initial test for exporting model.xmls. Correcting material maps
pshriwise abd3d51
Correcting doc strings for material mapping sent to
pshriwise 662d83a
Running some existing regression tests using single XML file
pshriwise 1e7dbe9
Making sure meshes are only written once to a model.xml file
pshriwise d034e1d
Apply suggestions from code review
pshriwise b76825f
Addressing some comments from PR
pshriwise d984716
Adding ability to specify an input filename to the executable or to t…
pshriwise 366f9e9
Test comment and updated error messages
pshriwise 1e6cb68
Addressing a few more comments in C++ code
pshriwise 7a203fb
Correcting parenthesis
pshriwise fe47d56
Spot check in error message from failed run
pshriwise 6f7a6fe
Updating model XML file format w/ indentation.
pshriwise 5d1ead2
Addressing some more PR comments. A couple of typo fixes too.
pshriwise e7f95b0
Docstring correction and a note about the recursion
pshriwise b3225be
Adding executor tests
pshriwise 72ff942
Checking RuntimeError message correctly
pshriwise 7f5d2a5
Refactoring model_xml regression tests a bit and adding input checking
pshriwise 3f4a9eb
Reordering geometry XML element attributes
pshriwise 83cf684
Updating expected inputs with reordering
pshriwise 2c27d10
Moving other relevant XML reorder calls and updating expected input a…
pshriwise cd7c3bf
Updates to import/export methods suggested by @paulromano
pshriwise ef2f690
Updating handling of single XML input to leverage existing settings::…
pshriwise 574845b
Moving Model.from_separate_xmls back to Model.from_xml.
pshriwise 270331a
Adding function to check if a path is a directory
pshriwise bc1791d
Updates to simplify model filename checking
pshriwise 5634d83
Changing name of input argument for Python executor
pshriwise d639d1d
Docstring correction
pshriwise 939adbe
Updating tests to reflect changes to API
pshriwise 7fdfed2
Some cleanup of file utils
pshriwise 55d77cf
Small improvements to read_model_xml
pshriwise 7873b4d
Update tests/regression_tests/model_xml/test.py
pshriwise ef13643
Some housekeeping in the Python classes
pshriwise 9c942db
Correct use of mesh memo in single XML export
pshriwise 916863f
Renaming and docstring/comment updates
pshriwise 145594e
Apply @paulromano suggestions from code review
pshriwise f3bcad3
Addressing comments from @paulromano
pshriwise 43687e5
Improve comment on recursion arguments
pshriwise 2eca57a
Adding mesh memo to Settings.from_xml_element
pshriwise 25f2bd5
Several small fixes
paulromano 64aee3b
Merge pull request #16 from paulromano/model-xml-pr
pshriwise File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.