implement sub-interpreters#380
implement sub-interpreters#380Vipul-Cariappa wants to merge 2 commits intocompiler-research:mainfrom
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| #include "xsystem.hpp" | ||
|
|
||
| #include "xmagics/multi_interpreter.hpp" | ||
|
|
There was a problem hiding this comment.
warning: included header multi_interpreter.hpp is not used directly [misc-include-cleaner]
| #include "multi_interpreter.hpp" | ||
|
|
||
| #include <iostream> | ||
| #include <iterator> |
There was a problem hiding this comment.
warning: included header iostream is not used directly [misc-include-cleaner]
| #include <iterator> | |
| #include <iterator> |
|
|
||
| #include <iostream> | ||
| #include <iterator> | ||
| #include <sstream> |
There was a problem hiding this comment.
warning: included header iterator is not used directly [misc-include-cleaner]
| #include <sstream> | |
| #include <sstream> |
| #include <sstream> | ||
| #include <string> | ||
| #include <vector> | ||
|
|
There was a problem hiding this comment.
warning: included header vector is not used directly [misc-include-cleaner]
|
|
||
| static std::vector<std::string> split(const std::string& input) | ||
| { | ||
| std::istringstream buffer(input); |
There was a problem hiding this comment.
warning: unused local variable 'buffer' of type 'std::istringstream' (aka 'basic_istringstream') [bugprone-unused-local-non-trivial-variable]
std::istringstream buffer(input);
^| return ret; | ||
| } | ||
|
|
||
| static constexpr size_t len(std::string_view n) |
There was a problem hiding this comment.
warning: no header providing "size_t" is directly included [misc-include-cleaner]
src/xmagics/multi_interpreter.cpp:10:
- #include <iostream>
+ #include <cstddef>
+ #include <iostream>| return ret; | ||
| } | ||
|
|
||
| static constexpr size_t len(std::string_view n) |
There was a problem hiding this comment.
warning: no header providing "std::string_view" is directly included [misc-include-cleaner]
src/xmagics/multi_interpreter.cpp:14:
- #include <vector>
+ #include <string_view>
+ #include <vector>| ************************************************************************************/ | ||
|
|
||
| #ifndef XEUS_CPP_MULTI_INTERPRETER_MAGIC_HPP | ||
| #define XEUS_CPP_MULTI_INTERPRETER_MAGIC_HPP |
There was a problem hiding this comment.
warning: header guard does not follow preferred style [llvm-header-guard]
| #define XEUS_CPP_MULTI_INTERPRETER_MAGIC_HPP | |
| #ifndef GITHUB_WORKSPACE_SRC_XMAGICS_MULTI_INTERPRETER_HPP | |
| #define GITHUB_WORKSPACE_SRC_XMAGICS_MULTI_INTERPRETER_HPP |
src/xmagics/multi_interpreter.hpp:32:
- #endif // XEUS_CPP_MULTI_INTERPRETER_MAGIC_HPP
+ #endif // GITHUB_WORKSPACE_SRC_XMAGICS_MULTI_INTERPRETER_HPP|
|
||
| #include <string> | ||
| #include <unordered_map> | ||
|
|
There was a problem hiding this comment.
warning: included header unordered_map is not used directly [misc-include-cleaner]
| #include <string> | ||
| #include <unordered_map> | ||
|
|
||
| #include "CppInterOp/CppInterOp.h" |
There was a problem hiding this comment.
warning: 'CppInterOp/CppInterOp.h' file not found [clang-diagnostic-error]
#include "CppInterOp/CppInterOp.h"
^
mcbarton
force-pushed
the
dev/multi-interp
branch
from
b20deed to
63271f0
Compare
| { | ||
| public: | ||
|
|
||
| XEUS_CPP_API |
There was a problem hiding this comment.
warning: no header providing "XEUS_CPP_API" is directly included [misc-include-cleaner]
src/xmagics/multi_interpreter.hpp:15:
- #include "xeus-cpp/xmagics.hpp"
+ #include "xeus-cpp/xeus_cpp_config.hpp"
+ #include "xeus-cpp/xmagics.hpp"|
|
||
| private: | ||
|
|
||
| std::unordered_map<std::string, Cpp::TInterp_t> interpreters; |
There was a problem hiding this comment.
warning: invalid case style for private member 'interpreters' [readability-identifier-naming]
| std::unordered_map<std::string, Cpp::TInterp_t> interpreters; | |
| std::unordered_map<std::string, Cpp::TInterp_t> m_interpreters; |
|
Please add some tests to this PR |
2 participants
Description
Implements a way to create sub-interpreter and use them with in a notebook session.
Fixes # (issue)
Type of change
Please tick all options which are relevant.