-
Notifications
You must be signed in to change notification settings - Fork 56
Add remaining Missing icons to the icon exports (ex. FULL_FRAME icons) #72
Comments
The names need to be unique, but wouldn't the numeric version of the ID also need to be too? Movement and Maneuver : Infantry,10121100 Needs to become... Unknown : Movement and Maneuver : Infantry,110121100 Where 1, 3, 4, and 6 at the beginning of the number represent the four standard identity variations on full frame icons (the actual standard identity code values from the SIDC - which equate to _0, _1, _2, and _3 in the svg/emf file names). For domain/value lookup purposes with our symbol renderers, would we need to add one row for each standard identity, even though we only have four full frame icons for each symbol and a couple of the entries below will point to the same svg/emf files, as a result? Pending : Movement and Maneuver : Infantry,010121100 ----> Uses 10121100_0.svg Or can we just have just the four symbol entries and expect our military symbology renderers to have some logic in them that can "map" seven different standard identities on "input" and reduce them to four possible standard identity symbols on "output"? |
For now, might be best to just get started by getting:
So let's focus on that for the short term. Also, we likely will not need the IDs to be numeric or have them included in the domain exports (so the ids/name can match the current file name convention with _0, _1, etc). Also, the affiliation/identity might make more sense at the end: |
Adding "Land unit icons – special entity subtypes" to the missing icons list that should be added. HEADQUARTERS ELEMENT - 10xxxx95_X |
I couldn't find boundary lines either : Command and Control Lines : Boundary : Lateral 110101 But I did find this: https://github.com/Esri/joint-military-symbology-xml/blob/master/instance/jmsml_D_Control_Measure.xml#L21 Not sure what the issue was though. |
@csmoore ( @joebayles ) : Lateral, forward, and rear boundaries are just examples of the use of boundaries. They are not separate different control measures, requiring their own unique SIDCs. The pictures in the standard are there to illustrate how boundaries are used, and that when they are drawn relative to the enemy or each other, they assume an appellation like, "Forward", "Lateral" or "Rear". UPDATE : I'm now investigating with the SSMC why Mike included SIDCs for Forward, Lateral, and Rear in Appendix A but not in the control measure appendix. Another inconsistency that missed review by members of the SSMC. The Army voting representative for SSMC tells me there is no need for those three extra boundaries. |
Added FULL_FRAME icons to export in #74 |
RE: boundaries - someone else could possibly make the opposite case looking at the standard so if we are not implementing can we just capture that explanation/information in some fashion in the the Tags? For now, we can just include the Change Proposal number with this issue just so it can be tracked. |
Added _0, _1, _2, and _3 full frame image/domain values. Changed order of name and tag elements in Amplifiers and HQTFFD. Updated svg files (renamed Frame svgs). Deleted redundant All doman value sample. Updated batch file to produce new All_Domains_Values files. Miscellaneous changes to support the above.
RE: boundaries - The removal of the SIDC from Appendix H (Control Measures) for the 'Forward', 'Lateral', and 'Rear' boundaries was approved right before the release of MIL-STD-2525D and should not be listed in Appendix A of the document. As a note: Appendix A is just a quick listing of all the symbols/codes within the other appendices and is secondary in terms of the authoritative SIDC. The codes listed within the symbol set appendices take precedence whenever there is a disconnect between the two. |
Thanks I think there was just some additional confusion for this particular one since those exact names/graphics also appeared in Appendix H, so it wasn't clear if the Codes had just been left off the Appendix H version. But explanation sounds good & if possible if we can get a CP or other change tracking number to refer anyone who might question - that will help also. |
Support for these is now in via PR/Merge #80 HEADQUARTERS ELEMENT - 10xxxx95_X |
Looks nearly complete, just need to add the final quick/very brief write-up to "Explain the rules of how we go from the name/tags to assembly a complete symbol for these" |
Will be updating OneNote today. |
@abouffard - a question regarding that (_0, _1, _2, _3) nomenclature/encoding, there seems to be 2 different encodings: 1: "FULL_FRAME" encoding: 2: Everywhere else: Would it be possible to make these consistent (using the 2nd encoding) so there isn't an opportunity to misinterpret these and everything is consistent? |
Nevermind - just realized that is already burned into all the file names (and we want to follow that convention) - perhaps something to consider for next time... Just make sure you explain that extra mapping step in the verbal explanation of how to derive these. |
That's really a question for DISA, as the svg file naming convention is theirs. @mepler : can we please have the _0, _1, _2,and _3 svg file names renamed to _1, _3, _4, and _6.respectively? That would make the file names also consistent. Alternatively, we could leave the files names as they are and just export the IDs using _1, _3, _4, and _6. Do we care if the file names use one naming scheme and our internal unique IDs use another? ** Update ** I just saw @csmoore's "nevermind" and I'd already written this. Perhaps for Change 1 we can rename the files, and then make the IDs consistent with _1, _3, _4, _6. |
Nevermind, let's keep them as they are for now & you all can evaluate that later - like I said I forgot about the file names (and was just looking at the IDs trying to figure out the breakout) - it is better that the ids/filenames match the existing convention - sorry for the sidetrack |
I am going to close this - although I was still awaiting an explanation of when/how to apply the _1, _2, _3, _4 logic/substitution But now that the "FULL_FRAME" tag is there, I imagine it is something like this: If I am starting backwards from the data store only (un-usual/symbol builder/editor case):
However this does introduce a complication when going from the SIDC (the usual, normal, messaging case) - because this complication means we can't necessarily use these Unique IDs as lookup keys from the SIDC, because the SIDC doesn't tell you when these "Full_Frame" cases apply. As a concrete example: Suppose I had Unknown : Land Unit : Command and Control : Signal;( (10, 01, 10, 0, 0, 00), (111000, 00, 00) ) Usually the SIDC is all I need to look up this symbol using the key 10111000 - but now I instead have to :
This is OK & no need to respond(unless you want let me know I got something wrong), just trying to capture this for my own benefit. |
Your logic for starting backwards from the data store only can also be... using "Command and Control : Signal : Unknown" for example:
Your second case logic is correct. The SIDC does not encode whether an icon is full frame or not. |
A known issue/already discussed, just adding an issue for tracking/testing.
Currently the only version of the icons exported for FULL_FRAME (center icons that touch the frame) icons are the "_2.svg" (Neutral) version ( Example ).
That was an optimization but now that we are entering the stabilization/refinement stage, it is probably time to address this issue so all of the icons are exported for these.
Some possible derived tasks/things to make sure are included/addressed:
The text was updated successfully, but these errors were encountered: