SC2 Replay Auto-Renamer - Page 8
Forum Index > SC2 General |
CharlieMurphy
United States22895 Posts
| ||
VonLego
United States519 Posts
| ||
warcode
Norway46 Posts
tested vs computer That will most likely crash, actually I am sure it will crash every single time. The replay data for matches against the computer is so different it wont parse nicely unless I redo a lot of the code, and I don't have the time for that. The program was made to auto-rename ladder replays against players, not for games vs the computer or non-regular custom games. | ||
Kinslayer
United States129 Posts
On September 30 2010 23:04 warcode wrote: Its pretty much a release-ready beta, which means its perfectly safe to try. Its just to make sure I got the proper settings for each area when gathering data from the replays. If no problems are found, this will be the new version I release. That will most likely crash, actually I am sure it will crash every single time. The replay data for matches against the computer is so different it wont parse nicely unless I redo a lot of the code, and I don't have the time for that. The program was made to auto-rename ladder replays against players, not for games vs the computer or non-regular custom games. It's trivial to fix at the moment as I said on your website. Wrap the call to the parse and rename method in a try..catch and it won't crash. It will silently fail to rename the replay and be fine. It's very annoying now to play on yabot game and not have any ladder games past that point get saved due to the crash. I fixed it on my machine but won't share it, it's your code, not mine. Thank you for making this. I was going to do it, but you pretty much built what I needed. If you want, open source it and others can build on it | ||
warcode
Norway46 Posts
As for making it open source, that is something I have been considering. The only problem with that is that its very easy to edit it to do something malicious, and if people start posting their versions, people might blame me for any trouble that causes. Its easy to say "please only download from links in my posts or on my website", but how many regular users actually listen to that? | ||
Kinslayer
United States129 Posts
On October 01 2010 01:34 warcode wrote: Oh, yeah I can do that. But I won't be fixing the actual parsing. As for making it open source, that is something I have been considering. The only problem with that is that its very easy to edit it to do something malicious, and if people start posting their versions, people might blame me for any trouble that causes. Its easy to say "please only download from links in my posts or on my website", but how many regular users actually listen to that? I don't think you need to fix the actual parsing. It's perfectly acceptable to say this is for ladder games only. Just having it crash for non ladder games is not nice I see your point about the open source thing. Specially that I don't think you'll have time to maintain it and make sure nothing bad goes in. Maybe open source or release the parsing portion as a library? I don't think there's a working parser library for .net at this point in time. There was one, but doesn't work at all anymore. Regardless, I personally love your version. Does one thing and does it well (minus the crash). Good job sir. | ||
MasTer9006
United States14 Posts
| ||
VonLego
United States519 Posts
Thanks again, I'll get back to this. | ||
VonLego
United States519 Posts
| ||
Perscienter
957 Posts
| ||
warcode
Norway46 Posts
Changelog: * The program will now silently fail to rename games vs AI opponents. THIS IS HOW IT IS SUPPOSED TO WORK. * Added some more languages to the internal translation of race names. If a race appears as an X, and it is NOT random, its because I don't know the correct translation for that race. * Some bugfixes Download link: SC2RARu10 | ||
zincsterio
119 Posts
It seems to be failing to detect races correctly on the latest 1.1.1 replays. I'm getting stuff like "2010.10.10.0918_XvX_alexander.vs.zincsterio.__[f44f2200]". Also, is there any way you can give the dynamic formatter more freedom? I would like to do: "yyyy-mm-dd_Player1(R)_vs_Player2(R)_Map_[Id]". But right now I don't think this can be done with the current dynamic renaming. Thanks! | ||
griff
Netherlands26 Posts
| ||
MatronStarcraft
35 Posts
| ||
warcode
Norway46 Posts
| ||
zincsterio
119 Posts
On October 11 2010 08:06 warcode wrote: The batch-renamer is so far behind that I have pretty much abandoned it. I could try to make SC2RAR a bit more batch-friendly once I get the time. Yeah, if you could incorporate batch renaming into SC2RAR that would be awesome, since I would like to rename some older replays. Any comments on whether you'd be able to make dynamic naming more flexible? Say "yyyy-mm-dd_Player1(R)_vs_Player2(R)_Map_[Id]" format? All the data is there, but just more control by user in defining pattern? Thanks. | ||
Patriot.dlk
Sweden5462 Posts
May you enter valhalla on the final day | ||
warcode
Norway46 Posts
On October 12 2010 16:15 zincsterio wrote: Any comments on whether you'd be able to make dynamic naming more flexible? Say "yyyy-mm-dd_Player1(R)_vs_Player2(R)_Map_[Id]" format? All the data is there, but just more control by user in defining pattern? Thanks. |Date|_|Normal.|_|on|_|Map|_[|ID|] is the closest you can get at this time. I was going to make a better dynamic system, but I never got around to it. And these days I have to really look to find any free time to spend on this. But when I do get some time it IS on my todo list. | ||
Jonoman92
United States9101 Posts
| ||
CharlieMurphy
United States22895 Posts
Also, when I set the delay between renaming to anything it just resets itself to 0 even if I save and close. Could the problem be related at all to the fact that I put custom names for maps into the txt file? I abbreviated most of the map names like this + Show Spoiler + Blistering Sands|BS Metalopolis|Metal Lost Temple|LT Kulas Ravine|KR Jungle Basin|JB Shakuras Plateau|SP Delta Quadrant|DQ Xel'Naga Caverns|XC Steppes of War|SoW Desert Oasis|DO Arid Wastes|AW Discord IV| D4 Tempest|Temp Tarsonis Assault|TA Twilight Fortress|TF War Zone|WZ Scorched Haven|SH Monolyth Ridge|MR High Orbit|HO Arakan Citadel|AC Colony 426|C426 Dig Site|DS Frontier|Front Monsoon|Mons Quicksand|QS The Bio Lab|TBL Typhon|Typh Ulaan Deeps|UD Extinction|Extinct High Ground|HG Lava Flow|LF Megaton|Mega Outpost|OP Sand Canyon|SC Toxic Slums|TS Tectonic Rift|TR Abyss|Aby I also use a dynamic setup like this: |Matchup|.|Players|.|on|.|Map| And I'm using windows XP sp2 i believe. | ||
| ||