Skip to content

Conversation

boutinb
Copy link
Contributor

@boutinb boutinb commented Aug 28, 2024

The levels constraints in Variables Lists did not take the label filtering into account.
Also the levels constraints should be checked dynamically, that is each time the filtering is changed, or the constraints self changes.
Fixes jasp-stats/jasp-issues#2853
Fixes https://github.com/jasp-stats/INTERNAL-jasp/issues/2639
Fixes https://github.com/jasp-stats/INTERNAL-jasp/issues/2640
Fixes jasp-stats/jasp-issues#2853

@JorisGoosen
Copy link
Contributor

@boutinb is this the pr that should go into the hotfix?
If so we should merge it to stable and not to development right?

@boutinb boutinb changed the base branch from development to stable September 4, 2024 10:47
@boutinb
Copy link
Contributor Author

boutinb commented Sep 4, 2024

@boutinb is this the pr that should go into the hotfix? If so we should merge it to stable and not to development right?

Yep! I have rebased the branch

@JorisGoosen
Copy link
Contributor

@JorisGoosen
Copy link
Contributor

@boutinb you will let me know when this is ready for review again right?

@boutinb
Copy link
Contributor Author

boutinb commented Sep 5, 2024

@boutinb you will let me know when this is ready for review again right?

This is now ready for review.
I have made an optimization for the counting of filtered levels: I tested it with 1M rows, adding several variables in a model is now significant quicker.

@JorisGoosen
Copy link
Contributor

Very nice!

@JorisGoosen JorisGoosen merged commit 382d498 into jasp-stats:stable Sep 5, 2024
1 check passed
@Joao-O-Santos
Copy link
Contributor

Joao-O-Santos commented Sep 6, 2024

Thank you for fixing this labeling issue so quickly and for all the work you do!
Apologies if this is not the correct place to ask this question.

@JorisGoosen, would you happen to have a date for the release with this bugfix (I'm guessing version 0.19.1)?
I'm collaborating in a course that recently switched to JASP and we're getting everything ready for this new academic year.
We really like that version 0.19 adds the option to check the qqplot for the residuals in more models/tests.
In the last semester we sometimes had exercises that require students to apply filters.
I'm just trying to see if we'll have a new release before classes start or if we should remove the exercises with filters until then (no worries either way).

I guess the fix is already in the nightly builds, but I fear teaching students to install the nightly builds is more trouble than its worth (it will be better to remove the exercises with filters until the bugfix is released in the stable build).

@boutinb
Copy link
Contributor Author

boutinb commented Sep 6, 2024

@Joao-O-Santos We will deliver a hotfix version (0.19.1) with this just this fix next week.

@Joao-O-Santos
Copy link
Contributor

Joao-O-Santos commented Sep 6, 2024

@Joao-O-Santos We will deliver a hotfix version (0.19.1) with this just this fix next week.

Thank you so much for the quick reply!
Amazing news!

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]: filter being ignored in 0.19

3 participants