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

3.8.0 Release Plan #130

Closed
48 tasks done
x87 opened this issue Feb 9, 2021 · 51 comments
Closed
48 tasks done

3.8.0 Release Plan #130

x87 opened this issue Feb 9, 2021 · 51 comments
Labels
Milestone

Comments

@x87
Copy link
Collaborator

x87 commented Feb 9, 2021

Download: v3.8.0

Features

Bug Fixes

@x87 x87 added this to the v3.8.0 milestone Feb 9, 2021
@x87 x87 pinned this issue Feb 9, 2021
@x87
Copy link
Collaborator Author

x87 commented May 30, 2021

3.8.0-alpha.0 is available for preview and testing

List of changes
Download

Please report any issues found in this version.

@wmysterio
Copy link

wmysterio commented Jun 1, 2021

3.8.0-alpha.0 is available for preview and testing

My test script compiled without problems, the game didn't crash. I checked out some of the new options. So far I have not seen anything suspicious.


Templates are not added to my code (CTRL+J). When I press Enter or click, the template name is inserted, not all of its content.

@x87
Copy link
Collaborator Author

x87 commented Jun 1, 2021

Templates are not added to my code (CTRL+J). When I press Enter or click, the template name is inserted, not all of its content.

thank you for checking 👍

@VitalRus95
Copy link

All in all, the update is great! I liked the new features, especially theme editor and opcode hints. The existing colour scheme settings have transferred to the new version nice, the only exception being the editor background, I had to set it again but that's a minor inconvenience. That's how SB looks with 3.8.0 alpha:
изображение

I also tested compiling a script from command line, works fine now even without {$NOSOURCE}, thank you, @x87!

As for small bugs, there are some:

  • just like @wmysterio, I can't use templates, only their names are being inserted into the code;
  • I have several tabs with different GTA games, and when, just having started SB, I switched from VC script tab to SA script tab, the status of language service remains “Scanning” until I manually choose the corresponding game again, even though it is already chosen in the bottom right corner and the script compiles without errors;
  • again tabs issues: if you have several tabs, then close the editor, then open a script or source from Windows Explorer, the tabs will all be replaced with the script you opened, which is a bit annoying;
  • when hovering over bottom panel, the hints like “Double click to open SB library in your browser” flicker. I also find the font to be quite small there, maybe it would be cool to have pop-up hints of the regular text size (which can be set by the user) right above/below the selected opcode when hovering your mouse over it. Yet it's not a big problem and I don't know if this idea can be implemented.

Other than that, the new version is a big step forward, I like it! By the way, will there be an offline version of the opcode library just in case? Good luck in further development and thanks for making SB better!

@x87
Copy link
Collaborator Author

x87 commented Jun 1, 2021

@VitalRus95 thank you for your support! I will look into the issues you've reported.

will there be an offline version of the opcode library just in case

in the current implementation the opcode library is completely offline. There are gta3.json/vc.json/sa.json files in the respective mode directories (controlled by the new <library> property in modes.xml). I plan to make an automatic and on-demand update of those files from within the IDE. Also you can download newer versions from https://github.com/sannybuilder/library and update your local installation.

https://github.com/sannybuilder/library/blob/master/gta3/gta3.json
https://github.com/sannybuilder/library/blob/master/vc/vc.json
https://github.com/sannybuilder/library/blob/master/sa/sa.json

@x87
Copy link
Collaborator Author

x87 commented Jun 6, 2021

@x87 x87 added the roadmap label Jun 6, 2021
@wmysterio
Copy link

Published 3.8.0-alpha.1

I have not noticed any problems. A later I will test the CLI options.

@VitalRus95
Copy link

Templates now work perfectly fine, thanks! I haven't tested opening a script or source from Windows Explorer, but the rest of the issues I listed above are still present. Also, there's a little problem I encountered as early as in 3.7.0 version of SB:
изображение

However, when I click OK, thus closing the error windows, SB continues working without any errors. Strange. But at least there's nothing that breaks the coding, just some inconvenience. I'm glad the programme is getting better and receives new nice features!

@x87
Copy link
Collaborator Author

x87 commented Jun 8, 2021

Also, there's a little problem I encountered as early as in 3.7.0 version of SB

is this reproducible in 3.8.0? Can you share the sanny.log ?

@VitalRus95
Copy link

Yes, it keeps happening in 3.8.0. I would willingly share the .log, but I don't have one near sanny.exe, unfortunately. All I have is these three errors at the same time:
изображение
When I close them, SB loads and works like nothing happened.

