1 | - BUGS |
---|
2 | |
---|
3 | 1- [DONE] After fitting, the fit panel is not updated with the output values of the fit. |
---|
4 | |
---|
5 | 2- [DONE] Do this: |
---|
6 | a) load the file line_in_x2_y.txt |
---|
7 | b) fit a straight line, close the Fit Dialog |
---|
8 | c) change the scale using the "Properties" menu to x vs y (linear vs linear) |
---|
9 | You will see an exception that should be fixed. |
---|
10 | |
---|
11 | 3- [DONE] Load the file line_in_x2_y.txt in miniView. Select x^2 and y for the scales. Fit it. |
---|
12 | a) The fit range is wrong, it's not update for the X^2 view. |
---|
13 | b) the fit result is wrong |
---|
14 | c) Even if you update the range, the fit and the plots are wrong. |
---|
15 | |
---|
16 | 4- [DONE]At the python command prompt, try: |
---|
17 | from sans.guitools.PlotPanel import PlotPanel |
---|
18 | |
---|
19 | You will get an exception: ImportError: No module named fittings |
---|
20 | |
---|
21 | Guitools should not depend on your miniView code. |
---|
22 | |
---|
23 | 5- [QUESTION] Each time you open and close the Fit Dialog, a new label "Theory1D" is created in the |
---|
24 | legend. There should always be only one, for the _last_ fit, and the name should be "Fit". |
---|
25 | [alina] is it fine if I delete the fit line from the graph |
---|
26 | and remove it when I close the Fit Dialog window? |
---|
27 | [Mathieu] That might work. I kind of like it. Let's do it like that and see what the users |
---|
28 | need. |
---|
29 | |
---|
30 | 6-[QUESTION ] Do this: |
---|
31 | a) Load the file 'line_in_logy_x.txt' |
---|
32 | b) Fit a straight line |
---|
33 | c) The fit shown does not start from the first point and it's not a straight line. |
---|
34 | [ALINA] then for transformation to log Do I have to transform x to log and set x linear? |
---|
35 | [Mathieu] No. That would confuse the user, who expects a log scale. The answer here is to fit |
---|
36 | y = A * log(x) + B, which is a straight line in the y vs log(x) representation. |
---|
37 | |
---|
38 | --> [ALINA] woring better...I am concerned how to fit a straight line when x is changing too? |
---|
39 | |
---|
40 | |
---|
41 | - NOTES that might help for bugs |
---|
42 | 1- Make sure you are fitting the View data, not the underlying data |
---|
43 | 2- Since you are using the log scale from matplotlib, the fit for when one of |
---|
44 | the axis is in log will NOT be a straight line |
---|
45 | 3- Make sure that you are filling the View of the fit result correctly. |
---|
46 | [ALINA] not using the log of matplotlib anymore but the axis are never in the log scale by the value are in log10 x |
---|
47 | [Mathieu] See my comment to point 6. The right answer is to go back to using the log scale of matplotlib. |
---|
48 | |
---|
49 | - NEEDED IMPROVEMENTS |
---|
50 | 1-[DONE] After loading a new file or changing the scale using the "Properties" menu, remove any previously displayed fit from the graph. |
---|
51 | |
---|
52 | 2- There are additional y scales we need: y, y^2, log(y), 1/y, 1/sqrt(y), log(y * x), log(y * x^2) |
---|
53 | [QUESTION] when plottong y=log(y*x) is the user allowed to do transformation on x at the same time |
---|
54 | for example x---> log X with y=log(x*y) |
---|
55 | [PROBLEM]WITH log(y * x), log(y * x^2) |
---|
56 | |
---|
57 | 3-[DONE] You should change the label according to what the user chose, and not hard-code them. |
---|
58 | For example, in PlotPanel._onEVT_FUNC_PROPERTY, replace |
---|
59 | |
---|
60 | self.graph.xaxis('\\rm{q^{2}} ', 'A^{-2}') |
---|
61 | |
---|
62 | with |
---|
63 | name, units = self.some_plottable.get_xaxis() |
---|
64 | self.graph.xaxis("(%s)^2" % name, "(%s)^2" % units) |
---|
65 | |
---|
66 | That way it still works even if the user is not plotting Intensity versus Q. |
---|
67 | |
---|
68 | 4- [DONE] In the fit dialog, put the "fit" and "Close" button in the bottom right corner, like in a standard file dialog. |
---|
69 | |
---|
70 | 5- The layout of the Fit Dialog should be improved. |
---|
71 | |
---|
72 | 6- The code should be well documented and clean-up (no print statements). |
---|
73 | |
---|