"Path tracing tile" starts at 0 instead of 1 #52574

New Issue

Danijel Boskovic · 2017-08-29T05:10:48+02:00

Danijel Boskovic commented

2017-08-29 05:10:48 +02:00

Small issue, when rendering in official RC2 (7b397cd), where it says path tracing tile it seems it says which ones are done, not which one it's working on. I think it makes more sense that it says which one is rendering right now, like it always did.

Small issue, when rendering in official RC2 (7b397cd), where it says path tracing tile it seems it says which ones are done, not which one it's working on. I think it makes more sense that it says which one is rendering right now, like it always did.

Danijel Boskovic commented

2017-08-29 05:10:48 +02:00

Changed status to: 'Open'

Danijel Boskovic commented

2017-08-29 05:10:48 +02:00

Added subscriber: @DBTessel

Carlo Andreacchio commented

2017-08-29 06:17:38 +02:00

Added subscriber: @candreacchio

Carlo Andreacchio commented

2017-08-29 06:17:38 +02:00

Duplicate of #51435 -- I agree it should be changed but it has been ruled invalid :(

Sebastian Koenig commented

2017-08-29 08:28:47 +02:00

Added subscriber: @sebastian_k

Sebastian Koenig commented

2017-08-29 08:28:47 +02:00

I replied in the other report as well.
And yeah, I also think it should be changed. It might not be a code bug, but a grammar bug it definitely is. For instance it says it is working on tile 11 from 12, when it is actually working on tile 12 from 12. A bit confusing for the user.

I replied in the other report as well. And yeah, I also think it should be changed. It might not be a code bug, but a grammar bug it definitely is. For instance it says it is working on tile 11 from 12, when it is actually working on tile 12 from 12. A bit confusing for the user.

Bastien Montagne commented

2017-08-30 17:37:12 +02:00

Added subscribers: @LukasStockner, @mont29

Lukas Stockner was assigned by Bastien Montagne

2017-08-30 17:37:12 +02:00

Bastien Montagne commented

2017-08-30 17:37:12 +02:00

@LukasStockner looks like you'll need to tweak the message at least here. But agree with others here, we developers are used to have 10 items with indices from 0 to 9, but this is absolutely counter-intuitive for normal people! First tile should be number 1, not number 0.

@LukasStockner looks like you'll need to tweak the message at least here. But agree with others here, we developers are used to have 10 items with indices from 0 to 9, but this is **absolutely** counter-intuitive for normal people! First tile should be number 1, not number 0.

Lukas Stockner commented

2017-08-31 19:10:24 +02:00

I agree that the message is misleading, but a simple +1 is not really the correct answer. The number that's currently displayed is the number of finished tiles, so it actually makes sense that it starts as 0. Just doing a +1 is wrong since there will be multiple tiles active at once.

So, my suggestion would be to change "Path Tracing Tile 0/50" to "Path Tracing, 0/50 tiles finished, 8 active". Sounds good for everybody?

I agree that the message is misleading, but a simple +1 is not really the correct answer. The number that's currently displayed is the number of *finished tiles*, so it actually makes sense that it starts as 0. Just doing a +1 is wrong since there will be multiple tiles active at once. So, my suggestion would be to change "Path Tracing Tile 0/50" to "Path Tracing, 0/50 tiles finished, 8 active". Sounds good for everybody?

Brecht Van Lommel commented

2017-08-31 23:22:37 +02:00

Added subscriber: @brecht