simulationcraft / simc

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

GUI Progress Bar not Updating for Reforge Plots #1014

Closed navv1234 closed 9 years ago

navv1234 commented 9 years ago

Originally reported on Google Code with ID 1015

What steps will reproduce the problem?

1. Just started simulation and waited for the finish.

What is the expected output? What do you see instead?

I expected the result output.

What version of the product are you using? On what operating system?

sim 4.3-1 on Win7

Please provide any additional information below.

It just doesn't show the result, I see "plot mastery" step (the last parameter to plot
in settings) at 100%. And CPU usage is about 100%. 

Reported by cherkonz on 2011-12-04 11:59:54

navv1234 commented 9 years ago
how long did you wait? Can you include the profile and sim settings you used? What was
the progress bar doing?  GUI or command line?

Reported by wowharly on 2011-12-04 12:21:20

navv1234 commented 9 years ago
I waited for a couple of hours after the progression bar has been at 100%. It was GUI
interface.

profile i tried to simulate:

#!./simc 

priest=Дрейкс
origin="http://eu.battle.net/wow/en/character/%D0%B0%D0%B7%D1%83%D1%80%D0%B5%D0%B3%D0%BE%D1%81/%D0%94%D1%80%D0%B5%D0%B9%D0%BA%D1%81/advanced"
level=85
race=troll
position=back
role=spell
use_pre_potion=1
professions=alchemy=525/tailoring=525
talents=http://www.wowhead.com/talent#priest-033211000000000000000000000000000000000000322032210201222100231
glyphs=shadow_word_pain/mind_flay/shadow_word_death/fade/spirit_tap/inner_fire/shadowfiend/shadow_protection/levitate
actions=flask,type=draconic_mind
actions+=/food,type=seafood_magnifique_feast
actions+=/fortitude
actions+=/inner_fire
actions+=/shadow_form
actions+=/vampiric_embrace
actions+=/snapshot_stats
actions+=/volcanic_potion,if=!in_combat
actions+=/volcanic_potion,if=buff.bloodlust.react|target.time_to_die<=40
actions+=/mind_blast
actions+=/berserking
actions+=/shadow_word_pain,if=(!ticking|dot.shadow_word_pain.remains<gcd+0.5)&miss_react
actions+=/devouring_plague,if=(!ticking|dot.devouring_plague.remains<gcd+1.0)&miss_react
actions+=/stop_moving,health_percentage<=25,if=cooldown.shadow_word_death.remains>=0.2|dot.vampiric_touch.remains<cast_time+2.5
actions+=/vampiric_touch,if=(!ticking|dot.vampiric_touch.remains<cast_time+2.5)&miss_react
actions+=/archangel,if=buff.dark_evangelism.stack>=5&dot.vampiric_touch.remains>5&dot.devouring_plague.remains>5
actions+=/start_moving,health_percentage<=25,if=cooldown.shadow_word_death.remains<=0.1
actions+=/shadow_word_death,health_percentage<=25
actions+=/shadow_fiend
actions+=/shadow_word_death,if=mana_pct<10
actions+=/mind_flay
actions+=/shadow_word_death,moving=1
actions+=/devouring_plague,moving=1,if=mana_pct>10
actions+=/dispersion
head=flickering_cowl_of_the_feverflare,heroic=1,type=cloth,ilevel=391,quality=epic,stats=1211armor_689sta_400int_276haste_276mastery,reforge=mastery_spi,gems=burning_shadowspirit_20int_20haste_30int,enchant=60int_35crit,suffix=-262
neck=firemind_pendant,ilevel=378,quality=epic,stats=227int_341sta_144spi_156mastery,reforge=spi_haste
shoulders=shoulderwraps_of_the_cleansing_flame,heroic=1,type=cloth,ilevel=391,quality=epic,stats=1118armor_322int_513sta_227crit_209haste,reforge=crit_spi,gems=20int_20haste_10int,enchant=50int_25haste
shirt=formal_white_shirt,ilevel=1
chest=vestment_of_the_cleansing_flame,heroic=1,type=cloth,ilevel=391,quality=epic,stats=1491armor_420int_689sta_254spi_310haste,gems=20int_20haste_20int_20spi_20int,enchant=20all
waist=majordomos_chain_of_office,type=cloth,ilevel=384,quality=epic,stats=808armor_300int_480sta_182mastery_214haste,reforge=mastery_spi,gems=40int_40int_10int
legs=leggings_of_the_cleansing_flame,heroic=1,type=cloth,ilevel=391,quality=epic,stats=1304armor_420int_689sta_301crit_271mastery,reforge=crit_haste,gems=40int_40int_20int,enchant=95int_55spi
feet=coalwalker_sandals,heroic=1,type=cloth,ilevel=391,quality=epic,stats=1025armor_322int_513sta_235crit_197haste,reforge=crit_spi,gems=40int,enchant=35mastery
wrists=wristwraps_of_arrogant_doom,heroic=1,type=cloth,ilevel=391,quality=epic,stats=652armor_256int_384sta_154crit_181haste,enchant=65haste
hands=gloves_of_the_cleansing_flame,heroic=1,type=cloth,ilevel=391,quality=epic,stats=932armor_322int_513sta_180spi_244haste,gems=20int_20spi_10int,enchant=50haste
finger1=crystalline_brimstone_ring,ilevel=378,quality=epic,stats=227int_341sta_162hit_133mastery,reforge=hit_spi
finger2=infernal_signet_of_the_avengers,ilevel=391,quality=epic,stats=236int_384sta_173haste_146hit,reforge=hit_spi,gems=40int_10int
trinket1=necromantic_focus,heroic=1,ilevel=391,quality=epic,stats=433int,equip=onspelltickdamage_44mastery_10stack_10dur
trinket2=variable_pulse_lightning_capacitor,ilevel=384,quality=epic,stats=406int,equip=onspellcrit_2926.3nature_15%_10stack_2.5cd_chance
back=wings_of_flame,heroic=1,ilevel=391,quality=epic,stats=745armor_256int_384sta_162crit_176mastery,reforge=crit_haste,enchant=lightweave_embroidery
main_hand=dragonwrath_tarecgosas_rest,ilevel=397,quality=legendary,stats=426int_730sta_314hit_271haste_2786sp,gems=40int_40int_40int_30int,enchant=power_torrent,weapon=staff_3.30speed_869min_1304max
ranged=trail_of_embers,heroic=1,ilevel=391,quality=epic,stats=145int_217sta_89hit_101haste,reforge=hit_spi
tabard=renowned_guild_tabard,ilevel=1,quality=epic
# Gear Summary
# gear_strength=20
# gear_agility=20
# gear_stamina=6887
# gear_intellect=5997
# gear_spirit=1160
# gear_spell_power=2786
# gear_hit_rating=554
# gear_crit_rating=746
# gear_haste_rating=2617
# gear_mastery_rating=1047
# gear_armor=9286
# meta_gem=burning_shadowspirit
# tier12_2pc_caster=1
# tier12_4pc_caster=1
# back=wings_of_flame,heroic=1,enchant=lightweave_embroidery
# main_hand=dragonwrath_tarecgosas_rest,weapon=staff_3.30speed_869min_1304max,enchant=power_torrent

Reported by cherkonz on 2011-12-04 12:57:50

navv1234 commented 9 years ago
And here is the settings from sims_gui.simc

item_db_source=local/bcpapi/wowhead/mmoc/armory/ptrhead
iterations=25000
default_world_lag=0.1
max_time=500
vary_combat_length=0.2
fight_style=Patchwerk
target_level=88
target_race=humanoid
default_skill=0.9
threads=2
optimal_raid=0
override.dark_intent=0
override.focus_magic=0
override.strength_of_earth=1
override.communion=1
override.devotion_aura=1
override.blessing_of_might=1
override.bloodlust=1
override.leader_of_the_pack=1
override.arcane_brilliance=1
override.mana_spring_totem=1
override.windfury_totem=1
override.replenishment=1
override.wrath_of_air=1
override.arcane_brilliance=1
override.demonic_pact=1
override.fortitude=1
override.blessing_of_kings=1
override.sunder_armor=1
override.mangle=1
override.bleeding=1
override.demoralizing_roar=1
override.blood_frenzy_physical=1
override.poisoned=1
override.hunters_mark=1
override.thunder_clap=1
override.critical_mass=1
override.earth_and_moon=1
calculate_scale_factors=1
scale_only=none,int,spi,sp,hit,crit,haste,mastery
dps_plot_stat=none,int,spi,sp,hit,crit,haste,mastery
reforge_plot_stat=none,spi,hit,mastery
reforge_plot_output_file=reforge_plot.csv
#!./simc 

Reported by cherkonz on 2011-12-04 13:02:17

navv1234 commented 9 years ago
You're telling the simulator to do scale factor calculation for all stats, DPS plots
for all stats, *and* a generate tri-stat reforge plot data. This will take much longer
than "a couple of hours" on almost any computer, so you'll just have to wait longer.

I guess the real issue here may be that we need make sure all of these things are included
in the progress bar in the GUI?

Reported by jon@valvatne.com on 2011-12-04 13:43:52

navv1234 commented 9 years ago
Yes, it could really help to see every major operation at the progress bar. In my case,
reforge plot status should be seen there after dps plots are over.

Thanks a lot!

Reported by cherkonz on 2011-12-04 13:51:09

navv1234 commented 9 years ago
We may also want to add a disclaimer that says if you simulate all this at once, be
ready to wait a long time and that your chart will have so much information on it at
once, that you won't be able to read it.

Reported by swbusche on 2011-12-04 18:29:02

navv1234 commented 9 years ago
Fixed in r10658

Reported by swbusche on 2011-12-14 23:42:43