Skip to content

NF: Piloting mode#6066

Merged
TEParsons merged 34 commits intopsychopy:devfrom
TEParsons:dev-runner-debug
Dec 18, 2023
Merged

NF: Piloting mode#6066
TEParsons merged 34 commits intopsychopy:devfrom
TEParsons:dev-runner-debug

Conversation

@TEParsons
Copy link
Contributor

Allow experiments to run in "debug mode", which overrides some experiment settings to make debugging easier, leaving the settings intact for when they're ready to run for real

TEParsons and others added 29 commits November 29, 2023 17:24
@codecov
Copy link

codecov bot commented Dec 4, 2023

Codecov Report

Merging #6066 (11bce7a) into dev (69ca59a) will increase coverage by 0.24%.
Report is 20 commits behind head on dev.
The diff coverage is 67.81%.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #6066      +/-   ##
==========================================
+ Coverage   46.12%   46.37%   +0.24%     
==========================================
  Files         333      334       +1     
  Lines       61216    61410     +194     
==========================================
+ Hits        28235    28478     +243     
+ Misses      32981    32932      -49     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@TEParsons TEParsons changed the title NF: Debug mode NF: Piloting mode Dec 6, 2023
@peircej
Copy link
Member

peircej commented Dec 18, 2023

@TEParsons given where we are in release cycle, are we going to pull this in and then work on fixing the test suite before the (major) release?

@TEParsons TEParsons merged commit f48f6a3 into psychopy:dev Dec 18, 2023
@TEParsons
Copy link
Contributor Author

Yeah I think that's best, I'd left it open as there's still discussion to be had around the appearance of the button which feels a bit feature-y, but any changes would be minor and easy to implement so I think we could stretch the definition of "bug fix" to include "making the button be a different direction" or "moving such and such label over here" ;)

@TEParsons TEParsons deleted the dev-runner-debug branch January 11, 2024 10:34
@peircej peircej added the 🌟 enhancement Proposed improvement or feature request. label Jul 19, 2024
@Bec1993
Copy link

Bec1993 commented Nov 12, 2024

Hello!
Unfortunately, I have accidentally run a couple of participants in pilot mode rather than experiment mode as I forgot to toggle the "running mode" on in builder. Are there any differences that would impact my ability to use this data, or should it be fine?
Thanks!

@peircej
Copy link
Member

peircej commented Nov 12, 2024

@Bec1993 you should use the forum for support queries. The psychopy issues list is for working out development issues (bugs etc.)
But in short - the key things that currently happen in pilotting mode:

  1. the study runs in a window rather than full-screen
  2. there's a big orange box and a message warning you that you're in pilot mode
  3. there are more debugging messages

1 and 3 might have impacted the timing of your study (I can't tell you if that matters in your study) and 1 and 2 would have affected the experience of the participant (I can't tell you if that matters in your study)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌟 enhancement Proposed improvement or feature request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants