Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Visual plugin: Numbershark GUI changes and fixes #237

Closed
grthor opened this issue Aug 12, 2019 · 10 comments
Closed

Visual plugin: Numbershark GUI changes and fixes #237

grthor opened this issue Aug 12, 2019 · 10 comments
Assignees
Labels
Area:UI+UX-Logic Everything related to SWT / general RCP UI Phase:Backlog Let's get to inbox zero Type:Improvement
Milestone

Comments

@grthor
Copy link
Member

grthor commented Aug 12, 2019

In the .pdf are listed some recommended changes für the numbershark Plugin. The changes affect the GUI and the handlers, which can be seen in the top right corner of the plugin.

Verbesserungsvorschläge.pdf

@simlei
Copy link
Member

simlei commented Oct 14, 2019

@grthor Have some or all of the points seen work since August?
I went ahead and re-assigned you but only because I got mixed up and was not sure whether you had already been assigned; it means nothing more than that.

@simlei simlei added Area:UI+UX-Logic Everything related to SWT / general RCP UI Phase:Backlog Let's get to inbox zero labels Oct 14, 2019
@simlei simlei assigned simlei and grthor and unassigned simlei Oct 14, 2019
@grthor
Copy link
Member Author

grthor commented Oct 17, 2019

@simlei i did not work on any of these points, so i will unassign myself.
But will take a look on it the next days. It did not seem to much work (ca. 8h) and i already know the Plugin from previous work.
When I start working on it i will reassign myself.

@grthor grthor removed their assignment Oct 17, 2019
@grthor
Copy link
Member Author

grthor commented Jan 7, 2020

I have already worked on this plugin and fixed some of the problems.

The next person working on this plugin should check if the problem is already fixed. Maybe its already done 😄

@grthor
Copy link
Member Author

grthor commented Jan 8, 2020

I have already worked on the plugin. So point no. 5 (not working forward and backward buttons) are fixed. All other problems are still open.

@simlei
Copy link
Member

simlei commented Mar 30, 2020

The improvement suggestions to number shark, in short:

  • new game button in the view, far right in the results box
  • a) Text: "und verwirft das bisher aktive Spiel."
  • new game dialogbox: Use more space for infos, e.g.
    • Zuletzt war Ihr Spielfeld Felder groß + Der Standard ist 20. Mit 40 Feldern ist es schon SEHR schwer + Sie können die Feldgröße zwischen 2 und 1024 variieren.
  • 2 as minimum for game size
  • Wenn man 2 Felder hatte und leider die 1 zuerst auswählte, ist die Empfehlung
    (mit n<2 zustarten) falsch. Dieser Fall für n=2 muss extra behandelt werden: Am
    besten nehmen Sie da einfach den Satz "Versuchen Sie es ... Zahlenvorrat."
    weglassen.
  • make dialogs resizable
  • "7) Bitte das Interval der Werte für die Spielfeldgröße n hier noch mit ausgeben im
    Beschreibungstext oben." -> @be4 , what was this point about? see linked pdf in first comment.
  • include difference column in optimal/heuristic strategy viewer (player - shark)

@simlei
Copy link
Member

simlei commented Apr 23, 2020

  • ensure that the dialogs have no critical errors on Windows (e.g. not show at all) @tassadarius

  • Toolbar-half-star-Dialog:

>> 1) Unter Windows kommt folgender Dialog hoch.
>> 2) Ich klickte dann auf den 2. der drei Radiobutton im Dialog (van Nek).
>> Dann kam das ff. Ergebnis-Fenster.
>> Dabei habe ich ff. Probleme:
>> a) Das Fenster war schon initial sehr groß und ließ sich in der Größe
>> (Höhe) nicht ändern.
>> b) Im Dialog zuvor konnte man die "Größe der zu untersuchenden
>> Spielfelder" eingeben.
>>    Ich ließ es auf 39, also die Größen 2-40.
>>    Aber im Ergebnis-Fenster hier werden die Größen 2-250 ausgegeben.
>>    Wenn sich die "Größe der zu untersuchenden Spielfelder" nicht auf alle drei
>>    Radiobuttons bezieht, sollten wir es da ausblenden, wo es keinen Einfluss hat.
  • still 250 instead of 519 optimal paths:
>> c) Warum ging es "nur" bis 250? Sie hatten ja schon von
>> vanNek optimale Werte bis 519 eingebaut, dann können wir 
>> die heuristischen Werte auch für höhere Werte berechnen oder.
  • misc notes:
>> 3) Ich klickte dann auf den Vollstern (optimale Strategien).
>> Da werden die "Größe der zu untersuchenden Spielfelder" nur
>> eingeblendet, wenn man den 2. der beiden Radiobuttons klickt.
>> So sollte es auch sein.

@simlei simlei self-assigned this Apr 23, 2020
@simlei simlei added this to the Final 1.0.0 milestone Apr 23, 2020
@simlei simlei changed the title Numbershark GUI changes and fixes Visual plugin: Numbershark GUI changes and fixes Apr 23, 2020
@simlei
Copy link
Member

simlei commented Apr 28, 2020

  • half-star result dialog size changeable
  • heuristic calculations up to 519
  • heuristic calculations: respect max field size
  • optimal calculations / vanNek: respect max field size
  • calculation dialogs: select first radio button by default

simlei pushed a commit that referenced this issue Apr 28, 2020
simlei pushed a commit that referenced this issue Apr 29, 2020
@simlei simlei closed this as completed Apr 30, 2020
@simlei
Copy link
Member

simlei commented May 4, 2020

1) Die Feldgröße der zu untersuchenden Spielfelder ist zu klein (sollte
mehr als 2 Stellen anzeigen können, mindestens 4).
edited by @grthor
2) Das Icon "Zahlenhai frisst Restzahlen" ist disabled, obwohl 15 hat
keine freien Teiler mehr hat. möglw. nach undo

@simlei simlei reopened this May 4, 2020
@grthor
Copy link
Member Author

grthor commented Jul 28, 2020

Fixed point 1 (Die Feldgröße der zu untersuchenden Spielfelder ist zu klein (sollte mehr als 2 Stellen anzeigen können, mindestens 4).) with 449c1e8.

@simlei
Copy link
Member

simlei commented Oct 29, 2020

Could not reproduce the issue with the disabled button:

2) Das Icon "Zahlenhai frisst Restzahlen" ist disabled, obwohl 15 hat
keine freien Teiler mehr hat. möglw. nach undo

The rest is fixed

@simlei simlei closed this as completed Oct 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:UI+UX-Logic Everything related to SWT / general RCP UI Phase:Backlog Let's get to inbox zero Type:Improvement
Projects
None yet
Development

No branches or pull requests

2 participants