I think I get what you want here. I agree that not anchoring the map to
the upper-left would be better. It’s not clear to me how to do that yet,
though.
Decision on whether to have memory-mapped files on or off by default.
Cancellation of requests for images scrolled off screen.
(maybe) Fix for a text property drawing bug (which I’m having trouble
replicating).
Is there anything else that we must have/do before releasing a beta? I don’t
want to sit on it too much longer—I was hoping we’d put out this release at
the start of December!
Hey it wouldn’t be a real IT project if it wasnt delayed at least 4 weeks
For the memory-mapped option I’ll do another performance test on an average size module and see what happens. I actually think there are a lot more PBEM games going on than live play so I’m leaning towards turning it off for at least the first full release until we’re comfortable that it is not going to be too big a shock for users.
Personally if I had to choose between 3.0.17 with a 5 second load and 3.1 Beta with a 16 second load time, I would have to go with 3.0.17 purely because I do tons of logs each day; If it affects me then I’m sure it will affect others too.
The last bug fix listed can’t, because it depends on some code which is
in one of the RFEs, which in turn depends on some code in both of the
other RFEs.
My thinking in this particular case is that 3.0 was not an especially
good release, and that putting bug fixes into 3.1.0 will put 3.0 to
rest quicker.
Thanks Joel.
Can we put this in a text file which goes out with each release? It would go into the root of the Vassal folder alongside Vassal.bat and Vassal.sh. A history of changes between releases, we could call it “release notes.txt” or something similar.
OK. I tried a similar speed test with the AVL module. All operations take under 1 second except for opening the module.
With memory mapped files turned on, we’re looking at a 5 second load time for the standard scenario. With memory mapped files turned off it takes around 1 second to load. Either way, zoom speed has been dramatically improved regardless of setting!
I would lean towards having memory mapped files turned off by default at this stage. If users are having memory problems then they can always turn it back on.
Ok. My main reason for developing them was to make it possible to run
large modules with less RAM, which purpose they serve just fine. We
need to be sure to convey to people that it is possible to turn it on,
though.
I think we could. I would be inclined to call it RELEASE-NOTES, though,
as is common practice among FOSS projects—either that or make it
HTML, becuase most Windows users aren’t going to know what it is
otherwise.
With the new zoom feature, I can’t seem to disable the button.
For example, if I want to have a map zoomed out permanently, without the user having the possibility to change it, I could remove the button text, remove the button icons, and the button wouldn’t show up. But with the new control, the button always shows up (the icon gets reset if i unset it).
One suggestion for the .bat and .sh files you have. Can you add %* and $* at the ends of the execution lines respectively. I think this will add any parameters from the command line so you can do this:
I think “$@” is what we want for the shell script, not $* (unless there’s
some reason to prefer $* of which I’m unaware). “$@” prevents filenames
containg spaces (ewwww!) from being broken into multiple arguments.
I don’t know anything about batch files, really. Is %* the right thing?
I really don’t know, but %* worked on XP when I tried it. I have no
idea if it works on other flavors of windows. I do think that %1 %2
%3 %4 etc. would probably work too, although that’s quite ugly.
I set out to fix this, but instead came across several nasty bugs in Zoomer
which I’ve now fixed. There’s still one left which I don’t know how to fix:
If the initial zoom level is the last zoom level on the list and you remove
that zoom level, then the initial zoom level has to change also. E.g., if
initial was 4 and now the list has only three elements, the initial zoom
level can’t remain 4. Handling this case is no problem. What is a problem
is updating the initial zoom text box in the dialog, since I seem not to
have access to that from within Zoomer. Is there a simple way to get that
text box so I can update it?
The way I’m removing the zoom buttons is by clearing the button text
and icon. I don’t see that it’s being reset if I clear them in the
editor and then start a game.