View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001365 | Spring engine | General | public | 2009-03-29 04:13 | 2009-04-01 00:00 |
| Reporter | Licho | Assigned To | imbaczek | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | resolved | Resolution | fixed | ||
| Product Version | 0.78.2.1 | ||||
| Summary | 0001365: Hardcoded reclaim speed for features | ||||
| Description | Feature.cpp contains "hardcoded" reclaiming speed of corpses: (100 + BP of worker) * 0.02 / max(10, reclaimTime of feature) It should instead be reclaimSpeed of worker.. This equation reduces control mods have over reclaiming. | ||||
| Tags | No tags attached. | ||||
| Checked infolog.txt for Errors | |||||
|
|
do you mean the whole expression should change? currently Feature::AddBuildPower uses what is passed to it in Builder.cpp:303: curReclaim->AddBuildPower(-reclaimSpeed, this)) i do not know any rationale behind the equation you cite, though. |
|
|
This is some leftover - probably hacked so that it works similar to OTA. It should only use part = bp / reclaimTime where bp is sent to function as -reclaimSpeed Current equation is just some random hardcoded crap leftover. |
|
|
bp/reclaimTime it is. raised the default reclaimTime 6 times to compensate. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-03-29 04:13 | Licho | New Issue | |
| 2009-03-31 16:01 | imbaczek | Note Added: 0003360 | |
| 2009-03-31 16:52 | Licho | Note Added: 0003362 | |
| 2009-03-31 23:59 | imbaczek | Note Added: 0003364 | |
| 2009-03-31 23:59 | imbaczek | Status | new => resolved |
| 2009-03-31 23:59 | imbaczek | Resolution | open => fixed |
| 2009-03-31 23:59 | imbaczek | Assigned To | => imbaczek |
