[2.1.15] Lab condition "set Research" not handled by cut/paste

Bugs which we just recently fixed in our development version and will reach you in the next release.
Franky.
Burner Inserter
Burner Inserter
Posts: 9
Joined: Fri Dec 06, 2024 3:29 pm
Contact:

[2.1.15] Lab condition "set Research" not handled by cut/paste

Post by Franky. »

(A1 action 1, o1 observation one, E1 expectation one)
What did you do?
A1. have roboport coverage for construction bots
A2. Have a lab ("lab1") with a red wire, e.g. attach to constant combinator.
A3. Check set research in circuit panel.
A4. Add two conditions for research
(A5. Enter Technology view)
(A6. actuate wire input, not necessary)
A7. press cut CTRL+X and remove lab.
A8. press paste CTRL+V and place lab again in wire reach
A9. wait for lab to be built, open lab, check research conditions and circuit panel
A10. Enter Technology view
A11. actuate wire input to change research
(additional quirk / recovery)
A12. fufil both conditions by constant combinator
A13. delete and re-add one of the research conditions.
A14. change wire input to fulfil only the research untouched in A12

What happened?
o4. The research changes as desired.
o5. "Research is being set by a lab (i)" is displayed as expected
o9. conditions and circuit panel restored
o10. "Research is being set by a lab (i)" is NOT displayed
o11. Research does not change
(additional quirk / recovery)
o13. the re-added research is set as researched technology
o14. the untouched research is NOT set as researched technology

What did you expect to happen instead?
e10. "Research is being set by a lab (i)" is displayed again
e11. Research changes as set by lab research conditions.

Does it happen always, once, or sometimes?
Happens always.
Checked with two saves, in one of them at least 3 times.
Might also trigger with deconstruction by player character.

Impact:
This might also affect blueprints with Lab conditions
Rseding91
Factorio Staff
Factorio Staff
Posts: 17434
Joined: Wed Jun 11, 2014 5:23 am
Contact:

Re: [2.1.15] Lab condition "set Research" not handled by cut/paste

Post by Rseding91 »

Thanks for the report. This was a visual issue with the GUI and is now fixed for the next release.
If you want to get ahold of me I'm almost always on Discord.
Franky.
Burner Inserter
Burner Inserter
Posts: 9
Joined: Fri Dec 06, 2024 3:29 pm
Contact:

Re: [2.1.15] Lab condition "set Research" not handled by cut/paste

Post by Franky. »

"visual issue"?
Step 11, "research not set according to lab" is more serious than just a display error. There is no recovery by un-checking and checking "set research" again.
Anyway, I will try to reproduce with the next release.
User avatar
boskid
Factorio Staff
Factorio Staff
Posts: 4826
Joined: Thu Dec 14, 2017 6:56 pm
Contact:

Re: [2.1.15] Lab condition "set Research" not handled by cut/paste

Post by boskid »

Reproduction steps have varying level of accuracy so by "actuate wire" i will consider it to be just changing signals on a wire such that research conditions change between being fulfilled or not. I was not able to reproduce any problems other than "visual" ones: once research conditions changed between fulfilled or not, when lab wanted to select a different technology than currently researched it had no problem with changing it and marking itself as one setting research. This being "visual" is because the only issue that can be observed here is that lab does not report itself as being one that set research until technology is changed. If you can provide a reproduction where i could observe lab failing to set a recipe i can take a look, but i would prefer if it involved a save file at some later stage of reproduction one or two steps before it fails because with so many steps before its hard to make sure if i am missing something or if you skipped some steps.

-- edit:

Due to lack of details around step A4, i assumed those research conditions have to be different such that it is possible to make them unfulfilled independently. With this when at step A11 i sent a signal that makes first research condition unfulfilled and second was still fulfilled, the technology was correctly switched. Due to lack of details i cannot rule out cases that you actually have the same research condition used twice and at step A11 you accidentaly made both conditions unfulfilled and expected lab to stop all research which is not how lab's "set research" is currently designed to operate.
Franky.
Burner Inserter
Burner Inserter
Posts: 9
Joined: Fri Dec 06, 2024 3:29 pm
Contact:

Re: [2.1.15] Lab condition "set Research" not handled by cut/paste

Post by Franky. »

When I tried to create a minimal example, it turned out you are right, just a visual issue. You CAN skip the rest of this post, in retrospect I am trying to make my error yours.

(Anyway) But let me turn this into more visual issue: I built a complex science switcher logic. Debugging "set research" is hard. On train/platform schedules and decider combinators, fulfilled conditions are marked green, i.e. The gray background turns green.
Request: In a science lab, the first fulfilled (and research available) box should turn green. The box that you can drag. Subsequent conditions that are fulfilled (but have no effect due to not being first) might be yellow.
With in interface improvement like this, I would have seen my error clearly.
(edit: the research icon color already kind of shows it. If it is either green or red, it wont have an effect.)
There is no error indication if lab tries to set a research that cannot be researched, then researched technology stays the same. This made me think the feature is broken.

Save 1 (actually multiplayer) Aquilo not visited yet. I set up to research Quantum Computing (First Aquilo) when Cryo science pack is available. Non-Repeatable.
This gets high priority. Priority is done by a Selector combinator! So only one condition in the lab is true.

copy blueprint, continue solo: Quantum computing already researched. My switcher tries to research something that already is researched, having no effect. It feels like a bug so I continue on multiplayer: guess what, aquilo reached now, Quantum Computing researched in my absence. <- I am not 100% sure what happened there.
Post Reply

Return to “Resolved for the next release”