Dota2GSI icon indicating copy to clipboard operation
Dota2GSI copied to clipboard

Patch 7.0 Destroyed Cooldown API

Open gentmat opened this issue 8 years ago • 5 comments

I noticed that attribute and items .Cooldown doesnt work with patch 7.0 . Anyone having this issue ?

gentmat avatar Dec 16 '16 16:12 gentmat

Will look into it in the next few days. Anything else get broken with 7.0?

On Dec 16, 2016, at 10:13, gentmat [email protected] wrote:

I noticed that attribute and items .Cooldown doesnt work with patch 7.0 . Anyone having this issue ?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

antonpup avatar Dec 16 '16 17:12 antonpup

I looked into this. The cooldowns aren't being sent to the library properly. It's always sending 0. It's an issue with Dota 2 and should be reported to Valve if possible.

I also looked into Talents. GSI does send talent information over, however, they are all labeled "attributes" and so due to a limitation in Newtonsoft, it does not allow multiple parameters to have the same name. And so it parses just one talent. I will see if I can find a workaround later for this.

antonpup avatar Dec 18 '16 21:12 antonpup

"It's always sending 0" It was always working prior patch 7 . Maybe they will fix it then i ll update the issue when they do . About the talent tree (yeah that would be nice) good job and hope you the best .

On Mon, Dec 19, 2016 at 1:11 AM, Anton Pupkov [email protected] wrote:

I looked into this. The cooldowns aren't being sent to the library properly. It's always sending 0. It's an issue with Dota 2 and should be reported to Valve if possible.

I also looked into Talents. GSI does send talent information over, however, they are all labeled "attributes" and so due to a limitation in Newtonsoft, it does not allow multiple parameters to have the same name. And so it parses just one talent. I will see if I can find a workaround later for this.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/antonpup/Dota2GSI/issues/11#issuecomment-267846950, or mute the thread https://github.com/notifications/unsubscribe-auth/AE9in4uBJ-TmxeRM8Iznub6KbZ_8vetlks5rJaGcgaJpZM4LPVuC .

gentmat avatar Dec 19 '16 07:12 gentmat

Looks like now the game is sending cooldowns properly.

"ability0": {
                        "name": "meepo_earthbind",
                        "level": 1,
                        "can_cast": false,
                        "passive": false,
                        "ability_active": true,
                        "cooldown": 8,
                        "ultimate": false
                },

Is the cooldown part of the issue resolved then?

a10nik avatar Jan 25 '17 09:01 a10nik

I m not playing from 2 weeks ! hospital :( I ll check it when i come back into game for sure. Ty for the update

On Wed, Jan 25, 2017 at 1:59 PM, Artyom Desyatnikov < [email protected]> wrote:

Looks like now the game is sending cooldowns properly.

"ability0": { "name": "meepo_earthbind", "level": 1, "can_cast": false, "passive": false, "ability_active": true, "cooldown": 8, "ultimate": false },

Is the cooldown part of the issue resolved then?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/antonpup/Dota2GSI/issues/11#issuecomment-275066274, or mute the thread https://github.com/notifications/unsubscribe-auth/AE9in4s5uGee8roI3Z2N3Ge7tAj0UKeuks5rVx0NgaJpZM4LPVuC .

gentmat avatar Jan 25 '17 12:01 gentmat