You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Current state\n\nhpc/.bashrc still contains two functions that hardcode a specific 2022 WezTerm AppImage filename:\n\nbash\nfunction imgcat() {\n\t~/WezTerm-20221119-145034-49b9839f-Ubuntu18.04.AppImage imgcat \"$1\"\n}\n\nfunction fzf-img() {\n\tlocal width=\"${1:-auto}\"\n\tfzf --preview \"case {} in\n\t\t*.png|*.jpg|*.tif) ~/WezTerm-20221119-145034-49b9839f-Ubuntu18.04.AppImage imgcat --width $width {} ;;\n\t\t*) echo not image ;;\n\tesac\" --preview-window='down'\n}\n\n\nBoth will silently fail on any HPC session that doesn't have that exact binary at ~/. The kitty replacements already exist in the same file:\n\nbash\nalias icat=\"kitten icat\" # replaces imgcat()\nfunction fzf-kitty-img() { ... } # replaces fzf-img()\n\n\n## Suggested fix\n\nRemove imgcat() and fzf-img() entirely — their kitten equivalents (icat and fzf-kitty-img) already cover the same use cases.\n\nNote: PR #90 proposed the same fix but was closed without merging.
Current state\n\n
hpc/.bashrcstill contains two functions that hardcode a specific 2022 WezTerm AppImage filename:\n\nbash\nfunction imgcat() {\n\t~/WezTerm-20221119-145034-49b9839f-Ubuntu18.04.AppImage imgcat \"$1\"\n}\n\nfunction fzf-img() {\n\tlocal width=\"${1:-auto}\"\n\tfzf --preview \"case {} in\n\t\t*.png|*.jpg|*.tif) ~/WezTerm-20221119-145034-49b9839f-Ubuntu18.04.AppImage imgcat --width $width {} ;;\n\t\t*) echo not image ;;\n\tesac\" --preview-window='down'\n}\n\n\nBoth will silently fail on any HPC session that doesn't have that exact binary at~/. The kitty replacements already exist in the same file:\n\nbash\nalias icat=\"kitten icat\" # replaces imgcat()\nfunction fzf-kitty-img() { ... } # replaces fzf-img()\n\n\n## Suggested fix\n\nRemoveimgcat()andfzf-img()entirely — their kitten equivalents (icatandfzf-kitty-img) already cover the same use cases.\n\nNote: PR #90 proposed the same fix but was closed without merging.