NotebookNavigator.nvim icon indicating copy to clipboard operation
NotebookNavigator.nvim copied to clipboard

first initialisation in latest Lazy nvim throws an erorr

Open parabolabamse opened this issue 9 months ago • 4 comments

Failed to run config for NotebookNavigator.nvim

vim/shared.lua:0: s: expected string, got number

stacktrace:

  • vim/shared.lua:0 in validate
  • vim/shared.lua:0 in startswith
  • vim/_init_packages.lua:0 in __index
  • vim/shared.lua:0 in islist
  • vim/shared.lua:0 in ****
  • vim/shared.lua:0 in tbl_deep_extend
  • /hydra.nvim/lua/hydra/layer/init.lua:116 in initialize
  • /hydra.nvim/lua/hydra/lib/class.lua:14 in Layer
  • /hydra.nvim/lua/hydra/init.lua:437 in _setup_pink_hydra
  • /hydra.nvim/lua/hydra/init.lua:281 in initialize
  • /hydra.nvim/lua/hydra/lib/class.lua:14 in hydra
  • /NotebookNavigator.nvim/lua/notebook-navigator/init.lua:236 in activate_hydra
  • /NotebookNavigator.nvim/lua/notebook-navigator/init.lua:327 in setup
  • ~/.config/nvim/lua/plugins/slime.lua:30 in config

parabolabamse avatar May 06 '25 17:05 parabolabamse

Having the same problem. New to neovim, so not sure what's going wrong.

irowberryFS avatar May 16 '25 21:05 irowberryFS

Hi, I encountered the same problem.

skprzyluska avatar May 17 '25 08:05 skprzyluska

same problem here

I can get around this by removing the config function but obviously the hydra key doesn't work. You can still go ahead and run cells though - I am just starting with this plugin so not sure the use of hydra keys anyways yet. I mostly just wanna run stuff. For clarity, here is my config

  {
    'GCBallesteros/NotebookNavigator.nvim',
    keys = {
      {
        ']h',
        function()
          require('notebook-navigator').move_cell 'd'
        end,
      },
      {
        '[h',
        function()
          require('notebook-navigator').move_cell 'u'
        end,
      },
      { '<leader>X', "<cmd>lua require('notebook-navigator').run_cell()<cr>" },
      { '<leader>x', "<cmd>lua require('notebook-navigator').run_and_move()<cr>" },
    },
    dependencies = {
      'echasnovski/mini.comment',
      'benlubas/molten-nvim', -- alternative repl provider
      'anuvyklack/hydra.nvim',
    },
    event = 'VeryLazy',
  },

nwpulverizer avatar Jul 01 '25 21:07 nwpulverizer

i also found switching to nvimtools/hydra instead of the unmaintained one suggested in the readme allows this function to run without issue, ~~but pressing leader h or whatever you decide to set the hydra key to doesn't do anything for me, although I don't really know what it is supposed to do.~~ nevermind, I must not have restarted my neovim instance i was testing this in. Switching to nvimtools/hydra fixes this. Here is a working config. Note I changed the activation key to leader p.

  {
    'GCBallesteros/NotebookNavigator.nvim',
    keys = {
      {
        ']h',
        function()
          require('notebook-navigator').move_cell 'd'
        end,
      },
      {
        '[h',
        function()
          require('notebook-navigator').move_cell 'u'
        end,
      },
      { '<leader>X', "<cmd>lua require('notebook-navigator').run_cell()<cr>" },
      { '<leader>x', "<cmd>lua require('notebook-navigator').run_and_move()<cr>" },
    },
    dependencies = {
      'echasnovski/mini.comment',
      'benlubas/molten-nvim', -- alternative repl provider
      'nvimtools/hydra.nvim',
    },
    event = 'VeryLazy',
    config = function()
      local nn = require 'notebook-navigator'
      nn.setup { activate_hydra_keys = '<leader>p' }
    end,

I have a fork on my profile waiting for approval of a PR with this change in the readme as well as a change in the library that prevents a warning from occuring due to deprecated hydra code in the library.

nwpulverizer avatar Jul 01 '25 22:07 nwpulverizer