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

Fixed bug with with ordered factors #9344

Merged
merged 4 commits into from
Jan 10, 2025

Conversation

Vitalis95
Copy link
Contributor

Fixes #9328
@rdstern @N-thony , I noticed that the issue also affects other dialogs, such as the Replace Values dialog.

@Vitalis95
Copy link
Contributor Author

@rdstern, It also resolves the regression in the One > Variable > Summarise dialog mentioned in #9311

rdstern
rdstern previously approved these changes Jan 10, 2025
Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

@Vitalis95 excellent. Could you say anything about how that one-variable regression happened?
If you have time for a cosmetic change then I note that when a table is produced it has escaped our editing and still says Save Table, rather than Store Table. That's both for 2 way and 3 way.
But, if not then that can be done later.
@N-thony I am approving and with the bug fix, we must merge this, before producing the version today. I am very releived it is fixed.

@@ -413,6 +413,8 @@ Public Class ucrReceiverMultiple
For i As Integer = 0 To strDataTypes.Count - 1
If strDataTypes(i).Contains("factor") OrElse strDataTypes(i).Contains("character") Then
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Vitalis95, how about this?
If strDataTypes(i).Contains("factor") OrElse strDataTypes(i).Contains("character") OrElse strDataTypes(i).Contains("ordered,factor") Then

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines 416 to 417
ElseIf strDataTypes(i).Contains("ordered") Then
strDataTypes(i) = "categorical"
Copy link
Collaborator

Choose a reason for hiding this comment

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

then remove 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.

done

@Vitalis95
Copy link
Contributor Author

@rdstern , the multiple receiver in One > Variable was not setting the R code properly

Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

Still ok to me
Back to you @N-thony

@N-thony N-thony merged commit d5ddca1 into IDEMSInternational:master Jan 10, 2025
2 checks passed
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.

Problem with ordered factors in 2/3 variable summaries
3 participants