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

Ensured that Default Column and Variable Name checkbox function correctly in New DataFrame dialog #8148

Conversation

derekagorhom
Copy link
Contributor

@derekagorhom derekagorhom commented Feb 17, 2023

fixes #8120
This is not ready for review as it needs further testing.
@N-thony , the fix works but it takes longer to load (probably from my laptop).
Also i found out that when you set the column to say 14 and reduce it to say 8, you get a column number between those 2 numbers.

@rdstern
Copy link
Collaborator

rdstern commented Feb 18, 2023

@derekagorhom while you are fixing that, I would like to explore with the Variable Name checkbox checked by default. Could you please make that change too.

@derekagorhom
Copy link
Contributor Author

@rdstern, i and @N-thony managed to reduce the loading time for the columns till approximately 290 columns, after that dialogue takes sometime to load the columns (but not as long as previously).
I believe this is ready for testing

@derekagorhom derekagorhom marked this pull request as ready for review February 20, 2023 11:41
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 this looks absolutely fine. @lloyddewit over to you.

@lloyddewit lloyddewit changed the title Fixing Bug in New DataFrame dialogue Ensured that Default Column and Variable Name checkbox function correctly in New DataFrame dialog Feb 21, 2023
@lloyddewit lloyddewit added the bug label Feb 21, 2023
@lloyddewit lloyddewit merged commit 3ca51d5 into IDEMSInternational:master Feb 21, 2023
@derekagorhom derekagorhom deleted the Output_System_TwoWay_Frequencies branch February 21, 2023 13:37
@ChrisMarsh82
Copy link
Contributor

@derekagorhom I am now unable to create a dataframe with less than 3 columns. Could you check that it wasn't this change that introduced this bug?

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.

Minor bug in the File > New Dataframe Dialogue
5 participants