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.

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

install tablicious 0.3.5 fails on Octave 7.x #91

Closed
geerligs opened this issue Apr 21, 2022 · 7 comments · Fixed by #95
Closed

install tablicious 0.3.5 fails on Octave 7.x #91

geerligs opened this issue Apr 21, 2022 · 7 comments · Fixed by #95
Assignees
Labels
bug Something isn't working octave-7 Octave 7.x compatibility
Milestone

Comments

@geerligs
Copy link

When trying to install the tablicious package I get some errors:

>> pkg install "https://github.com/apjanke/octave-tablicious/archive/v0.3.6.tar.gz"
warning: function C:\Users\han\AppData\Roaming\octave\api-v57\packages\tablicious-0.3.6\endsWith.m shadows a core libra
ry function
warning: called from
    doc_cache_create>gen_doc_cache_in_dir at line 146 column 5
    doc_cache_create at line 62 column 12
    install>generate_lookfor_cache at line 840 column 5
    install at line 241 column 7
    pkg at line 603 column 9

warning: function C:\Users\han\AppData\Roaming\octave\api-v57\packages\tablicious-0.3.6\startsWith.m shadows a core lib
rary function
warning: called from
    doc_cache_create>gen_doc_cache_in_dir at line 146 column 5
    doc_cache_create at line 62 column 12
    install>generate_lookfor_cache at line 840 column 5
    install at line 241 column 7
    pkg at line 603 column 9

error: '__event_manager_register_doc__' undefined near line 41, column 7
error: called from
    load_tablicious at line 41 column 7
    C:\Users\han\AppData\Roaming\octave\api-v57\packages\tablicious-0.3.6\PKG_ADD at line 3 column 1
    doc_cache_create>gen_doc_cache_in_dir at line 146 column 5
    doc_cache_create at line 62 column 12
    install>generate_lookfor_cache at line 840 column 5
    install at line 241 column 7
    pkg at line 603 column 9

I tried also in admin mode.

Any clues how to solve this?

My ver is:

----------------------------------------------------------------------
GNU Octave Version: 7.1.0 (hg id: 04120d65778a)
GNU Octave License: GNU General Public License
Operating System: MINGW32_NT-6.2 Windows 6.2  x86_64
----------------------------------------------------------------------
Package Name         | Version | Installation directory
---------------------+---------+-----------------------
              audio  |   2.0.4 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\audio-2.0.4
             biosig  |   2.4.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\biosig-2.4.0
     communications  |   1.2.4 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\communications-1.2.4
            control  |   3.4.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\control-3.4.0
     data-smoothing  |   1.3.0 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\data-smoothing-1.3.0
           database  |   2.4.4 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\database-2.4.4
          dataframe  |   1.2.0 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\dataframe-1.2.0
              dicom  |   0.4.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\dicom-0.4.1
          financial  |   0.5.3 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\financial-0.5.3
               fits  |   1.0.7 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\fits-1.0.7
fuzzy-logic-toolkit  |   0.4.6 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\fuzzy-logic-toolkit-0.4.6
                 ga  |  0.10.3 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\ga-0.10.3
            general  |   2.1.2 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\general-2.1.2
      generate_html  |   0.3.2 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\generate_html-0.3.2
           geometry  |   4.0.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\geometry-4.0.0
                gsl  |   2.1.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\gsl-2.1.1
              image  |  2.14.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\image-2.14.0
 instrument-control  |   0.7.1 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\instrument-control-0.7.1
           interval  |   3.2.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\interval-3.2.1
                 io  |   2.6.4 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\io-2.6.4
     linear-algebra  |   2.2.3 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\linear-algebra-2.2.3
               lssa  |   0.1.4 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\lssa-0.1.4
              ltfat  |   2.3.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\ltfat-2.3.1
            mapping  |   1.4.2 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\mapping-1.4.2
            matgeom  |   1.2.3 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\matgeom-1.2.3
      miscellaneous  |   1.3.0 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\miscellaneous-1.3.0
                nan  |   3.6.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\nan-3.6.1
             netcdf  |  1.0.14 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\netcdf-1.0.14
              nurbs  |   1.4.3 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\nurbs-1.4.3
                ocs  |   0.1.5 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\ocs-0.1.5
            octproj  |   2.0.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\octproj-2.0.1
              optim  |   1.6.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\optim-1.6.1
        optiminterp  |   0.3.7 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\optiminterp-0.3.7
         quaternion  |   2.4.0 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\quaternion-2.4.0
           queueing  |   1.2.7 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\queueing-1.2.7
             signal  |   1.4.1 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\signal-1.4.1
            sockets  |   1.3.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\sockets-1.3.0
          sparsersb  |   1.0.9 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\sparsersb-1.0.9
            splines  |   1.3.4 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\splines-1.3.4
         statistics  |   1.4.3 | ...\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\statistics-1.4.3
                stk  |   2.7.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\stk-2.7.0
            strings  |   1.2.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\strings-1.2.0
             struct  |  1.0.17 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\struct-1.0.17
           symbolic  |   2.9.0 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\symbolic-2.9.0
             tisean  |   0.2.3 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\tisean-0.2.3
                tsa  |   4.6.3 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\tsa-4.6.3
              video  |   2.0.2 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\video-2.0.2
            windows  |   1.6.2 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\windows-1.6.2
             zeromq  |   1.5.3 | D:\Program Files\GNU Octave\Octave-7.1.0\mingw64\share\octave\packages\zeromq-1.5.3
