Removed redundant installation of tagfiles in lite

[kr-2003]

Fixes #267

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.78%. Comparing base (9e2abcb) to head (df6106a).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #295   +/-   ##
=======================================
  Coverage   81.78%   81.78%           
=======================================
  Files          20       20           
  Lines         950      950           
  Branches       87       87           
=======================================
  Hits          777      777           
  Misses        173      173           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[anutosh491]

Hey @kr-2003,

Would need some help !

Could you please try all the inline documentation use cases mentioned here (https://xeus-cling.readthedocs.io/en/latest/inline_help.html)

There are 3 sections I suppose. These are xeus-cling docs, we need to see if everything works for lite. If yes, we need to update xeus-cpp's documentation to also add the same.

[kr-2003]

Could you please try all the inline documentation use cases mentioned here (https://xeus-cling.readthedocs.io/en/latest/inline_help.html)

I have tried for std::vector and xt::xtensor. It works fine for st::vector.

To enable documentation for third-party libs, we need to follow these steps.

[anutosh491]

Okay let's try it out .

Getting and using xtensor with xeus-cpp-lite would only require it to be a part of the host prefix (as header only)

1. So please fetch it from emscripten-forge and try a clean build

2. Then try supporting the xtensor usecase.

[kr-2003]

I have added xtensor tagfiles.

[anutosh491]

I have added xtensor tagfiles.

Ohh the xtensor based files are also being preloaded and not being picked up from the prefix correct ?

[kr-2003]

I have added xtensor tagfiles.

Ohh the xtensor based files are also being preloaded and not being picked up from the prefix correct ?

Yes

[anutosh491]

Hey @kr-2003 , thanks for the work here but as discussed I think we would have to keep the tagfiles in the prefix for jupyterlite-xeus to pick them up !

[vgvassilev]

I’d like these discussions to happen under the relevant issues and not in private.

[SylvainCorlay]

I have added xtensor tagfiles.

Ohh the xtensor based files are also being preloaded and not being picked up from the prefix correct ?

Yes

I agree that xeus-cpp should not include third-party tag files. Each third-party package should be responsible for including their tag files, which are then picked up by xeus-cpp.