Skip to content
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

Add function for flipping coherent dots in a trial #147

Merged
merged 3 commits into from
Jan 12, 2025

Conversation

Hoboki
Copy link
Contributor

@Hoboki Hoboki commented Nov 25, 2024

This PR improves PLUGIN-RDK and adds a function that flip the direction of coherent dots.

You can change the direction of moving dots oppositely during a trial as many times as you want.

Information about added parameter follows below:
Name: flip_timestamps
Type: Array
Default: []

This function is similar one implemented in this study, where dots' directions are flipped probabilistically.
https://elifesciences.org/articles/08825

How to verify

please cd plugin-rdk, npm i and npm run build.
I modified example1.html so you can check how it works.

Thank you in advance!

Copy link

changeset-bot bot commented Nov 25, 2024

🦋 Changeset detected

Latest commit: f7655f2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@jspsych-contrib/plugin-rdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Hoboki
Copy link
Contributor Author

Hoboki commented Dec 9, 2024

Anyone can review this?

@Hoboki
Copy link
Contributor Author

Hoboki commented Jan 11, 2025

Do I need to make new plugin? @jodeleeuw

@jodeleeuw
Copy link
Member

@jadeddelta if you have a chance to look at this. I will try soon!

Copy link
Collaborator

@jadeddelta jadeddelta left a comment

Choose a reason for hiding this comment

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

looks good to me! just switch up the version bump type and i think we should be good to go

@@ -0,0 +1,5 @@
---
"@jspsych-contrib/plugin-rdk": patch
Copy link
Collaborator

Choose a reason for hiding this comment

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

we should probably make this a minor version bump, patch is usually reserved for bug fixes and minor is reserved for backwards-compatible feature releases like this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've changed the version bump from patch into minor

@Hoboki Hoboki requested a review from jadeddelta January 11, 2025 21:54
Copy link
Collaborator

@jadeddelta jadeddelta left a comment

Choose a reason for hiding this comment

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

everything looks good! i'll run it through one last time with Josh and we'll merge it!

@jadeddelta jadeddelta merged commit 1a7b167 into jspsych:main Jan 12, 2025
@github-actions github-actions bot mentioned this pull request Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants