Please help testing upcoming openMSX release!

Page 7/10
1 | 2 | 3 | 4 | 5 | 6 | | 8 | 9 | 10

By ren

Paragon (1155)

ren's picture

15-10-2018, 11:34

Okay, been looking a bit at the reverse. The following things I'm noticing:

  1. manual/commands.html is missing the enable_reverse & disable_reverse commands. Though perhaps you want to encourage users to use toggle_reverse command.
  2. Some command provide feedback after issuing them. That's nice. Others don't. E.g. (suggestions):

    toggle_reversebar -> "Reversebar disabled. Note: reverse itself is still running. To disable reverse issue the /reverse stop/ command."

  3. When calling enable_reversebar when it's already enabled, nothing seems to happen. Some feedback would be helpful there as well: "Reversebar already enabled"
    Or, show & fade-out the bar once more as a vis. indicator (nonetheless, I like console feedback, which I would welcome for other commands as well, like e.g. toggle_cursors (that shows '1', when disabling?)
  4. set auto_enable_reverse gui: mention in the manual this is the default setting (on desktop/PC)
  5. Simply issuing reverse: instead of showing "Missing subcommand", it could display the status (enabled or disabled).
  6. Another response example: reverse stop -> "Reverse disabled"
    Or: reverse stop -> "Reverse already disabled"
  7. I believe reverse can consume quite some MB's. Perhaps nifty if reverse status could show the (est.) memory consumption or something like that?
    And perhaps a way to purge history up to a certain point? E.g.: purge up to 10m ago.

I might come back with some other points..

Coolios!

By mth

Champion (475)

mth's picture

15-10-2018, 19:33

@PatrickR: The new crash log seems to be truncated, it ends somewhere in the core dump and doesn't include the backtrace.
Are you sure you're running the version of openMSX you compiled? By default openMSX is installed in /opt/openMSX; the /usr/bin/openmsx could be the version from Ubuntu instead.

@ren: The reverse mechanism auto-purges the majority of older snapshots: the snapshot density will be lower the further back you go in time.

By PatrickR

Resident (39)

PatrickR's picture

15-10-2018, 21:38

it is, I didnt install openmsx via apt but I do a sudo make install OPENMSX_FLAVOUR=debug -j4 when building.

By Manuel

Ascended (14738)

Manuel's picture

15-10-2018, 21:45

PatrickR, I could reproduce the assertion failure, but not anyore after the fix of wouterv. In the running openMSX, please double check the version you see in the console. That crash should be gone, unless there's something extra special we should do to trigger it now.

By Manuel

Ascended (14738)

Manuel's picture

15-10-2018, 21:53

ren wrote:

[*] manual/commands.html is missing the enable_reverse & disable_reverse commands. Though perhaps you want to encourage users to use toggle_reverse command.

I guess you mean enable_reversebar & disable_reversebar and toggle_reversebar?
I'll see where (and if) I can add them.

Same for these responses.

By PatrickR

Resident (39)

PatrickR's picture

15-10-2018, 22:25

Confirmed fixed! seems that each time I reinstalled catapult (which I did from apt) it also installed openmsx from apt with it ..... I just switched catapult to also pull it from git myself and it does use the correct version.

you're quick with fixing these things guys! Chapeau.

I have a little side question regarding the slotexpander extension: how many slots/extensions does this thing have max?

By Manuel

Ascended (14738)

Manuel's picture

16-10-2018, 00:14

Great you got it confirmed! (Why did you reinstall Catapult all the time?)

A slotexpander works the same as a real slotexpander: max 4 slots. More subslots are not supported in the MSX architecture.

By mth

Champion (475)

mth's picture

16-10-2018, 04:30

It's best to install things you built yourself in /opt or in /usr/local, to avoid conflicts with packages from the distro. But I'm glad it's confirmed fixed.

By PatrickR

Resident (39)

PatrickR's picture

16-10-2018, 09:38

Manuel wrote:

Great you got it confirmed! (Why did you reinstall Catapult all the time?)

Good question I asked myself as well ... can not answer until a lot of coffee has been consumed. Anyways, for future bugs/crashes I'll find it's fixed, I'm working fully with the github versions of both openmsx and catapult now without conflicts.

By ren

Paragon (1155)

ren's picture

16-10-2018, 11:12

mth wrote:

@ren: The reverse mechanism auto-purges the majority of older snapshots: the snapshot density will be lower the further back you go in time.

Ah, yeah.. So I figure it should reach a ceiling at a certain point in regard to memory usage right? (Or not?)

@Manuel: yes, I goofed!
Perhaps a bit nitpicky and rather minor here & there, but hey, small things matter Wink
(edit: uhmm.. Murdoch) (My pen is huge btw Wink)

Page 7/10
1 | 2 | 3 | 4 | 5 | 6 | | 8 | 9 | 10
My MSX profile