[
@geerligs geerligs changed the title install failed install tablicious 0.3.5 fails Apr 21, 2022
@apjanke
Copy link
Owner

apjanke commented Apr 22, 2022

This looks like a problem with Tablicious not being compatible with Octave 7.1.

See this?

error: '__event_manager_register_doc__' undefined near line 41, column 7
error: called from
    load_tablicious at line 41 column 7
    C:\Users\han\AppData\Roaming\octave\api-v57\packages\tablicious-0.3.6\PKG_ADD at line 3 column 1
    doc_cache_create>gen_doc_cache_in_dir at line 146 column 5
    doc_cache_create at line 62 column 12
    install>generate_lookfor_cache at line 840 column 5

That's Tablicious trying to register its interactive documentation stuff with the Octave GUI/IDE (with the whole Qt Help system thing). And it's using - okay, fine, a probably-unsupported undocumented private function to do so. And it looks like this private function has been changed or removed in Octave 7.x.

This is Tablicious's fault: it needs to add support for the Octave 7.x way of doing this. And regardless of that, it should be trapping the help-system registration with a try/catch in its package-loading code so that a failure registering the GUI help stuff doesn't break the whole library-loading process.

Thanks for the bug report! I'll see if I can get to this over this weekend.

@apjanke apjanke added bug Something isn't working octave-7 Octave 7.x compatibility labels Apr 22, 2022
@apjanke apjanke added this to the 0.4.0 milestone Apr 22, 2022
@apjanke apjanke added this to Needs triage in Octave-Tablicious via automation Apr 22, 2022
@apjanke apjanke self-assigned this Apr 22, 2022
@apjanke apjanke moved this from Needs triage to High priority in Octave-Tablicious Apr 22, 2022
@geerligs
Copy link
Author

hi Andrew,

thanks for the prompt response. Looking forward to a fix.

--Han

@carlosal1015
Copy link

carlosal1015 commented May 10, 2022

I reproduce the same issue in Arch Linux with octave 7.1.0-3 and tablicious 0.3.6.

Also, with the last commit b42f860a35aa05140eb1196c3c33b2dfbbeead0d from master.

@geerligs
Copy link
Author

Hi Andrew,

any news from the bug-fixing?

regards, Han

@gkourachanis
Copy link
Contributor

gkourachanis commented Jul 9, 2022

Hello! This pull request fixes the issue.
It fixes it for me in GNU Octave v7.1.0, tested in Archlinux and Debian Sid.

@apjanke
Copy link
Owner

apjanke commented Jul 13, 2022

Hi y'all! Sorry for the delay in getting this tested & merged. Personal & security-related stuff that I can't really talk about yet. Should get to this soon.

@apjanke
Copy link
Owner

apjanke commented Jul 14, 2022

Okeedoke, I found some time to test the code in the PR. Code looks good; just want to clean up the Git history aspects of the PR before merging and then we're good to go here.

This is a significant enough fix that I'll cut a new release of Tablicious once this PR is merged.

Octave-Tablicious automation moved this from High priority to Closed Aug 26, 2022
@apjanke apjanke modified the milestones: 0.4.0, 0.3.7 Sep 1, 2022
@apjanke apjanke changed the title install tablicious 0.3.5 fails install tablicious 0.3.5 fails on Octave 7.x Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working octave-7 Octave 7.x compatibility
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants