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.

Sign up for GitHub

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

Jump to bottom

Ensured output from Correlation dialog is correct #8117

Merged
merged 8 commits into from
Feb 13, 2023
Merged

Ensured output from Correlation dialog is correct #8117

merged 8 commits into from
Feb 13, 2023

Conversation

Vitalis95
Copy link
Contributor

Fixes partially #8104
@lloyddewit , @Patowhiz , @rdstern .

  • In multiple receiver , I have added the following for the new output system
    Save Table
    When Display Options is checked, it has
    image

For Save Table and Save data frame when the buttons are checked respectively

  • In Two columns it has Save Model control for saving the model
  • The sdg changes is because of setting R-code bug

@lloyddewit
Copy link
Contributor

@berylwaswa please could you test? thanks

@berylwaswa
Copy link

@Vitalis95 @lloyddewit this seems to work fine.

lloyddewit
lloyddewit requested changes Feb 10, 2023
View reviewed changes
berylwaswa
berylwaswa previously approved these changes Feb 10, 2023
View reviewed changes
@Vitalis95 @lloyddewit
Update instat/dlgCorrelation.vb
f87f0b5 
Co-authored-by: lloyddewit <57253949+lloyddewit@users.noreply.github.com>
Vitalis95 and others added 4 commits February 10, 2023 14:51
@Vitalis95 @lloyddewit
Update instat/dlgCorrelation.vb
66dbb52 
Co-authored-by: lloyddewit <57253949+lloyddewit@users.noreply.github.com>
@Vitalis95 @lloyddewit
Update instat/sdgCorrPlot.vb
b76725c 
Co-authored-by: lloyddewit <57253949+lloyddewit@users.noreply.github.com>
@Vitalis95
changes made
c100d32
@Vitalis95
Merge branch 'Multivariate_corr' of /~https://github.com/Vitalis95/R-In…
c9a0995 
…stat into Multivariate_corr
berylwaswa
berylwaswa reviewed Feb 10, 2023
View reviewed changes
Copy link

@berylwaswa berylwaswa left a comment

Choose a reason for hiding this comment

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

This looks fine.

@lloyddewit lloyddewit changed the title Ensured output from correlation dialog is correct Ensured output from Correlation dialog is correct Feb 13, 2023
@lloyddewit lloyddewit added the bug label Feb 13, 2023
@lloyddewit lloyddewit merged commit f069a18 into IDEMSInternational:master Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@berylwaswa berylwaswa berylwaswa approved these changes

@lloyddewit lloyddewit lloyddewit approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Vitalis95 @lloyddewit @berylwaswa