simulationcraft / simc

Simulationcraft engine/GUI
GNU General Public License v3.0
1.37k stars 687 forks source link

SimulationCraft encountered an error! #3959

Closed ben61694 closed 6 years ago

ben61694 commented 7 years ago

SimulationCraft encountered an error!

All options used for simulation:

### Begin GUI options ###
ptr=0
item_db_source=ptrhead/local/bcpapi/wowhead
target_error=0
iterations=10000
default_world_lag=0.1
max_time=300
vary_combat_length=0.2
fight_style=Patchwerk
tmi_window_global=6
target_level+=3
target_race=Humanoid
optimal_raid=0
override.bloodlust=1
override.bleeding=1
override.mortal_wounds=1
threads=8
process_priority=Low
calculate_scale_factors=1
scale_only=str,agi,sta,int,crit,haste,mastery,vers
statistics_level=1
### End GUI options ###
### Begin simulateText ###
# Paste into a SimC profile to compare your current loadout against a proposed loadout

mage="Saniaria - Fire (Equipped)"
talents=3112123
artifact=54:0:0:0:0:748:1:749:4:750:4:751:4:752:4:753:4:754:4:755:4:756:4:757:4:758:1:759:1:760:1:761:1:762:1:763:1:1340:1:1372:1:1533:4:1534:1:1535:1:1536:5:1640:1
race=blood_elf
level=110
role=spell
spec=fire
position=back
head=blackened_mask_of_disgrace,id=146991,bonus_id=3562/1507/3336
neck=belerons_choker_of_misery,id=140899,enchant_id=5439,bonus_id=3518/1502/3528
shoulder=mantle_of_the_arcane_tempest,id=147150,bonus_id=3562/1517/3337
back=cloak_of_everburning_knowledge,id=138365,enchant_id=5436,bonus_id=3518/1502/3528
chest=robes_of_the_arcane_tempest,id=147149,bonus_id=3562/1808/1507/3336,gem_id=130248
tabard=thunder_bluff_tabard,id=45584
wrist=marquee_bindings_of_the_sun_king,id=132406,bonus_id=1811/3570
hands=gloves_of_everburning_knowledge,id=138309,bonus_id=3516/1487/3528
waist=koralons_burning_touch,id=132454,bonus_id=3459/3570
legs=leggings_of_the_arcane_tempest,id=147148,bonus_id=3562/1497/3528
feet=outcast_wanderers_footrags,id=140914,bonus_id=3519/1507/3528
finger1=yathaes_thumb_ring,id=147021,enchant_id=5428,bonus_id=3561/1487/3336
finger2=spellblades_gemmed_signet,id=140895,enchant_id=5427,bonus_id=3516/1808/1487/3528,gem_id=130219
trinket1=whispers_in_the_dark,id=140809,bonus_id=3517/1497/3336
trinket2=erratic_metronome,id=140792,bonus_id=3445/1497/3528
main_hand=felomelorn,id=128820,relic_id=3562:1497:3528/3445:1512:3337/3562:1497:3528,bonus_id=730,gem_id=147091/140812/147088
off_hand=heart_of_the_phoenix,id=133959,relic_id=0/0,gem_id=0/0/0/0
# Relics: Cleansing Ignition Catalyst, Soggy Manascrubber Brush, Smoldering Thumbscrews
### End simulateText ###
desired_targets=1
### Begin overrides ###
# User-specified persistent global and player parameters will be set here.

### End overrides ###
### Begin final options ###
### End final options ###
### END ###
AlexanderKenny commented 7 years ago

Because of some basic validation that is performed on the artifact= string it must be declared AFTER the class/spec lines. For your personal use this output should run if you simply move the spec=fire line above the artifact= line.

However as this appears to be an export from the simbuilder addon you may wish to contact the author to fix the issue on their end.

ben61694 commented 7 years ago

The thing is if I imported it into a previous nightly build it will Sim properly. Is this a new change with the way Simcraft works in the future? If so, I'll contact the simbuilder dev.

navv1234 commented 7 years ago

It changed a few commits ago, i'll see what i can do.

navv1234 commented 7 years ago

Should be fine in the next nightly, maybe.

ben61694 commented 7 years ago

If I install using the latest 8/25 nightly installer, that's what gives me the issue. If I extract simcraft.exe from simc-725-02-win64-550bbb3c1.7z and replace the executable installed by the installer, it works. Hope this helps.

navv1234 commented 7 years ago

It has already been fixed at the cost of input validation.