-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Blocks removed from projection even when result is not in violation #109
Comments
I'm going to need log and config for this issue. Testing showed it's working as intended. It calculates the projector owner's limit and remove base on the limit it calculates. It's not always based on the grid's limit. It can also be affected by faction and player limit. |
Restart can't affect count. Counts are only saved during sessions and a new count is done on server start. Concealment affecting count is also new to me and the is very unlikely. You are likely experiencing an issue with owned projectors increasing your count. If you own projectors on other ships, if the project a block, it will count as yours instead of the owner of the ship. Projector count is done immediately the block starts projecting. |
I have 1 Projector only am aware of that issue, they get deleted after builds. As per the pic limits are per grid. |
Counting already placed block is as intended. If the block exists, it counts it. The way the projection limit work is, it takes the block count on the grid and subtracts that from the limit placed. Whatever the result is will be the threshold of the projection. In the case of your reactor, only 3 will be able to be projected cause 30 - 27 = 3. The rest will be removed from the projection. |
Counting already placed blocks is fine but the limiter is not waiting long enough for the projector to remove the 27 already built reactors. It thinks I am at 57 reactors hence the high number of removed blocks in pic. Therefore the final 3 reactors are never projected as they are block limited. I have to manually hunt thru the ship and find where they were intended to be without even a projected image to indicate the location. |
5x6 grid of reactors for total of 30. 15 blocks removed from projection before it's removed built blocks resulting in 6 additional reactors being projected. This is preventing multiple people on our server from repairing damaged ships. |
@N1Ran I stepped away from the game for a bit and missed the activity on this issue, but I am getting back into it. I agree with @Privatteer 's observations. If you'll accept a PR I'll put one together or if you're not convinced there is a defect I'll see what I can get together to make it a little more obvious |
I'm seeing on the server I play on, a block that is limited to 10 per grid, where 5 are already placed, having the 5 that are not placed removed from the projection. The result is only 5 blocks are placed on the finished grid, instead of the 10 allowed.
It looks like a fairly simple fix (🤞 ) around https://github.com/N1Ran/BlockLimiter/blob/master/BlockLimiter/Patch/ProjectionPatch.cs#L129 to prefer to remove blocks from the projection that are already placed. Would you accept a PR for this?
The text was updated successfully, but these errors were encountered: