After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 691892 - Regression tool generates references with redundant sheet
Regression tool generates references with redundant sheet
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: Analytics
git master
Other All
: Normal normal
: ---
Assigned To: Morten Welinder
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2013-01-16 20:45 UTC by Morten Welinder
Modified: 2013-01-24 21:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Morten Welinder 2013-01-16 20:45:37 UTC 
For example, I see

=mmult(mmult(mmult('Regression (1)'!$A$21:$B$2797,minverse(mmult(transpose('Regression (1)'!$A$21:$B$2797),'Regression (1)'!$A$21:$B$2797))),transpose('Regression (1)'!$A$21:$B$2797))*munit('Regression (1)'!$B$8+1),'Regression (1)'!$A$21:$A$2797)

It is unnecessary to include the sheet in those references since they
occur on sheet 'Regression (1)'.
Comment 1 Andreas J. Guelzow 2013-01-17 04:34:07 UTC 
This is not regression specific since several tools call dao_get_rangeref. There is only one call:
src/tools/dao.c: return gnm_go_data_vector_new_expr (dao->sheet, gnm_expr_top_new (dao_get_rangeref (dao, ax, ay, bx, ...
that might need to set the sheet to non-NULL.
Comment 2 Morten Welinder 2013-01-17 13:14:42 UTC 
If we were to change dao_get_cellref and dao_get_rangeref, would anything
break?

Equivalently, do these formula ever need to reach another sheet?
Comment 3 Andreas J. Guelzow 2013-01-17 17:07:37 UTC 
For use in the formulas analysis tools neither dao_get_cellref nor dao_get_rangeref needs to use an external sheet.

There is only one place, in dao_go_data_vector, where we might need to have the sheet specified since that creates a GOData to be attached to graph, that at the time of attachment resides on the same sheet.
Comment 4 Andreas J. Guelzow 2013-01-24 21:44:39 UTC 
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.