Peek support for ipynb(jupyter notebook)

[ohhh-yang]

Description of the new feature / enhancement

Peek should support more file format, such as ipynb(jupyter notebook)

Scenario when this would be used?

Peek does not support many file format, it shoudn't be.

Supporting information

No response

[github-actions]

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

• [Peek] Support for GrahPad, ChemDRAW, NMR (#32016), similarity score: 0.76

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

[ohhh-yang]

No, not the same.

…

---Original--- From: ***@***.***> Date: Wed, Jun 19, 2024 10:42 AM To: ***@***.***>; Cc: ***@***.******@***.***>; Subject: Re: [microsoft/PowerToys] Peek should support more file format, suchas ipynb(jupyter notebook) (Issue #33434) Hi I'm an AI powered bot that finds similar issues based off the issue title. Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you! Open similar issues: [Peek] Support for GrahPad, ChemDRAW, NMR (#32016), similarity score: 0.76 Note: You can give me feedback by thumbs upping or thumbs downing this comment. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: ***@***.***>

[htcfreek]

@Paulinho00
Your contribution is welcome and thank you for your work on this. Feel free to open a PR. If you have any further questions don't worry to ask them.

[Paulinho00]

I investigated it a bit and I've not found a way to display a notebook as it is shown e.g. in VSC (with cells and output formatting). I considered converting notebook to already supported file extensions (e.g. html or markdown), but there is no easy way of doing it. It basically would require writing our converter. I also considered displaying notebook as json but I'm not sure about it, because it is not very readable and probably it wasn't what notebook users expects. Do you have any idea how can it be displayed?

[htcfreek]

@Paulinho00
Sory I can't help here. I never used this files in my computer life yet.

[Paulinho00]

How to debug Peek? Is there any way to debug without building installer etc. ?

[htcfreek]

How to debug Peek? Is there any way to debug without building installer etc. ?

cc: @davidegiacometti , @jaimecbernardo

[jaimecbernardo]

It's possible to build all projects from Visual Studio and start from the runner.
The Runner starts the Peek process and then you can attach a debugger to it.
I also use this plugin for some of these scenarios so that a debugger is automatically attached to new processes created by processes we are debugging: https://marketplace.visualstudio.com/items?itemName=vsdbgplat.MicrosoftChildProcessDebuggingPowerTool2022

[Paulinho00]

Do you know why I get Failed to load PowerToys.FindMyMouse.dll and a couple more similar looking errors immediately after the build process is completed? I build solution step by step as prescribed in guidelines. Here is example call stack

KernelBase.dll!00007ffc1a99f39c() Unknown
PowerToys.exe!CxxThrowException(void * pExceptionObject, const s__ThrowInfo * pThrowInfo) Line 81 C++
PowerToys.exe!winrt::throw_hresult(const winrt::hresult result, const std::source_location & sourceInformation) Line 5163 C++
PowerToys.exe!winrt::throw_last_error(const std::source_location & sourceInformation) Line 5226 C++
PowerToys.exe!winrt::check_pointer<HINSTANCE>(HINSTANCE__ * pointer, const std::source_location & sourceInformation) Line 5272 C++
PowerToys.exe!load_powertoy(const std::basic_string_view<wchar_t,std::char_traits<wchar_t>> filename) Line 16 C++
PowerToys.exe!runner(bool isProcessElevated, bool openSettings, std::string settingsWindow, bool openOobe, bool openScoobe, bool showRestartNotificationAfterUpdate) Line 175 C++
PowerToys.exe!WinMain(HINSTANCE__ * formal, HINSTANCE * __formal, char * lpCmdLine, int __formal) Line 452 C++
PowerToys.exe!invoke_main() Line 107 C++
PowerToys.exe!__scrt_common_main_seh() Line 288 C++
PowerToys.exe!__scrt_common_main() Line 331 C++
PowerToys.exe!WinMainCRTStartup(void * __formal) Line 17 C++
kernel32.dll!00007ffc1b3d257d() Unknown
ntdll.dll!00007ffc1d00af28() Unknown

[htcfreek]

@Paulinho00
Are you still working on this? (@asif4318 is asking if he can work on this.)

[asif4318]

Is the goal to have it displayed in the editor formatted with cells & markdown?
.ipynb are just JSON files internally

Posted some screenshots to clarify.

[Paulinho00]

@htcfreek No, I'm not.

[htcfreek]

@asif4318
I personally think the notebook view makes most sense if possible. Because that the use case of this format. What do you think @crutkas ?