search

Aireil / FFLogsViewer

Dalamud plugin to view FF Logs ranking percentiles in-game
47 stars 16 forks source link

Show ndps as default for parses from stormblood #39

Closed frukbachers closed 5 months ago

frukbachers commented 5 months ago

Less of an issue, more of a suggestion.

I personally have my layout set up to view historical ultimate parses (among others):

grafik

I'd like to keep rDPS as default metric to view, but I have to swith each time I want to check the ultimate parses from stormblood.

Would it be possible to set a default metric inside the layout that overrides the top selection? grafik

Aireil commented 5 months ago

Unfortunately, the way it's currently setup would make this a little bit annoying to implement (AKA rework how the data is fetched) and I don't feel like it would be worth the hassle :/.

Aireil commented 5 months ago

I could maybe add a way to overwrite just dps for specific zones, but the UI wouldn't match the correct loaded metric using the default (if set to rDPS). Which I guess wouldn't really matter for those zones as rDPS is never valid 🤔 .

frukbachers commented 5 months ago

Alright, thanks for letting me know your thoughts. Closing issue then, and have a great day.

Aireil commented 5 months ago

Now available in the latest update, just force aDPS (it's aDPS, not nDPS) on those encounters and it should work as you expected to (hopefully 👁️).

frukbachers commented 5 months ago

I just now noticed the change and wanted to say many thanks. You're right, I actually meant aDPS instead of nDPS and since it only applies to Parses from Stormblood (at least on my end), one could just make a note like this in the encounter alias.

grafik

Either way, thank you very much for your consideration.