chore: get rid of ExecutorchModule, cleanup TypeConcepts

Author: chmjkb

packages/react-native-executorch/common/rnexecutorch/RnExecutorchInstaller.cpp

@@ -1,6 +1,5 @@
 #include "RnExecutorchInstaller.h"
 
-#include <rnexecutorch/bindings/ExecutorchModule.h>
 #include <rnexecutorch/host_objects/JsiConversions.h>
 #include <rnexecutorch/models/classification/Classification.h>
 #include <rnexecutorch/models/image_segmentation/ImageSegmentation.h>

packages/react-native-executorch/common/rnexecutorch/RnExecutorchInstaller.h

@@ -26,7 +26,7 @@ class RnExecutorchInstaller {
                     FetchUrlFunc_t fetchDataFromUrl);
 
 private:
-  template <UnloadableExternalMemoryAware ModelT>
+  template <typename ModelT>
   static jsi::Function
   loadModel(jsi::Runtime *jsiRuntime,
             std::shared_ptr<react::CallInvoker> jsCallInvoker,

packages/react-native-executorch/common/rnexecutorch/TypeConcepts.h

@@ -13,32 +13,4 @@ concept HasGenerate = requires(T t) {
   { &T::generate };
 };
 
-// When i make BaseModel have all of this methods we can then just check from
-// is_base_of instead of doing this for every single method
-template <typename T>
-concept HasForward = requires(T t) {
-  { &T::forward };
-};
-
-template <typename T>
-concept HasMethodNames = requires(T t) {
-  { &T::methodNames };
-};
-
-template <typename T>
-concept HasGetInputShape = requires(T t) {
-  { &T::getInputShape };
-};
-
-template <typename T>
-concept HasIsLoaded = requires(T t) {
-  { &T::isLoaded };
-};
-
-template <typename T>
-concept UnloadableExternalMemoryAware = requires(T t) {
-  { t.unload() } -> std::same_as<void>;
-  { t.getMemoryLowerBound() } -> std::same_as<std::size_t>;
-};
-
 } // namespace rnexecutorch