-
Notifications
You must be signed in to change notification settings - Fork 41
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
6.0.0: Final Fixes #3122
6.0.0: Final Fixes #3122
Conversation
Whatsnew's parent should be the mainwindow, not the gui manager. Also whatsnew is not passing on its parent to the QDialog super class.
…e resulting data is plottable
Hmm the installation doesn't work for me (Win10, 64bit).
and SasView window doesn't show up |
Try deleting the config6.json file in your |
…d this before...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jamescrake-merani and I looked at code, checked installers on windows and mac, LGTM!!
I am getting the following error when with fitting with DREAM: I think we fixed it at some point? |
I guess we need to pull #3110 |
Then the DREAM error will need to be investigated. Can you reliably reproduce the issue? |
The error I see is very reproducible. I've tested both arm and x86 installer and it is reproducible. It appears no matter which fit algorithm I use before switching to DREAM. The error also appears regardless if I choose Default Fit algorithm or Fit Algorithm Screen.Recording.2024-10-15.at.20.56.40.mov |
@wpotrzebowski - I can reproduce it on Windows as well. This might hold up the release. |
Also able to reproduce this error on the Linux Flatpak build. |
The issue noted by @wpotrzebowski is going to hold up the release. I've been trying to diagnose this and can't find a proper solution. In the mean time, I'm merging this PR so the release branch is up to date |
Description
This is a series of changes that are needed for the final 6.0.0 release. Items changed in this PR:
NaN
which isn't plottable. (Note, this is a quick hack to access a sasdata class method that should really be its own function.)ANY OTHER FIXES MAY GO HERE
Refs https://github.com/orgs/SasView/discussions/3116
Review Checklist:
[if using the editor, use
[x]
in place of[ ]
to check a box]Documentation (check at least one)
Installers
Licencing (untick if necessary)