Author Topic: MAPVIEW upgrade  (Read 104471 times)

Offline robin

  • Commander
  • *****
  • Posts: 1075
  • ULTIMATE ROOKIE
    • View Profile
Re: MAPVIEW upgrade
« Reply #75 on: January 19, 2015, 12:22:59 pm »
I recommend pm the guy who made recent version.
Done.

Offline TheBigSot

  • Sergeant
  • **
  • Posts: 44
  • Fight, Die, Repeat.
    • View Profile
Re: MAPVIEW upgrade
« Reply #76 on: January 19, 2015, 03:45:39 pm »
Hello, thanks for the feedback, and posting the error information.

I'll be reviewing the bugs and the feature requested, and I'll let you know when I'm finish.

Offline robin

  • Commander
  • *****
  • Posts: 1075
  • ULTIMATE ROOKIE
    • View Profile
Re: MAPVIEW upgrade
« Reply #77 on: January 19, 2015, 04:09:40 pm »
Hello, thanks for the feedback, and posting the error information.

I'll be reviewing the bugs and the feature requested, and I'll let you know when I'm finish.
Thank you a lot!

Offline TheBigSot

  • Sergeant
  • **
  • Posts: 44
  • Fight, Die, Repeat.
    • View Profile
Re: MAPVIEW upgrade
« Reply #78 on: January 19, 2015, 06:56:19 pm »
...

System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at XCom.XCTile.set_Tiles(XCTile[] value)
   .....

Added the console window
Now it will shown a message in the console similar to 'WARNING: In the MCD file U_EXT02-C, the tile entry 34 have an invalid alternative tile (# 49 of 41 tiles)'

Now checking "SpecialType" so far i found some bugs. and here is the current options:

    public enum SpecialType
    {
        Tile = 0,
        StartPoint,
        IonBeamAccel,
        DestroyObjective,
        MagneticNav,
        AlienCryo,
        AlienClon,
        AlienLearn,
        AlienImplant,
        Unknown9,
        AlienPlastics,
        ExamRoom,
        DeadTile,
        EndPoint,
        MustDestroy
    };

Offline robin

  • Commander
  • *****
  • Posts: 1075
  • ULTIMATE ROOKIE
    • View Profile
Re: MAPVIEW upgrade
« Reply #79 on: January 19, 2015, 07:15:36 pm »
Yeah, default goes up to 14 "MustDestroy". Any higher value and the whole MCDs library (containing the MCD with the unsupported value) is not available anymore in MapView.