@x87
Copy link
Collaborator Author

x87 commented Jun 9, 2021

@VitalRus95
Copy link

VitalRus95 commented Jun 9, 2021

@x87, thanks for the new alpha!

Two problems have been fixed:

  • startup error messages;
  • “Scanning” status of the language service staying on screen forever.

Two little things remain:

  • pop-up hints of the bottom panel still flicker;
  • tabs get cleared and replaced by a file you open from Windows Explorer.

By the way, I noticed that templates are reset to standard ones after installing new versions, which is also an inconvenience, since I, for one, have my own CLEO template and slightly modified standard templates of conditions and loops. It would be cool to have them saved until a serious change is needed.

A thought crossed my mind about templates: maybe if a user creates their own CLEO template, it could be used by SB in “Create new CLEO script” menu option. But these are just suggestions, not a high priority thing to consider.

I like how fast you react to the feedback, good luck with further development! I'll also try to spread the word about the new features that will appear in 3.8.0 to make people more interested in updating. :-)

@x87
Copy link
Collaborator Author

x87 commented Jun 9, 2021

A thought crossed my mind about templates: maybe if a user creates their own CLEO template, it could be used by SB in “Create new CLEO script” menu option.

this is exactly what should happen. New CLEO script menu is a combo of two actions: Open a new document and Apply a template with the name "cleo".

@VitalRus95
Copy link

VitalRus95 commented Jun 9, 2021

Hmm, it should but in my case it doesn't for some reason. I have the following template:

={$CLEO} // Script by Vital (Vitaly Ulyanov)
=script_name '|'
=
=const
= plr = $PLAYER_ACTOR
= pgr = $PLAYER_GROUP
= mis = $ONMISSION
=end // const
=
=while true
=    
=end // while
=
=terminate_this_custom_script

But whenever I click “New CLEO script” in SB, I only get

{$CLEO .cs}
0000:

0A93: terminate_this_custom_script

P. S. It's strange that now in Ctrl+J menu I have two CLEO templates, and both of them insert the same standard code…
изображение

@x87
Copy link
Collaborator Author

x87 commented Jun 9, 2021

I have the following template:

Which file is that? Templates are of two kinds: shared and exclusive. https://docs.sannybuilder.com/edit-modes/code-templates#types-of-templates

You might have added your custom template to data\templates.txt and it has a lower priority than the template with the same name cleo defined in data\sa\templates.txt

@VitalRus95
Copy link

I intended the template to be shared, as it is suitable for all GTA games I write scripts for. But yes, you are right, moving the same code to sa\templates.txt helped, now in SA mode it inserts my template. So, I think now I'll have to copy it to each game's templates.txt to make it work for GTA 3, VC, and SA at once. Thanks for the prompt!

@x87
Copy link
Collaborator Author

x87 commented Jun 9, 2021

@VitalRus95 also you can delete all exclusive versions of "cleo" template and keep only the shared one if that works for you.

@x87
Copy link
Collaborator Author

x87 commented Jun 9, 2021

I haven't found a good way to deal with user modifications yet. I know that it could be painful to migrate from one version to another if you heavily modified data files. I can open a new discussion for this topic, so we evaluate what options do we have.

@x87
Copy link
Collaborator Author

x87 commented Jun 13, 2021

Published 3.8.0-alpha.3
https://github.com/sannybuilder/dev/releases/tag/v3.8.0-alpha.3

@VitalRus95
Copy link

I have a small suggestion, don't know if that's possible to implement, but it would be nice. I noticed that you can't call class commands from a constant. For instance, if you declare 3@ as an Actor and assign a constant swt to the value 3@, then you can use

yet you won't be able use

swt.Create(…)

The latter, I think, may make constants even more helpful in SB (again, if possible and reasonable).

@x87
Copy link
Collaborator Author

x87 commented Jun 19, 2021

Published 3.8.0-alpha.4
https://github.com/sannybuilder/dev/releases/tag/v3.8.0-alpha.4

@VitalRus95
Copy link

So far 3.8.4 is good, yet I found two issues:

  • I created a new script (thus adding a new tab) and after restarting the programme this tab was gone; I reopen the script from the “File” menu, but its tab disappears again after restarting SB;
  • last opened documents section of the “File” menu doesn't work and shows nothing.

@VitalRus95
Copy link

@x87, sure, here's my settings.ini.

As for the file's path, it only has English letters.
settings.txt

@x87
Copy link
Collaborator Author

x87 commented Jun 23, 2021

@VitalRus95 as I mentioned in #146 (comment) it appears that your settings.ini does not have newer configuration entries. Can you investigate, why it does not have a theme name and renamed options from #93 ? was looking into older file.

Do you have multiple sanny installations? Did you install sanny in a UAC protected directory such as Program Files? Is this settings.ini read-only?

@VitalRus95
Copy link

VitalRus95 commented Jun 23, 2021

Hm, as for the theme, the file contains the line Editor::Theme=Dark (by Vital). And is not marked as read-only.

My settings.ini is located in C:\Users\ulian\AppData\Local\VirtualStore\Program Files (x86)\Sanny Builder 3\data.
Sanny Builder's folder is C:\Program Files (x86)\Sanny Builder 3.

I have one SB installation, yet I usually do not uninstall previous versions before installing new ones. I did do a complete reinstallation when I noticed the issue, but it didn't help.

Does it all mean that it's buggy only because of Program Files (x86)? Why on earth does the OS prompt to install programmes there if they keep breaking this way…

@x87
Copy link
Collaborator Author

x87 commented Jun 23, 2021

@VitalRus95 check #43 for details

@VitalRus95
Copy link

@x87, thanks for the link! I set SB shortcut to run as admin, and it seems to have resolved the issue! Also, now I found settings.ini in SB's data folder. Hope running as admin would prevent further conflicts with Windows. Never thought the system is that quirky.

@x87
Copy link
Collaborator Author

x87 commented Jun 23, 2021

You can install SB anywhere outside of Program Files (x86) to avoid the need in admin mode.

@XMDS
Copy link

XMDS commented Jun 27, 2021

@x87 3.8.0-alpha.4 When I choose a translation language, after closing SB, opening SB again will restore to the default English language

@x87
Copy link
Collaborator Author

x87 commented Jun 27, 2021

@x87 3.8.0-alpha.4 When I choose a translation language, after closing SB, opening SB again will restore to the default English language

Do you run Sanny as admin? See discussion above

@x87
Copy link
Collaborator Author

x87 commented Jul 12, 2021

@x87 3.8.0-alpha.4 When I choose a translation language, after closing SB, opening SB again will restore to the default English language

there is an issue in alpha.4 with renamed options. When settings.ini contains both the old and the new name, the former takes a priority. A workaround is to delete your settings.ini and re-run Sanny.

@x87
Copy link
Collaborator Author

x87 commented Jul 24, 2021

Published v3.8.0-beta.0

https://github.com/sannybuilder/dev/releases/tag/v3.8.0-beta.0

@VitalRus95
Copy link

It seems creating a new CLEO script with a custom template works fine, now it's more intuitive, thank you very much for this! But I noticed this version doesn't seem to recognise neither tabs nor several (in my case four) spaces, substituting them with just one space. Very strange. In templates (both custom and predefined) there are four spaces in some templates, they don't work any more.

изображение

@x87
Copy link
Collaborator Author

x87 commented Jul 24, 2021

this version doesn't seem to recognise neither tabs nor several (in my case four) spaces, substituting them with just one space.

thanks for the report, confirmed.

@x87
Copy link
Collaborator Author

x87 commented Jul 25, 2021

@wmysterio
Copy link

Published v3.8.0-beta.1

Perhaps this is the way it should be, but I noticed that the path to the game is not saved after installation and I have to specify it again in the program settings (the "CLEO" button is highlighted in red, although I also installed it and indicated the path to the game).

@x87
Copy link
Collaborator Author

x87 commented Jul 30, 2021

the path to the game is not saved after installation

can you confirm if there is an exe file inside the directory that you chose? (gta3.exe, gta-vc.exe or gta[_-]sa.exe)

the "CLEO" button is highlighted in red, although I also installed it and indicated the path to the game

you installed it separately from Sanny Builder? Is there cleo.asi file in the game directory?

@wmysterio
Copy link

can you confirm if there is an exe file inside the directory that you chose? (gta3.exe, gta-vc.exe or gta[_-]sa.exe)

I renamed the specified file ("gta_sa.exe"), since the game starts with this name only once.

you installed it separately from Sanny Builder? Is there cleo.asi file in the game directory?

I installed CLEO only with the Sunny Builder installer. The file "cleo.asi" existed. Shouldn't it be overwritten?

@x87
Copy link
Collaborator Author

x87 commented Jul 30, 2021

sorry, I'm a bit confused about the order of the steps you made. I see multiple issues here.

Installing CLEO during Sanny Builder installation does not work. Even if you specify the correct directory and select the option install CLEO it does nothing, this code is broken. So after Sanny Builder installation there should be no changes made to the game directory.

I have fixed this and in the next version Sanny Builder's installer should properly copy CLEO files to the game directory.

Selecting game directory during install should work and Sanny Builder should keep this path in the settings. Can you make some testing on your end and confirm that it does not happen to you? Try to make a fresh install, select the game directory and then launch Sanny Builder and see if it has the path in the options?

@wmysterio
Copy link

@x87 I did tests on "3.8.0-beta.1" and they showed that the path to the game is not saved if the folder does not contain "gta_sa.exe". In all other cases, the path disappears from the settings. If this behavior is expected, then there will be no more questions. I just wondered why the "gta_sa.exe" plays a strong role when only the directory is needed.

@x87
Copy link
Collaborator Author

x87 commented Jul 31, 2021

@wmysterio this is because when you select a directory Sanny Builder should somehow identify the edit mode it belongs to. It can be GTA III, VC or SA. Exe file is a good marker. If, let's say, the directory has gta3.exe Sanny will remember this directory for GTA III edit mode.

Possibly this option can be rethought, and instead of asking for one directory the installer could offer 3 different fields, one for each game. Then there won't be a need for an executable in the directory.

@wmysterio
Copy link

@x87 Hmm, now I installed SB without CLEO and with "gta_sa.exe". Result: edit mode "gta sa 1.0", but the path to the game was not saved o_0. Some strange behavior.

@x87
Copy link
Collaborator Author

x87 commented Jul 31, 2021

@wmysterio can you share settings.ini from this installation?

@wmysterio
Copy link

@x87 sure, settings.ini.

@x87
Copy link
Collaborator Author

x87 commented Jul 31, 2021

@wmysterio I don't see any game related settings. I looks like the installer was unable to identify the game for that directory. I tried to reproduce the same: created a new folder and added file gta_sa.exe in it. Then I run 3.8.0-beta.1 installer, selected that folder, skipped CLEO library installation and did not run Sanny Builder after installation. My settings ini has the following lines:

[SA]
GamePath=C:\gta-sa
[Main]
EditMode=2
[Lang]
Lang=1033

Can you do one more experiment. Repeat all steps again, but don't launch Sanny Builder automatically after installation. Go to its folder and share what's in the settings.ini. There should be only a few lines.

Result: edit mode "gta sa 1.0"

this is a default choice if no other edit mode was set.

@wmysterio
Copy link

@x87 File after these steps, but without autorun:

[SA]
GamePath=D:\Programm\GTA_SA
[Main]
EditMode=2
[Lang]
Lang=1049

Hmm, tried another identical test, but with autostart and "settings.ini" it became identical to the lines above o_0. Very strange. I wonder if other users have similar incidents or is it just me? ..

@x87
Copy link
Collaborator Author

x87 commented Jul 31, 2021

@wmysterio so, first of all, there is another issue that I found looking into our initial settings.ini. Installer generates the file with EditMode value set to an integer, whereas Sanny Builder expects this option to be a string: https://docs.sannybuilder.com/cli#mode The correct values are 'gta3', 'vc', 'sa'. I have fixed this.

As for your observations, I can't say for sure what is going on, but maybe you have multiple Sanny Builder installations and you are installing SB in one directory, but checking settings.ini in the other directory?

@wmysterio
Copy link

wmysterio commented Jul 31, 2021

@x87

As for your observations, I can't say for sure what is going on, but maybe you have multiple Sanny Builder installations and you are installing SB in one directory, but checking settings.ini in the other directory?

Before installing, I always delete the "Sanny Builder 3" folder and I will install all new versions from scratch. I also have a "Sanny Builder 3.5.2" folder, but she's unlikely to be involved in this. I'll do some more tests for now. I don’t know what’s going on yet.

@x87 x87 closed this as completed Aug 4, 2021
@wmysterio
Copy link

Rapported by @VitalRus95 . The error is also reproducible for me.:

again tabs issues: if you have several tabs, then close the editor, then open a script or source from Windows Explorer, the tabs will all be replaced with the script you opened, which is a bit annoying;

The behavior of the tabs remains the same.

@x87
Copy link
Collaborator Author

x87 commented Aug 17, 2021

@wmysterio we have a ticket #151

@x87 x87 unpinned this issue Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants