-
Notifications
You must be signed in to change notification settings - Fork 32
Remove all spaces after semi colons in the APP6-B STYLX #291
Comments
@BobBooth Please add additional information to this issue. Add the link to the "Comp Test Data" that is referenced in the table. Add more information as to what Runtime version you tested this on. Add the screenshots of both the expected symbol and the symbol that is rendered in Runtime in your issue. Do these same symbols render correctly using the style file in the ArcMap solution? |
Overlapping set of problems detected in ArcMap using the Military Features Append Military Features Workspace tool. The tool ran on import of the Comprehensive test data, but reported errors. |
@topowright - there are a few different image compare tools and ways to approach this. But this one might be good if you go with an HTML image table approach. This might justify creating a new issue to capture this testing capability. |
Moved in progress for QA task to create and doc app to export images and create verification table |
Removing QA label: Added style to SIDC image export test app at: And used app to create a more complete set of Image Test Outputs for App6b from the style file: Just the issues observed (about 20 symbols aren't being created): The entire set of symbols: This independently confirmed what @BobBooth reported above with the exception of the minefield control measures. It looks like the style is expecting these to be polygon symbols and we had these as point symbols in the truth and test data. The depiction and geometry type of the mines/minefield symbols has been the source of confusion for some time. An attempt to clarify this is in CP 16-018-AR for 2525DChange1, App6D. |
We found something else in our testing. All tags in this style file are preceded with a space (there is a space after each semicolon). This makes it difficult to search for specific tags in Runtime, and is inconsistent with the 2525[x] stylx files:
We should remove these spaces before publishing the file is possible. |
@kerryrobinson - that is a simple fix ( ex. |
@dfoll @ACueva @topowright Please read all comments in this issue, understand the issue, and identify the remaining work that needs to be completed. |
Many of these are fixed in |
I am moving this to done being that it seems like this issue has a perpetual title that really does not allow us to understand what is next or what has been done. If this is not correct @dfoll @BobBooth @csmoore @ACueva I suggest we open other very clear small effort items to address what is needed. |
This issue can be moved to done once this pull request has been reviewed: |
Issues identified during a review of APP6B symbols not drawing in the runtime app, probably due to issues with the STYLX.
Comprehensive test data at:
https://github.com/Esri/military-features-data/tree/dev/military-symbology-styles/test-data/app6b
Used Runtime app created by @csmoore, from his comment on
#276
"...
And the runtime data loader sample app (using 100.2) at: https://esri.box.com/s/dt7ejubj12h566mabxotvk1l0fwh4og9
Using the sample app (and ArcMap data creation steps in sample app readme) - you should be able to create and view sample data in Runtime.
Here is how the comprehensive data set looks in the sample app (about 3% of the symbols showing as unknown at this point):
..."
I test-loaded a more recent version of the comprehensive test data using ArcMap 10.6.0.8321 and the latest released Military Features APP6B styles from here http://solutions.arcgis.com/defense/help/military-features/
and I am seeing a different set of issues, described here:
#292 (comment)
(includes lists of loading errors and PDFs of point, line, and area symbols as they draw in ArcMap).
Brief summary of issue encountered:
The text was updated successfully, but these errors were encountered: