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 bugs on the New Data Frame dialog #8169

Merged

Conversation

derekagorhom
Copy link
Contributor

Fixes #8167

This fixes the recent bug found when the dialogue fails to create a dataframe with less than 3 columns.

@ChrisMarsh82 , @rdstern, this PR is ready for review.

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.

@derekagorhom that's a releif that I think the main bug is fixed. I couldn't get an error. But there are still some more bits to correct.
a) As mentioned previously, when I untick the Variable Name checkbox it also works. But when I return to the dialogue the Variable Name is ticked for me again. The rule, in R-Instat is that the controls "remember" how they were left. So please keep that control unticked if that was done the last time Ok was pressed.
b) I am also happy that changing a name in the variables now seems ok. But changing the type to numeric, or factor doesn't work yet. Please can those be fixed too.

@rdstern
Copy link
Collaborator

rdstern commented Feb 24, 2023

@derekagorhom I hope this is still work in progress - but you need to comment more when you make changes, so I know when I should look again. I am very keen that this is fixed in time for the February release this weekend.
So I now see the change has simply unticked the Variable Name (as it was originally). That's not what I asked for.
I am happy that it starts by being ticked, as the initial default.
But when I then choose to untick it and press OK, it should then remember that setting for when I re-open the dialogue.

And the other problems - only character variables despite what I change - still remain. Please also discuss with @N-thony because he did the original code and it worked fine then.

@derekagorhom
Copy link
Contributor Author

@rdstern i fixed the issue with the variable type and also made code changes to check variable name so that it is default and it can be ticked and unticked with no issues.

rdstern
rdstern previously approved these changes Feb 27, 2023
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.

@derekagorhom, well done it all seems to work fine now.
@lloyddewit it would be good to merge this, once you are happy with the code. @ChrisMarsh82 found serious problems with the Version 0.7.8 version.

instat/dlgNewDataFrame.vb Outdated Show resolved Hide resolved
instat/dlgNewDataFrame.vb Outdated Show resolved Hide resolved
instat/dlgNewDataFrame.vb Show resolved Hide resolved
instat/dlgNewDataFrame.vb Outdated Show resolved Hide resolved
@lloyddewit
Copy link
Contributor

@rdstern I noticed that this PR reversed some of the changes made in PR #8148 . Please could you test that the problem solved by PR #8148 is still fixed?
If you're happy, then we can merge, thanks

@lloyddewit lloyddewit added the bug label Mar 1, 2023
@lloyddewit lloyddewit changed the title Fixed Bugs on the NewDataFrame Dialogue Fixed regression bugs on the New Data Frame dialog Mar 1, 2023
@lloyddewit lloyddewit merged commit 44ed584 into IDEMSInternational:master Mar 1, 2023
@rdstern rdstern changed the title Fixed regression bugs on the New Data Frame dialog Fixed bugs on the New Data Frame dialog Mar 1, 2023
@derekagorhom derekagorhom deleted the Fixes_to_Script_Widow branch March 1, 2023 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New bug with File > New Data Frame dialogue
3 participants