Maybe Warboy (or volutar, I don't know) can explain better the whole "Target_Type" -->  ruleset's "specialType" relationship. As far as I know it's just a value, and if the value is equal for both the item is linked to the MCD, so that each instance of the MCD present on the map means an instance of that specific item present in the battlescape (and eventually recoverable at the end of the mission).

(I don't know if this kind of info are useful to you or not. Probably not).

Offline volutar

  • Colonel
  • ****
  • Posts: 355
  • Vanilla digger & Quality assistant
    • View Profile
Re: MAPVIEW upgrade
« Reply #80 on: January 19, 2015, 07:34:29 pm »
I've made addition to rmp format description and something in respective talk area of https://ufopaedia.org. I bet there is quite detailed info. I can't say more now since I'm at vacation in Hungary.

Offline TheBigSot

  • Sergeant
  • **
  • Posts: 44
  • Fight, Die, Repeat.
    • View Profile
Re: MAPVIEW upgrade
« Reply #81 on: January 20, 2015, 02:08:39 am »
Latest Version is available.  ;D :D

Added the console to mapview.
Improved how the errors are shown (now some are warnings that are shown in the console) [Invalid Alternate Tile, invalid Dead Tile]
Changed the way nodes connect in the "RmpView", now there are 3 options "Connect One way" "Connect Two ways" "Dont Connect", instead of autoconnect
Changed the text of "RmpView" to "Waypoint view"
Now the app wont crash or raise errors if the MCD contains a not defined "SpecialType" (higher than 14)
    ** (But they wont have a background, the MCD Viewer will show the number)
Added message when changing the map or closing the app and there are routes RMP not saved
Fixed some errors performing Copy or Cut in the Map Editor


(Next things that might be worked on in another time.)
   Changing the details of a RMP should raise the save message if necessary
   Improve the way floors are added   
   Issues with the palette

Thanks again for all the feedback.
GL HF

Offline robin

  • Commander
  • *****
  • Posts: 1075
  • ULTIMATE ROOKIE
    • View Profile
Re: MAPVIEW upgrade
« Reply #82 on: January 20, 2015, 12:14:34 pm »
Nice!

Offline Hobbes

  • Commander
  • *****
  • Posts: 3502
  • Infiltration subroutine in progress
    • View Profile
Re: MAPVIEW upgrade
« Reply #83 on: January 20, 2015, 03:07:16 pm »
Thank you!

Offline hellrazor

  • Commander
  • *****
  • Posts: 2254
  • Deep Ruleset Digger & Bughunter
    • View Profile
    • Github Account
Re: MAPVIEW upgrade
« Reply #84 on: January 21, 2015, 01:08:37 pm »
I am just testing this with wine on Linux,
is there a copy of the source code anywhere, i would like to get a gtk build of.


Wow that's a total fail on wine. I guess the drawing code is really only for windows....
Not even the menues are working properly...
« Last Edit: January 21, 2015, 01:31:28 pm by hellrazor »

Offline jackstraw2323

  • Colonel
  • ****
  • Posts: 208
    • View Profile
Re: MAPVIEW upgrade
« Reply #85 on: January 24, 2015, 08:40:00 pm »
Using virtualbox and windows XP, having issues saving image of the map out. I get the "OOPS" message but no details.

System.NullReferenceException: Object reference not set to an instance of an object.
   at XCom.Interfaces.Base.IMap_Base.SaveGif(String file)
   at MapView.MainWindow.miSaveImage_Click(Object sender, EventArgs e)
   at System.Windows.Forms.MenuItem.OnClick(EventArgs e)
   at System.Windows.Forms.MenuItem.MenuItemData.Execute()
   at System.Windows.Forms.Command.Invoke()
   at System.Windows.Forms.Command.DispatchID(Int32 id)
   at System.Windows.Forms.Control.WmCommand(Message& m)
   at System.Windows.Forms.Control.WndProc(Message& m)
   at System.Windows.Forms.ScrollableControl.WndProc(Message& m)
   at System.Windows.Forms.ContainerControl.WndProc(Message& m)
   at System.Windows.Forms.Form.WndProc(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
« Last Edit: January 24, 2015, 08:48:11 pm by jackstraw2323 »

Offline TheBigSot

  • Sergeant
  • **
  • Posts: 44
  • Fight, Die, Repeat.
    • View Profile
Re: MAPVIEW upgrade
« Reply #86 on: January 24, 2015, 11:12:09 pm »
Good, Ill review the SaveGif method once my computer stops acting like an Idiot (upgrading then degrading the windows 8.1 installation)

Thanks for the info

Offline TheBigSot

  • Sergeant
  • **
  • Posts: 44
  • Fight, Die, Repeat.
    • View Profile
Re: MAPVIEW upgrade
« Reply #87 on: January 24, 2015, 11:40:58 pm »
Should be fixed now, let me know if this fixed the issue. :)

Offline TheBigSot

  • Sergeant
  • **
  • Posts: 44
  • Fight, Die, Repeat.
    • View Profile
Re: MAPVIEW upgrade
« Reply #88 on: January 24, 2015, 11:44:32 pm »
Wow that's a total fail on wine. I guess the drawing code is really only for windows....
Not even the menues are working properly...

Hi there, I'm not sure how to perform test for your environment, but I may be able to help if you attach some screenshots or videos of the problems.

You can find the code in 'https://github.com/TheBigSot/OpenXCOM.Tools'

Offline hellrazor

  • Commander
  • *****
  • Posts: 2254
  • Deep Ruleset Digger & Bughunter
    • View Profile
    • Github Account
Re: MAPVIEW upgrade
« Reply #89 on: January 25, 2015, 06:48:02 pm »
Since Code is in C#, i maybe will give it a try and compile it manually here on Linux.

I did this before with the stuff from pmporg, but this build was also failing on the drawing and also had the windows errors.
Maybe the code should be revisited. Do you use WPF (System.Windows.Forms and System.Drawing and so) for the menues?