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

Computed column fixes #5676

Merged
merged 3 commits into from
Sep 25, 2024
Merged

Conversation

JorisGoosen
Copy link
Contributor

Fixes jasp-stats/jasp-issues#2927 (getting TRUE/FALSE instead of 1/0) and https://github.com/jasp-stats/INTERNAL-jasp/issues/2653 (computed columns dont work in development...)

Do this by including the R-files with "friendly functions" from jaspBase into Engine and then passing that as an init-script to the engine.
Fixes jasp-stats/INTERNAL-jasp#2653
@boutinb
Copy link
Contributor

boutinb commented Sep 24, 2024

This does not work for Ordinal or Nominal computed column: if you change the computed column to Ordinal or Nominal it still gives NA's. If you also make directly an ordinal computed column (with the + button in Analysis mode), it also gives NA's.
Screenshot 2024-09-24 at 15 18 03

@JorisGoosen
Copy link
Contributor Author

JorisGoosen commented Sep 24, 2024

Disable "Converting types", or in any case, make sure all the types make sense

I will remove that functionality once #5667 gets merged

@boutinb boutinb merged commit 072b045 into jasp-stats:development Sep 25, 2024
1 check passed
JorisGoosen added a commit that referenced this pull request Sep 30, 2024
* Handle TRUE/FALSE better in compute columns

Fixes jasp-stats/jasp-issues#2927

* dont require jaspBase for computed columns etc

Do this by including the R-files with "friendly functions" from jaspBase into Engine and then passing that as an init-script to the engine.
Fixes jasp-stats/INTERNAL-jasp#2653

* actually fix the problem
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.

[Bug]: Computing data using drag-and-drop
2 participants