Open scheibo opened 5 years ago
Foul play message fixed in #373
Foul play message fixed in #373
Updated
Some moves in 'moves.ts' are missing the quotes (' ') and are as such unavailable in the showdown selector. This includes moves such as absorb and barrage.
Edit: Looking further into it moves that are still missing the quotes (such as thunderbolt) are still available in the selector. I have no clue what's happening.
Some moves in 'moves.ts' are missing the quotes (' ') and are as such unavailable in the showdown selector. This includes moves such as absorb and barrage.
Edit: Looking further into it moves that are still missing the quotes (such as thunderbolt) are still available in the selector. I have no clue what's happening.
Our live version If the calc is not up to date with the github version. Currently we are reworking moves so it looks all kinds of messed up right now, don’t worry about it too much :)
Our master
branch is not guaranteed to be safe. There are two ways to get to a 'good' version - using a tagged version from when we released the @smogon/calc
package (or better, depending on @smogon/calc
directly via NPM instead of depending on the source), and eventually, by looking at where the gh-pages
branch is at (as we plan to use this in the near future for all deploys going forward)
I feel like I had seen this asked somewhere but I surprisingly cannot find it (so sorry in advance if it is, and it is in that eventuality that I am posting here instead of opening a new issue): Would it be possible, like it is done for Z-Moves, to have the base power of the Max Move displayed in the result text? At the moment it is not, so there is a loss of information on that matter.
Note that this is different from what is already requested but not approved, knowingly that the base name of the move be displayed for a ZMove - which could also be extended for a Max Move I guess). I have no strong feelings about that one, but I hope that the BP for Max Move displayed is approved because it is directly similar to what is already done for Z-Moves.
Would it be possible, like it is done for Z-Moves, to have the base power of the Max Move displayed in the result text? At the moment it is not, so there is a loss of information on that matter.
Yes, this should be displayed. Feel free to open up a PR
knowingly that the base name of the move be displayed for a ZMove
This is still TBD, but if we display it for ZMove we should display it for Max Move
KO chance + exact KOs:
What exactly is the problem here?
KO chance + exact KOs:
What exactly is the problem here?
There's at least something fishy going on with KO chance: https://www.smogon.com/forums/threads/pok%C3%A9mon-showdown-damage-calculator.3593546/post-8460619
As for the quoted calcs in questions: https://www.smogon.com/forums/threads/pok%C3%A9mon-showdown-damage-calculator.3593546/post-7798375 is the original report, I haven't tried to grok exactly what is fishy with it yet which is why its a 'Potential' and not 'Verified' bug
:)
EDIT: see my reply, I think both the recent report and the original report are the same bug?
Ok I was just wondering if I understood the problem correctly. I think the calcs are actually correct, and that is because of the rounding down. If Ttar has 204 HP EVs, its total HP are 392, so the Spikes + SR damage is 2 * floor(0.125*392) = 98 which brings its HP down to 294, so the max roll OHKOes. If it has 200 HP EVs, though, its total HP are 391, so the damage upon entry is 2*floor(0.125*391) = 96, which results in a HP amount of 295, so even max roll can't OHKO. If I'm not mistaken with how entry hazards damage works it seems those results are normal.
Is there a preferred way to run the damage calculation from a python script? I've already got an application going in python and would prefer not to start over in js, but I haven't found any python implementations of this.
I think that the calculator for gen I doesn't always crit for Karate-Chop. the damage calculation seems more similar to 'cut' (also base 50 power) than to slash(which seems to work properly), this is visible in the code itself. I will try to make a pull request with a fix later today, once I figure out how to.
edit: I run the fix locally, run npm tests locally and opened a PR #449
Click to copy text done in #441 and #512
Dark theme done in #525, #540, #543, and #545
Past reported bugs and feature requests from the Smogon thread. I'm unsure if the 'Potential' bugs are still relevant, they should be moved under 'Verified' once confirmed (or removed).
Bugs can still be addressed as they are reported on the thread and is still the correct venue for reporting bugs, but this issue just exists to make track things which may have fallen through the cracks and been forgotten about.
Bugs
Verified
[x] Parental Bond: https://github.com/Zarel/honko-damagecalc/issues/330
[x] Gen 7 modifer order: https://github.com/Zarel/honko-damagecalc/issues/331
[x] Natural Gift: "Not all of the berries for natural gift work, and one isn't even there (Lum Berry), which forces you to make up a move and adjust the BP to do an accurate calc"
[x] Foul play message: "The message displayed when calculating foul play should reference the target's physical attack EV's instead of the user's." https://github.com/Zarel/honko-damagecalc/issues/373
[ ] Screen shift on long pokemon set names: https://www.smogon.com/forums/threads/pok%C3%A9mon-showdown-damage-calculator.3593546/post-8242263
[ ] Weight Calculations: https://www.smogon.com/forums/threads/ultra-sun-ultra-moon-battle-mechanics-research-read-post-2.3620030/page-10#post-8095060
[ ] Support for Past gen moves being used more than once: https://github.com/smogon/damage-calc/commit/2ec87f87ba1e634e6b5bf2f68c6b4d344a37b644 lol i just realized you never added the damage calculation for past gens ><_
Potential
[ ] Recoil damage: "There seems to be a problem with recoil calculation when the damage done by a recoil move exceeds the targets maximum HP, causing it to become something close to 0. Please either fix or implement this on PS."
[x] Stamina does not work with Multi-Hits moves: "I know only mudsdale has it, but as a PU player, it would help me a lot since I always have to calc in my head how low will mudsdale be after two hits, the second hit being physical: first calc the first hit, then put +1 in defense and calc a second hit."
[ ] KO chance broken for repeated use of moves which drop stats
[ ] Imports broken: "I pasted an importable of a 1v1 team on the Import/Export text box, clicked the "Import" button, clicked on "Only show imported sets", and then tried to select a set from the newly imported Pokemon. It showed the Pokemon and the spread all right, but it doesn't seem to display the moves"
[ ] Export stops working after loading custom sets: "The export button stops working after I load a custom set "
[ ] KO chance + exact KOs:
[ ] HP getting mixed up for Blank Sets:
[ ] Weather Ball Z-Move: "When used as a Z-Move, Weather Ball will be turned into Breakneck Blitz, but then Breakneck Blitz will turn into the Z-Move of the type corresponding to the type Weather Ball would be in that weather. The calculator doesn't update the type of Z-Weather Ball during the weathers"
[x] ~~Aerilate in "All vs One": "Noticed that Aerilate doesn't seem to be calculating correctly on "All vs One". Mega Blaziken getting hit with Mega Salamence's Double-Edge only does 73% damage apparently..."~~
[x]
Normalize on moves already Normal type: "Normalize doesn't add the 20% bonus damage on moves that are already normal type"[x]
Metronome (item): "The damage calculator is implementing Metronome as no boost for the first use, then x1.1, then x1.32 (applying both a 1.1 and a 1.2 boost), then x1.716 (all of 1.1, 1.2, and 1.3), then x2.4024 (tacking on a 1.4 as well). Battle source, on the other hand, simply adds a flat 1.2 to the multiplier each time...with the implication that the move wouldn't max out until the 6th use, rather than the 5th?"[x] ~~Fire Spin Physical: "
0- Atk Blaze Mewtwo-Mega-Y Fire Spin vs. 0 HP / 4 Def Kartana: 68-84 (26.2 - 32.4%) -- guaranteed 3HKO after trapping damage
"~~[x]
GSC Struggle damage: "Seems to only be an issue with the type "None" option in the damage calculator. Haven't tested it for other gens"Features
Approved
Requested
252+ SpA Aegislash-Blade Flash Cannon vs. 0 HP / 0 SpD Dark Type Greninja: 211-250 (74 - 87.7%)
"