-
Notifications
You must be signed in to change notification settings - Fork 214
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
data export with x- and y-axis as log scale #345
Comments
Hello, I am wondering if you have a sample data file that triggers the problem. Sorry for the slow response |
https://yadi.sk/d/MPqkjXODey32gw this is the corresponding file. I only made the changes in Coordinates- and in Export Format Settings (see figures below). Then I tried to export the data in csv format. But the program starts to hang (with 100% CPU load in an one thread) when the two axes are set to log and you click on the export settings. |
Thanks. I am able to see the problem. This should be fixed quickly. After testing you can try out the fixed version - about 2 days. |
There is a 64 bit Windows pre-release at https://ci.appveyor.com/project/markummitchell/engauge-digitizer/branch/master/job/tahk5voottucfy38/artifacts. You can extract that file using tools like 7zip or winzip. |
That is working as designed. The step factor in a log plot is multiplicative. So values greater than 1 are needed since the pattern is ymin, ymin * scale, ymin * scale^2, ..., ymax The Export Settings dialog now disables the Ok button, and does not insert a series of points, when the step is less than one for log scale. |
Good, I did not know that. The program works as you described. So I will mark this issue as solved. Many thanks :) |
Thanks for the input. This fix will help many people in the future |
When I set x- and y-axis as log scale and try export data the program hangs. It seams to be an endless while loop in
ExportValuesXOrY ExportXThetaValuesMergedFunctions::periodicLogGraph(...)
function.
The text was updated successfully, but these errors were encountered: