View Issue Details
ID | Project | Category | Date Submitted | Last Update | |
---|---|---|---|---|---|
0023962 | AI War 2 | GUI | Oct 12, 2020 10:29 pm | Dec 15, 2021 10:41 pm | |
Reporter | Asteroid | Assigned To | Chris_McElligottPark | ||
Status | resolved | Resolution | fixed | ||
Product Version | 2.612 Divide By Zero Hotfix | ||||
Summary | 0023962: R-view shows ARS as having 5x multiplier vs Orbital Mass Driver | ||||
Description | First anomaly in the title. Second anomaly, the ARS is also bolded which is supposed to mean it's in the local system - but there's no ARS in Mesa. Third anomaly, the ARS is also listed in the list of things you can capture. Basically, something really wonky is going on. I've also see this happen with Unarmed Guard Post being counted as on your team and having a 1x multiplier again some enemy unit. | ||||
Tags | No tags attached. | ||||
related to | 0023930 | new | R view thinks Astro Train Station is on my team | |
related to | 0022593 | resolved | Daniexpert | Eyebot bonus attack damage is not scaling? |
related to | 0021739 | new | Strong Against is always "None" in C-click view for capturable fleets | |
related to | 0025813 | resolved | Chris_McElligottPark | Damage multiplier tooltip quite broken |
|
|
|
I'm not sure what is happening. This was originally your code, and I'm having trouble following what is going on. I upgraded LINQ in a few ways and it's possible that there's something up with that, I'm not sure. Looking at the code, I don't see anything that jumps out at me as wrong with it, and really there's not a lot of LINQ being used here. |
|
I think it's probably a bug in my code that has been there from the start (sorry!). I remember Nuc Temeron reporting similar funny business a long while ago, and other people were complaining about some multipliers being inexact, too (https://steamcommunity.com/app/573410/discussions/0/1637550282076996257/ - though I did try to take mark into account). Since people are taking interest again in the feature (tip of the day added recently about R view, discussion on Discord) I thought I'd report it. The R-view has always been more of a proof of concept that I hoped you'd pick up and implement with better knowledge of the engine, TBH. A proper implementation I had in mind before leaving development was pre-computing at load time a big table that lists the bonuses of every unit of every mark vs every unit of every mark, instead of computing this stuff every frame when the user is holding R. This way all that'd be left for the display would be determining what is in what faction and what's on the local planet, and a simple looking in the table. |
|
Here's the exact post by Nuc Temeron where he found out that the R-view sometimes thinks you're the AI: https://steamcommunity.com/app/573410/discussions/0/1637550282076996257/?ctp=2#c1637550282083856529 Including the Tractor Array listing a 2x bonus *facepalm*. It's funny in a way, but I wish this feature was working for players to use it, it's sorely needed. So the most buggy part of the code is likely the one that checks which faction a unit is in, which if I remember correctly involves first checking the planet. There's always the possibility that Badger changed something with factions after I wrote the initial code... there was a lot of work done on being able to have factions on your team around that time, I think. |
|
R-view should be all fixed up now. |
Date Modified | Username | Field | Change |
---|---|---|---|
Oct 12, 2020 10:29 pm | Asteroid | New Issue | |
Oct 12, 2020 10:29 pm | Asteroid | File Added: 2020-10-12 22_22_45-AIWar2.png | |
Oct 12, 2020 10:29 pm | Asteroid | File Added: RViewDamageBonusForARS.save | |
Oct 16, 2020 7:23 pm | Chris_McElligottPark | Note Added: 0059255 | |
Oct 16, 2020 7:48 pm | Chris_McElligottPark | Relationship added | related to 0023930 |
Oct 16, 2020 9:59 pm | Asteroid | Note Added: 0059263 | |
Oct 16, 2020 10:11 pm | Asteroid | Note Added: 0059266 | |
Oct 16, 2020 10:29 pm | Asteroid | Relationship added | related to 0022593 |
Oct 17, 2020 6:22 pm | Asteroid | Relationship added | related to 0021739 |
Nov 13, 2021 9:42 am | Daniexpert | Relationship added | related to 0025813 |
Dec 15, 2021 10:41 pm | Chris_McElligottPark | Assigned To | => Chris_McElligottPark |
Dec 15, 2021 10:41 pm | Chris_McElligottPark | Status | new => resolved |
Dec 15, 2021 10:41 pm | Chris_McElligottPark | Resolution | open => fixed |
Dec 15, 2021 10:41 pm | Chris_McElligottPark | Note Added: 0063474 |