TripleA Logo TripleA Forum
    • TripleA Website
    • Categories
    • Recent
    • Popular
    • Users
    • Groups
    • Tags
    • Register
    • Login

    Missing Image Trouble

    Scheduled Pinned Locked Moved Map Making
    28 Posts 7 Posters 8.8k Views 6 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • redrumR Offline
      redrum Admin @Cernel
      last edited by

      @Cernel Like this?

      d488e4a0-2f9f-44fd-8ce7-d5932ae8ee61-image.png

      TripleA Developer with a Passion for AI: https://forums.triplea-game.org/topic/105/ai-development-discussion-and-feedback

      C 1 Reply Last reply Reply Quote 2
      • C Offline
        Cernel Moderators @redrum
        last edited by

        @redrum Yeah, that seems ok. I would write it this way, though, also adding the zone, if this thing was meant to make the game somewhat playable anyways (before, the game just crashed, with any missing images):

        MISSING UNIT IMAGE (this unit is not displayed): American_infantry (owner: Russians; zone: Rostov)

        Mostly the question here is how it works. If it still takes a placement spot, but you see nothing, like if you would have a blank image, I would say "not visible". If it is not drawn at all on the board, I would say "not displayed".

        Looking at the current warning, I guess it works also for missing images that are not units? I mean also something else than "UnitType"? If so, I guess those will need to be reworded, if we want to be specific here.

        This would be the current format; not sure why it is worded this way:

        MISSING IMAGE (this unit or image will be invisible): UnitType{name=American_infantry}

        It makes me think there is a series of other instances of this warning that are not "UnitType". Is this so?

        redrumR 1 Reply Last reply Reply Quote 2
        • redrumR Offline
          redrum Admin @Cernel
          last edited by

          @Cernel Ah, good idea on location:
          c8093fc1-4026-46cf-81b3-c9931244b4cb-image.png

          TripleA Developer with a Passion for AI: https://forums.triplea-game.org/topic/105/ai-development-discussion-and-feedback

          C 1 Reply Last reply Reply Quote 1
          • C Offline
            Cernel Moderators @redrum
            last edited by

            @redrum If I recall correctly (not sure), correct English doesn't officially allow missing the subject; so it should be "it won't be displayed". If the rest is in that format, I would put the specific elements within quotation marks.

            Meaning:

            MISSING UNIT IMAGE (it won't be displayed): "American_infantry" owned by "Russians" in "Timguska".

            Not sure if the whole thing should have a full stop at the end or just no terminal punctuation at all.

            1 Reply Last reply Reply Quote 0
            • redrumR Offline
              redrum Admin
              last edited by

              Good enough. Here is the PR: https://github.com/triplea-game/triplea/pull/4922

              TripleA Developer with a Passion for AI: https://forums.triplea-game.org/topic/105/ai-development-discussion-and-feedback

              1 Reply Last reply Reply Quote 3
              • C Offline
                Cernel Moderators
                last edited by

                Should this topic be moved to feature requests, at this point?

                1 Reply Last reply Reply Quote 0
                • RoiEXR Offline
                  RoiEX Admin
                  last edited by

                  @Cernel nah, I think we should be fine

                  1 Reply Last reply Reply Quote 0
                  • B Online
                    beelee
                    last edited by

                    Well I can take consolation in the fact that my stupidity actually had a beneficial side effect this time :face_with_tears_of_joy: :face_with_tears_of_joy: :face_with_tears_of_joy:

                    1 Reply Last reply Reply Quote 1

                    Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                    Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                    With your input, this post could be even better 💗

                    Register Login
                    • 1
                    • 2
                    • 2 / 2
                    • First post
                      Last post
                    Copyright © 2016-2018 TripleA-Devs | Powered by NodeBB Forums