jsparkes/innovation

Port of Innovation card game to .NET

innovation's Issues

RFE - Add mouseover for Achievements 1-9

When you mouse-over the non numerical Achievements a card is displayed with text on it describing it.

This doesn't happen on the 1-9 Achievements. I don't always remember the values :-(

Also mousing over the icons would be nice most are obvious but Factories and blue one...

Update to 2017 Deluxe version add revised card text and code to support

Most of these will require code updates? But for sure Innovation.txt

https://boardgamegeek.com/thread/1817162/innovation-deluxe-change-list

Base set
Age 1. Oars demand adds “If you do, draw a 1 and repeat this dogma effect!"

Age 2. Fermenting dogma is now “Draw a 2 for every color on your board with one or more leaves.”

Age 3. Feudalism dogma adds “If you do, unsplay that color of your cards!"

Age 5. Statistics demand is now “I demand you transfer all the highest cards in your score pile to your hand!”

Age 5. Measurement dogma is now “You may reveal and return a card…”

Age 5. Societies demand is now “I demand you transfer a card with a bulb higher than my top card of the same color from your board to my board! If you do, draw a 5!”

Age 6. First Industrialization dogma is now “Draw and tuck a 6 for every color on your board with one or more factories.”

Age 7. Combustion demand is now “I demand you transfer one card from your score pile to my score pile for every four crowns on my board!” and adds a dogma: “Return your bottom red card.”

Age 9. Fission dogma adds “Draw a 10.”

BUG - when testing.txt file exists causes app crash

Not sure why. No vbstuff it is happening in the array functions putting arr definition at top seemed to fix push3 but seems to be failing in Unshift3 now.

Tested original code and it works fine and shipped with a commented out testing.txt file.

   at Innovation.Main_Renamed.launch_game(Object& players) in C:\Users\frankk\Projects\innovation\innovation\main.vb:line 797
   at Innovation.Main_Renamed.cmd2Players_Click(Object eventSender, EventArgs eventArgs) in C:\Users\frankk\Projects\innovation\innovation\main.vb:line 684
   at System.Windows.Forms.Control.OnClick(EventArgs e)

RFE - Add debugging code & some Static variables

Currently debugging messages are spread throughout the code and has to be added handrolically ..

Revised Build VS2022 version Agriculture card played first is not displayed properly on tableau

There is a white square with error text that isn't readable. It might be related to color id warning/error messages?

The card display in top left have the text with matching card displayed correctly.

Add BGG info

Add links to original BGG thread into the README.md.
Add Jeff Till's email address.

Add dumber AI? And/or understand/document opacity 0-100% (5 levels)? Document the “AI”’s algorithm..?

Current level of AI difficulty to me is hard/extreme and setting opacity doesn’t seem to have any noticeable effect? If “opacity” doesn’t do anything maybe it should be removed from UI?

The code seems to have a variable called “cheat”?

Document the “AI”’s algorithm..?

Yes, multiple items here for now this is the generic AI issue?

VS 2022 build results in mouse over events not fully working

Playing with the app and I noticed the achievement area mouse over events are working (except the missing rfe ones).

The overall mouse clicks seems to be working properly in that part of the form. The VB6NET converter wizard has this warning:

Event behavior has changed - Some events that were handled by container controls may be blocked by Line and Shape controls

ms-help://MS.VSCC.v90/dv_commoner/local/redirect.htm?keyword="8C49BE24-2706-4243-8090-5E48F6703313"

Innovation AI cheating, or breaking rules?

Seems a game we played scoring cards 3 of them 4 4 4 came out in the second round (if my recall is correct).

In order to further diagnose maybe we need to review code for debugging options and maybe add save log option? (although log file should be c'n'p ' able)... saving to has been implemented to log,txt

Bug - AI opacity 0-100% does work but how

Seems this was implemented in designer only and changes do not happen at run time.

Disabling gui elements for now but leaving in the project.

VS2022 build is resulting in non functional binary in built package seems security blocked?

The install seems to work. But trying to run the binary results in nothing running and no security warning.

Installing and running as administrator does not fix.

Jeff, can you try the version on git and rebuilding on your system? I don't have a security certificate. It worked originally but seems to have stopped after I got a false positive security warning about a virus from the published build .....

Here is the same problem with no solution... maybe because I'm using community edition? https://docs.microsoft.com/en-us/answers/questions/124017/trying-to-install-my-application-but-getting-a-sec.html

Nope: https://superuser.com/questions/1252575/unable-to-install-clickonce-application-due-to-security-settings-windows-10

Innovation.txt file issues .... missing/corrupt file causes app to crash

If the file Innovation.txt is corrupted or missing the app threw out of bounds error (see below for messages)

Added check (in initialize_card_data() ) for missing data file. But this doesn't address corrupted data (which should never happen).

Is there a better way to embed data into code? could it just be a bunch of DIM statements?

----------------------snip snip-------------------------------------------------------------

Errors:

Unhandled exception has occurred in your application. If you click Continue, the application will ignore this error and attempt to continue. If you click Quit, the application will close immediately.

Index was outside the bounds of the array.

---

See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.IndexOutOfRangeException: Index was outside the bounds of the array.
at Innovation.Main_Renamed.launch_game(Object& players)

Show cards pop up is broken (when you have more then 1 card in tableau pile)

... example below was trigger by clicking on blue "3 - Alchemy"

See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.Exception: Control array element 1 doesn't exist.
at Microsoft.VisualBasic.ErrObject.Raise(Int32 Number, Object Source, Object Description, Object HelpFile, Object HelpContext)
at Microsoft.VisualBasic.Compatibility.VB6.VB6Errors.RaiseError(Int32 nError, String strMessage)
at Microsoft.VisualBasic.Compatibility.VB6.BaseControlArray.BaseGetItem(Int16 Index)
at Microsoft.VisualBasic.Compatibility.VB6.LabelArray.get_Item(Int16 Index)
at Innovation.showArray.load_pictures(Int16 player, Int16 Index, String kind) in C:\Users\frankk\Projects\innovation\innovation\showArray.vb:line 67
at Innovation.Main_Renamed.lblOppDetail_Click(Object eventSender, EventArgs eventArgs) in C:\Users\frankk\Projects\innovation\innovation\main.vb:line 8098
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.Label.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)

Loaded Assemblies ---DELETED

RFE - Output players best option in text window

Since the AI is checking the player’s moves as well? As part of its move evaluation can we output the player’s best move to text window?

later either add button (Recommend Move) or option to always do that..

RFE - Add about box?

Here’s a great example of a VB about box written by one of the “stack overflow founders”:

https://blog.codinghorror.com/about-the-about-box/

RFE - add undo action/button

Add the ability to undo an action.

It will need a new UI element I’m think before the commit button

RFE - Fully Implement Save/Load game

Seems the code is saving and loading stuff to save,txt still not sure why..?

We now have File menu item and it could support File-->Load and FIle -->Save