Skip to content

Fix: correct unpacking and array conversion in basic experimentalist tutorial#701

Merged
musslick merged 2 commits intoAutoResearch:mainfrom
acesmeci:fix-basic-experimentalist-tutorial
Aug 11, 2025
Merged

Fix: correct unpacking and array conversion in basic experimentalist tutorial#701
musslick merged 2 commits intoAutoResearch:mainfrom
acesmeci:fix-basic-experimentalist-tutorial

Conversation

@acesmeci
Copy link
Contributor

Description

Fixed incorrect unpacking and array conversion in Tutorial Ib: Experimentalists.

Previously, running the notebook as-is produced [0] for new experimental conditions in both examples, and Example 2 contained duplicated cells and mismatched markdown from Example 1.

falsification_score_sample now correctly returns condition values and z-scored scores.
Replaced np.array(list(new_conditions)), which returned [0], with .to_numpy() conversion.
Removed duplicated cells from Exercise 2.

Type of change

  • fix: A bug fix

Remarks (Optional)

Ensures both Example 1 and Example 2 run correctly top-to-bottom and when individual cells are re-run. Output now consistently reflects selected experimental conditions.

@musslick musslick self-requested a review August 11, 2025 07:12
@musslick musslick added documentation Improvements or additions to documentation. good first issue Good for newcomers. priority 1 - needed These are highly desirable to be fixed, ideally within 2 weeks. labels Aug 11, 2025
Copy link
Collaborator

@musslick musslick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for fixing!

@musslick musslick added this pull request to the merge queue Aug 11, 2025
Merged via the queue into AutoResearch:main with commit 959844d Aug 11, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. good first issue Good for newcomers. priority 1 - needed These are highly desirable to be fixed, ideally within 2 weeks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants